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

errors related to controller-runtime v0.11 upgrade #384

Closed
tchughesiv opened this issue Jan 13, 2022 · 0 comments · Fixed by #385
Closed

errors related to controller-runtime v0.11 upgrade #384

tchughesiv opened this issue Jan 13, 2022 · 0 comments · Fixed by #385

Comments

@tchughesiv
Copy link
Contributor

What did you do to encounter the bug?
run unit-test or vet in main branch

What did you expect?
successful tests

What happened instead?

$ make unit-test
go test -race -cover ./pkg/...
# sigs.k8s.io/json/internal/golang/encoding/json
../../go/pkg/mod/sigs.k8s.io/json@v0.0.0-20211020170558-c049b76a60c6/internal/golang/encoding/json/encode.go:1249:12: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
../../go/pkg/mod/sigs.k8s.io/json@v0.0.0-20211020170558-c049b76a60c6/internal/golang/encoding/json/encode.go:1255:18: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
FAIL	github.com/mongodb/mongodb-atlas-kubernetes/pkg/api/v1 [build failed]
?   	github.com/mongodb/mongodb-atlas-kubernetes/pkg/api/v1/project	[no test files]
?   	github.com/mongodb/mongodb-atlas-kubernetes/pkg/api/v1/provider	[no test files]
FAIL	github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlas [build failed]
FAIL	github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlascluster [build failed]
FAIL	github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlasdatabaseuser [build failed]

Additional context
appears to require go1.17
kubernetes-sigs/controller-tools#643
https://github.com/kubernetes-sigs/controller-runtime/blob/1f06556c918de094e1060e54bd73a2d70ba9b544/go.mod#L3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants