Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dav/pr to official #5

Draft
wants to merge 33 commits into
base: dav/man
Choose a base branch
from
Draft

Dav/pr to official #5

wants to merge 33 commits into from

Conversation

davnerson-dn
Copy link

No description provided.

anjan-keysight and others added 7 commits January 4, 2024 15:33
…g#476)

Install net-tools (ifconfig), iptables, nftables (nft).
…ig#485)

* juniper cptx, ncptx allow inside gRPC port other than 32767

This change helps allow users change inside port other than 32767.
If user wants to configure a port other than default 32767, they
need to modify the gnmi service in KNE protobuf config. In the absense
of config the default 32767 will be used. Both reset to factory and cert
generation will make sure configured port is used.

* juniper cptx, ncptx allow inside gRPC port other than 32767

This change helps allow users change inside port other than 32767.
If user wants to configure a port other than default 32767,
they need to modify the gnmi service in KNE protobuf config.
In the absense of config the default 32767 will be used.

removes some unnessary changes from previous commit and looks up for
gnmi service in the map of services to find the inside port.
…te easier for users (openconfig#484)

* Initial commit handling defaults in vendor node to make KNE topo create easier for users

* Simple topo example

* Resolve comments

---------

Co-authored-by: Neha Manjunath <[email protected]>
* add registry package

* add registry to deploy

* switch to refresher

* get refresh working

* Refactor in packages

* remove leftover file

* cleanup tests and imports

* fix lint

* linter

* linter

* add more tests

* finish unit tests

* linter after merge

* address comments
@davnerson-dn davnerson-dn force-pushed the dav/pr_to_official branch 4 times, most recently from df047c3 to 65faf27 Compare January 30, 2024 09:38
@davnerson-dn davnerson-dn force-pushed the dav/pr_to_official branch 7 times, most recently from 295bcdd to c42a592 Compare February 5, 2024 08:53
@davnerson-dn davnerson-dn force-pushed the dav/pr_to_official branch 2 times, most recently from 02b5194 to 9f15188 Compare February 7, 2024 13:18
NehaManjunath and others added 2 commits February 12, 2024 13:12
* Use names in topo proto to create Services

* Add unit tests

* Remove debug

* Resolve comments

* Add tests with empty strings

* Fix issues in testing

* Incorporate comments

* Incorporate comments

* Incorporate comments

* Incorporate comments

* Fix tests

* Fix tests

* Incorporate comments

* Revert "Incorporate comments"

This reverts commit 412c0e0.

* Add UT

---------

Co-authored-by: Neha Manjunath <[email protected]>
dependabot bot and others added 5 commits February 15, 2024 16:24
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.14.0 to 0.17.0.
- [Commits](golang/crypto@v0.14.0...v0.17.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Revert "Bump golang.org/x/crypto from 0.14.0 to 0.17.0 (openconfig#474)"

This reverts commit 6b835bc.
* add better semver parse

* go mod tidy

* fix test

* add more tests

* fix edge case with uint wrap

* lint
* work towards cptx deprecation

* fix test

* fix vendors test

* fix bad default for arista
* update vendor service defaults and add tests

* fix tests and fmt

* cleanup vendors test presub to test conn only
@davnerson-dn davnerson-dn force-pushed the dav/pr_to_official branch 2 times, most recently from 3e0872a to 7686d5f Compare March 11, 2024 13:17
dependabot bot and others added 2 commits March 15, 2024 10:48
Bumps google.golang.org/protobuf from 1.32.0 to 1.33.0.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* rollforward deps to fix postsubmit

* fix typo

* add gopath

* fix
@davnerson-dn davnerson-dn force-pushed the dav/pr_to_official branch 2 times, most recently from bb91e6e to 4677d27 Compare March 17, 2024 16:09
@davnerson-dn davnerson-dn force-pushed the dav/pr_to_official branch 3 times, most recently from e0aa86d to 8a3329b Compare March 17, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants