Skip to content

Conversation

rolinh
Copy link
Member

@rolinh rolinh commented Dec 21, 2023

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.

@rolinh rolinh added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.15 labels Dec 21, 2023
@rolinh rolinh force-pushed the pr/rolinh/mitchellh branch from 6c2a2b5 to 1c7c2d5 Compare December 21, 2023 13:15
@rolinh rolinh temporarily deployed to release-base-images December 21, 2023 13:15 — with GitHub Actions Inactive
@rolinh
Copy link
Member Author

rolinh commented Dec 21, 2023

/test

@rolinh rolinh marked this pull request as ready for review December 21, 2023 13:25
@rolinh rolinh requested review from a team as code owners December 21, 2023 13:25
Copy link
Member

@sayboras sayboras left a 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

go-viper/mapstructure#6

 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

@pippolo84 pippolo84 self-requested a review December 22, 2023 17:30
Copy link
Member

@pippolo84 pippolo84 left a 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.

@rolinh rolinh marked this pull request as draft January 24, 2024 15:40
@rolinh rolinh force-pushed the pr/rolinh/mitchellh branch from 1c7c2d5 to 4edb59b Compare January 24, 2024 15:41
@rolinh rolinh had a problem deploying to release-base-images January 24, 2024 15:41 — with GitHub Actions Failure
Copy link

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Feb 24, 2024
rolinh added 3 commits March 4, 2024 17:34
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>
@rolinh rolinh force-pushed the pr/rolinh/mitchellh branch from 4edb59b to 7c35d02 Compare March 4, 2024 16:37
@rolinh rolinh temporarily deployed to release-base-images March 4, 2024 16:37 — with GitHub Actions Inactive
@rolinh rolinh removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Mar 4, 2024
Signed-off-by: Cilium Imagebot <noreply@cilium.io>
@auto-committer auto-committer bot temporarily deployed to release-base-images March 4, 2024 17:00 Inactive
Copy link

github-actions bot commented Apr 4, 2024

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Apr 4, 2024
Copy link

This pull request has not seen any activity since it was marked stale.
Closing.

@github-actions github-actions bot closed this Apr 19, 2024
@rolinh
Copy link
Member Author

rolinh commented May 28, 2024

Replaced by #32739

@rolinh rolinh deleted the pr/rolinh/mitchellh branch May 28, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants