-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Upgrade SDK to use Kubernetes 1.26.2 #6352
Upgrade SDK to use Kubernetes 1.26.2 #6352
Conversation
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.
Todo for myself: Add changelog :)
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.
Overall the changes look good to me - great work @rashmigottipati!
The only glaring thing for me is pulling in Kubebuilder 3.9.0 which has a known bug that could break the scaffolding of Go operators.
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.
Just two changes, rest looks good! Thanks for working on this @rashmigottipati :)
/lgtm
Signed-off-by: rashmigottipati <chowdary.grashmi@gmail.com>
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.
/lgtm
Regarding the upcoming major changes in the plugin, please note that `go/v3` is being deprecated by Kubebuilder hence Operator SDK would also be migrating to `go/v4` soon. | ||
Following are the changes to be addressed in your project structure, `Makefile` and `go.mod` files: | ||
1) [go/v3, ansible/v1, helm/v1] Update your `go.mod` file to upgrade the dependencies and run `go mod tidy` to download them | ||
```go | ||
k8s.io/api v0.26.2 | ||
k8s.io/apiextensions-apiserver v0.26.2 | ||
k8s.io/apimachinery v0.26.2 | ||
k8s.io/cli-runtime v0.26.2 | ||
k8s.io/client-go v0.26.2 | ||
k8s.io/kubectl v0.26.2 | ||
sigs.k8s.io/controller-runtime v0.14.5 | ||
sigs.k8s.io/controller-tools v0.11.3 | ||
sigs.k8s.io/kubebuilder/v3 v3.9.1 | ||
``` | ||
2) [go/v3, ansible/v1, helm/v1] With the bump to Kubebuilder 3.9.1, the `go/v3` projects are now scaffolded with 0.13.1 version of kube-rbac-proxy. Modify the version of kube-rbac-proxy in the following scaffolded files: | ||
- config/default/manager_auth_proxy_patch.yaml | ||
- bundle/manifests/memcached-operator.clusterserviceversion.yaml |
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.
@rashmigottipati This isn't rendering as expected in the docs https://sdk.operatorframework.io/docs/upgrading-sdk-version/v1.28.0/#upgrade-k8s-versions-to-use-126
Description of the change:
go.mod
andgo.sum
resulted from bumping versions to support k8s 1.26.2sets.String
is used, which has been deprecatedMotivation for the change:
Support for k8s 1.26
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs