-
Notifications
You must be signed in to change notification settings - Fork 89
add windows build #785
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
Open
mcarbonneaux
wants to merge
35
commits into
openconfig:main
Choose a base branch
from
mcarbonneaux:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
add windows build #785
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
8cea597
Merge pull request #748 from hyposcaler-bot/hyposcaler/issue-747
karimra 052515d
Merge pull request #746 from nokia/refactor-interfaces
karimra 3d3587b
Bump golang.org/x/crypto from 0.41.0 to 0.42.0
dependabot[bot] 4fed1b2
Merge pull request #734 from openconfig/dependabot/go_modules/golang.…
karimra 1a8ee6f
handle `[null]` and `null` when format is `event`
karimra b6933bc
add missing processor name to proc list
karimra 7a1995e
fail xpath parsing when there are trailing chars after a key and befo…
karimra 07786fa
Merge pull request #754 from openconfig/fix753
karimra 0314bda
Merge pull request #755 from openconfig/reorg-proc-list
karimra 938d926
Merge pull request #756 from openconfig/fix751
karimra 5c8542a
bump api pkg go version and main pkg dependencies
karimra a387236
Merge pull request #757 from openconfig/bump-api-deps
karimra 65d4abe
feat: capability for jetstream input to set MaxAckPending
marco-minervino 1e57a5f
Fix leading slash handling for flat format
earies e6f5eef
Merge pull request #758 from marco-minervino/feat-maxackpending-input…
karimra 0514e8c
fix event_group_by not handling delete events
alkshmir d6839c2
fix #770 The API path to patch subscriptions for a target ID is docum…
ashvanth48t 107dc97
Bump golangci/golangci-lint-action from 8 to 9
dependabot[bot] 0686dee
Address karima comments
earies add67c2
Handle prefix without origin
earies 4b93215
Bump golang.org/x/crypto from 0.43.0 to 0.45.0
dependabot[bot] fc37d57
Merge pull request #771 from ashvanth48t/main
karimra 8d3249f
Merge pull request #769 from alkshmir/main
karimra 2db8319
Merge pull request #773 from openconfig/dependabot/github_actions/gol…
karimra 1172391
Merge pull request #760 from earies/flat-root
karimra 055b727
Merge pull request #776 from openconfig/dependabot/go_modules/golang.…
karimra 1f85795
add windows build
mcarbonneaux 55003e6
change github.com/c-bata/go-prompt to https://github.com/elk-language…
mcarbonneaux 21b7995
add gnmic.exe to .gitignore
mcarbonneaux d28cf36
ignore arm arch for windows
mcarbonneaux d8c5d6f
add cache and fail fast to debug windows test (without macos fail)
mcarbonneaux 691114a
to remove Warning: Input 'fail_on_error' has been deprecated with mes…
mcarbonneaux 2dad991
try to fix staticcheck fail on macos and windows by updating reviewdog
mcarbonneaux ae82099
try to use version in place on hash to try to fix
mcarbonneaux b3ec6f3
remove staticcheck on windows/macos only one is needed
mcarbonneaux File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,26 +16,37 @@ on: | |
| - "main" | ||
| - "!releases/**" | ||
| env: | ||
| GOVER: 1.24.7 | ||
| GOVER: 1.24.x | ||
|
|
||
| jobs: | ||
| test: | ||
| runs-on: ubuntu-22.04 | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| os: [ubuntu-22.04, macos-latest, windows-latest] | ||
| runs-on: ${{ matrix.os }} | ||
| steps: | ||
| - uses: actions/checkout@v5 | ||
| - uses: actions/setup-go@v6 | ||
| with: | ||
| go-version: ${{ env.GOVER }} | ||
| cache: true | ||
| - run: ./tests/run_tests.sh | ||
| if: runner.os != 'Windows' | ||
| env: | ||
| CGO_ENABLED: 0 | ||
| - run: bash ./tests/run_tests.sh | ||
| if: runner.os == 'Windows' | ||
| env: | ||
| CGO_ENABLED: 0 | ||
| # run staticcheck | ||
| - uses: reviewdog/action-staticcheck@73cfd0daa6fdbba9a858dcb0f62844012fa8317d | ||
| - uses: reviewdog/[email protected] | ||
| if: runner.os == 'Linux' | ||
| with: | ||
| github_token: ${{ secrets.GITHUB_TOKEN }} | ||
| # Change reviewdog reporter if you need [github-pr-check,github-check,github-pr-review]. | ||
| reporter: github-pr-review | ||
| # Report all results. | ||
| filter_mode: nofilter | ||
| # Exit with 1 when it find at least one finding. | ||
| fail_on_error: true | ||
| fail_level: error | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,6 +6,7 @@ builds/ | |
| dist | ||
| *.log | ||
| gnmic | ||
| gnmic.exe | ||
| *.tmp | ||
| *.work* | ||
| .idea | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
36 changes: 36 additions & 0 deletions
36
examples/deployments/1.single-instance/11.kafka-kraft-output/containerlab/gnmic.yaml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,36 @@ | ||
| # © 2022 Nokia. | ||
| # | ||
| # This code is a Contribution to the gNMIc project (“Work”) made under the Google Software Grant and Corporate Contributor License Agreement (“CLA”) and governed by the Apache License 2.0. | ||
| # No other rights or licenses in or to any of Nokia’s intellectual property are granted for any other purpose. | ||
| # This code is provided on an “as is” basis without any warranties of any kind. | ||
| # | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| username: admin | ||
| password: NokiaSrl1! | ||
| skip-verify: true | ||
| encoding: json_ietf | ||
| log: true | ||
|
|
||
| # docker target loader | ||
| loader: | ||
| type: docker | ||
| filters: | ||
| - containers: | ||
| - label=clab-node-kind: nokia_srlinux | ||
| label=containerlab: lab12 | ||
|
|
||
| subscriptions: | ||
| # Add subscriptions configuration here | ||
| # e.g: | ||
| sub1: | ||
| paths: | ||
| - /interface/statistics | ||
| stream-mode: sample | ||
| sample-interval: 10s | ||
|
|
||
| outputs: | ||
| kafka-output: | ||
| type: kafka | ||
| address: clab-lab12-kafka-server:9092 | ||
| topic: telemetry |
62 changes: 62 additions & 0 deletions
62
examples/deployments/1.single-instance/11.kafka-kraft-output/containerlab/kafka.clab.yaml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,62 @@ | ||
| # © 2022 Nokia. | ||
| # | ||
| # This code is a Contribution to the gNMIc project (“Work”) made under the Google Software Grant and Corporate Contributor License Agreement (“CLA”) and governed by the Apache License 2.0. | ||
| # No other rights or licenses in or to any of Nokia’s intellectual property are granted for any other purpose. | ||
| # This code is provided on an “as is” basis without any warranties of any kind. | ||
| # | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| name: lab12 | ||
|
|
||
| topology: | ||
| defaults: | ||
| kind: nokia_srlinux | ||
|
|
||
| kinds: | ||
| nokia_srlinux: | ||
| image: ghcr.io/nokia/srlinux | ||
|
|
||
| nodes: | ||
| spine1: | ||
| spine2: | ||
| leaf1: | ||
| leaf2: | ||
| leaf3: | ||
| leaf4: | ||
|
|
||
| gnmic: | ||
| kind: linux | ||
| image: ghcr.io/openconfig/gnmic:latest | ||
| binds: | ||
| - ./gnmic.yaml:/app/gnmic.yaml:ro | ||
| - /var/run/docker.sock:/var/run/docker.sock | ||
| cmd: '--config /app/gnmic.yaml --log subscribe' | ||
|
|
||
| kafka-server: | ||
| kind: linux | ||
| image: bitnami/kafka:latest | ||
| ports: | ||
| - 9092:9092 | ||
| - 9093:9093 | ||
| env: | ||
| KAFKA_ENABLE_KRAFT: yes | ||
| ALLOW_PLAINTEXT_LISTENER: yes | ||
| KAFKA_CFG_NODE_ID: 0 | ||
| KAFKA_CFG_PROCESS_ROLES: broker,controller | ||
| KAFKA_CFG_CONTROLLER_LISTENER_NAMES: CONTROLLER | ||
| KAFKA_CFG_LISTENERS: PLAINTEXT://:9092,CONTROLLER://:9093 | ||
| KAFKA_CFG_LISTENER_SECURITY_PROTOCOL_MAP: CONTROLLER:PLAINTEXT,PLAINTEXT:PLAINTEXT | ||
| KAFKA_CFG_ADVERTISED_LISTENERS: PLAINTEXT://clab-lab12-kafka-server:9092 | ||
| KAFKA_CFG_CONTROLLER_QUORUM_VOTERS: 0@clab-lab12-kafka-server:9093 | ||
|
|
||
| links: | ||
| # spine1 links | ||
| - endpoints: ["spine1:e1-1", "leaf1:e1-1"] | ||
| - endpoints: ["spine1:e1-2", "leaf2:e1-1"] | ||
| - endpoints: ["spine1:e1-3", "leaf3:e1-1"] | ||
| - endpoints: ["spine1:e1-4", "leaf4:e1-1"] | ||
| # spine2 links | ||
| - endpoints: ["spine2:e1-1", "leaf1:e1-2"] | ||
| - endpoints: ["spine2:e1-2", "leaf2:e1-2"] | ||
| - endpoints: ["spine2:e1-3", "leaf3:e1-2"] | ||
| - endpoints: ["spine2:e1-4", "leaf4:e1-2"] |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this wouldn't work right ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
windows runner are build with bash but you need to call explicitly to use it.