-
Notifications
You must be signed in to change notification settings - Fork 3.4k
vendor: replace 2 unmaintained dependencies with their blessed forks #30028
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
Conversation
6c2a2b5
to
1c7c2d5
Compare
/test |
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.
The changes are looking innocent enough, however integration test failure is due to the recent upstream patch
level=error msg="Invoke failed" ="tables_test.fixture.func1 (pkg/datapath/tables/node_address_test.go:460)" error="could not build arguments for function \"github.com/cilium/cilium/pkg/datapath/tables_test\".fixture.func1 (/home/runner/work/cilium/cilium/pkg/datapath/tables/node_address_test.go:460): failed to build statedb.Table[github.com/cilium/cilium/pkg/datapath/tables.NodeAddress]: could not build arguments for function \"github.com/cilium/cilium/pkg/datapath/tables\".newNodeAddressController (/home/runner/work/cilium/cilium/pkg/datapath/tables/node_address.go:195): failed to build tables.NodeAddressConfig: received non-nil error from function \"github.com/cilium/cilium/pkg/hive/cell\".(*config[...]).Apply.func2 (/home/runner/work/cilium/cilium/pkg/hive/cell/config.go:154): failed to unmarshal config struct tables.NodeAddressConfig: 1 error(s) decoding:\n\n* error decoding 'nodeport-addresses[0]': invalid CIDR address: 2001::/16,10.0.0.0/8.\nHint: field 'FooBar' matches flag 'foo-bar', or use tag `mapstructure:\"flag-name\"` to match field with flag" subsys=hive
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.
Changes LGTM, but failures seem related as Tam already said.
1c7c2d5
to
4edb59b
Compare
This pull request has been automatically marked as stale because it |
Mitchell Hashimoto has announced that he'll be archiving his Go repositories in early 2024. Switch mapstructure to using his "blessed fork". Ref: https://gist.github.com/mitchellh/90029601268e59a29e64e55bab1c5bdc Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
Mitchell Hashimoto has announced that he'll be archiving his Go repositories in early 2024. Switch to using the "blessed fork" for protoc-gen-go-json. Ref: https://gist.github.com/mitchellh/90029601268e59a29e64e55bab1c5bdc Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
4edb59b
to
7c35d02
Compare
Signed-off-by: Cilium Imagebot <noreply@cilium.io>
This pull request has been automatically marked as stale because it |
This pull request has not seen any activity since it was marked stale. |
Replaced by #32739 |
Mitchell Hashimoto has announced that he'll be archiving his Go repositories in early 2024.
This PR updates 2 of these dependencies with their blessed forks.