Skip to content

Conversation

taylorsilva
Copy link
Member

This reverts commit da8e4b2.

When running go build -o concourse ./cmd/concourse we'd get this error:

# k8s.io/apimachinery/pkg/util/managedfields/internal
../../../go/pkg/mod/k8s.io/apimachinery@v0.30.0/pkg/util/managedfields/internal/structuredmerge.go:56:4: unknown field IgnoredFields in struct literal of type merge.Updater
../../../go/pkg/mod/k8s.io/apimachinery@v0.30.0/pkg/util/managedfields/internal/structuredmerge.go:73:4: unknown field IgnoredFields in struct literal of type merge.Updater

Seems like go isn't following the dependency path correctly. k8s.io/apimachinery@v0.30.0 uses sigs.k8s.io/structured-merge-diff/v4@v4.4.1 in it's go.mod but it appears when building go decides to use v4.4.2 of that package where the IgnoredFields field is gone. Unblocking pipeline by reverting this deps bump for now.

@taylorsilva taylorsilva added the release/undocumented This didn't warrant being documented or put in release notes. label Nov 5, 2024
@taylorsilva taylorsilva requested a review from a team as a code owner November 5, 2024 14:44
This reverts commit da8e4b2.

Signed-off-by: Taylor Silva <dev@taydev.net>
@taylorsilva taylorsilva merged commit 754cc99 into master Nov 5, 2024
3 checks passed
@taylorsilva taylorsilva deleted the revert-go-deps-bump branch November 5, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
misc release/undocumented This didn't warrant being documented or put in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant