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

chore: Bump golang to 1.17 (PROJQUAY-2781) #625

Merged
merged 1 commit into from
Jan 17, 2022

Conversation

harishsurf
Copy link
Contributor

go build main.go is failing with:

#14 3.972 # sigs.k8s.io/json/internal/golang/encoding/json
#14 3.972 vendor/sigs.k8s.io/json/internal/golang/encoding/json/encode.go:1249:12: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
#14 3.972 vendor/sigs.k8s.io/json/internal/golang/encoding/json/encode.go:1255:18: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)

causing our CI to fail. The issue is fixed by bumping golang version to 1.17.
See here for details

Signed-off-by: harishsurf hgovinda@redhat.com

Copy link
Contributor

@ricardomaraschini ricardomaraschini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You gonna have to run "go mod tidy" before sending up this PR (otherwise the CI is going to fail). Could you assign this PR to https://issues.redhat.com/browse/PROJQUAY-2781 ? Make sure that the Jira card points to 3.7.0

@harishsurf harishsurf force-pushed the bump-go branch 2 times, most recently from 797aa59 to dd692a1 Compare January 14, 2022 16:06
@harishsurf harishsurf changed the title chore: Bump golang to 1.17 chore: Bump golang to 1.17 (PROJQUAY-2781) Jan 14, 2022
Fixes failing "go build" since the reflect pkg was updated
to include isExported() with go v1.17

Signed-off-by: harishsurf <hgovinda@redhat.com>
@ricardomaraschini ricardomaraschini merged commit fcdab44 into quay:master Jan 17, 2022
@harishsurf harishsurf deleted the bump-go branch January 18, 2022 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants