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

Move operator-sdk to v1.33.0 #33

Merged
merged 2 commits into from
Dec 22, 2023
Merged

Move operator-sdk to v1.33.0 #33

merged 2 commits into from
Dec 22, 2023

Conversation

ramperher
Copy link
Collaborator

@ramperher ramperher commented Dec 21, 2023

Consequently, also bump kustomize version to the version used by operator-sdk v1.33.0, as shown in this PR

@ramperher
Copy link
Collaborator Author

check dallas ocp-4.13-vanilla example-cnf

Copy link
Collaborator

@tkrishtop tkrishtop left a comment

Choose a reason for hiding this comment

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

Hi @ramperher I think we need 3.8.7 for the kustomized as well, when moving to operator-sdk 1.33.0. I checked fast, it seems like to be the case already, so looks good

@ramperher
Copy link
Collaborator Author

Hi @ramperher I think we need 3.8.7 for the kustomized as well, when moving to operator-sdk 1.33.0. I checked fast, it seems like to be the case already, so looks good

I can change that, curiously we have one operator using 3.8.7 but the others using 3.5.x. I can bump that too

@ramperher
Copy link
Collaborator Author

ramperher commented Dec 21, 2023

@tkrishtop , in fact, I've found some interesting info related to kustomize in the v1.33.0 release.

If you go to the release notes, it highlights the following change:

Breaking change: (kustomize/v2-alpha): kustomize/v2 is now stable and is the default version used in the plugin chain when using go/v4, ansible/v1, helm/v1, and hybrid/v1-alpha plugins. (operator-framework/operator-sdk#6613)

By accessing to that PR, we can see the following info:

(kustomize/v2): kustomize/v2 is now stable and is the default version used in the plugin chain when using go/v4, ansible/v1, helm/v1, and hybrid/v1-alpha plugins
...
fixes operator-framework/operator-sdk#4975 (kustomize/v2 plugin uses v5.1.1)

So, my impression is that operator-sdk v1.33.0 then supports kustomize v5.1.1? Should we move to that version?

@tonyskapunk
Copy link
Collaborator

@tkrishtop , in fact, I've found some interesting info related to kustomize in the v1.33.0 release.

If you go to the release notes, it highlights the following change:

Breaking change: (kustomize/v2-alpha): kustomize/v2 is now stable and is the default version used in the plugin chain when using go/v4, ansible/v1, helm/v1, and hybrid/v1-alpha plugins. (operator-framework/operator-sdk#6613)

By accessing to that PR, we can see the following info:

(kustomize/v2): kustomize/v2 is now stable and is the default version used in the plugin chain when using go/v4, ansible/v1, helm/v1, and hybrid/v1-alpha plugins ... fixes operator-framework/operator-sdk#4975 (kustomize/v2 plugin uses v5.1.1)

So, my impression is that operator-sdk v1.33.0 then supports kustomize v5.1.1? Should we move to that version?

You want to look at the upgrading doc for v1.33.0 instead https://sdk.operatorframework.io/docs/upgrading-sdk-version/v1.33.0/

There is also a reference to kustomizev2, that explains that in some cases is should be used v4 and in others v5 of kustomize. The references made in there are for v5.0.0, but if you create an operator (golang based) from scratch, you will see it is using v5.0.1. So most likely you want to upgrade to v5.0.1 and not v5.1.1

❯ operator-sdk version
operator-sdk version: "v1.33.0", commit: "542966812906456a8d67cf7284fc6410b104e118", kubernetes version: "1.27.0", go version: "go1.21.5", GOOS: "linux", GOARCH: "amd64"

❯ operator-sdk init --domain example.com --repo example.com 2>/dev/null 2>&1 && grep ^KUSTOMIZE_VERSION Makefile
Writing kustomize manifests for you to edit...
Writing scaffold for you to edit...
Get controller runtime:
$ go get sigs.k8s.io/controller-runtime@v0.15.0
Update dependencies:
$ go mod tidy
Next: define a resource with:
$ operator-sdk create api
KUSTOMIZE_VERSION ?= v5.0.1

@ramperher
Copy link
Collaborator Author

ramperher commented Dec 22, 2023

@tkrishtop , in fact, I've found some interesting info related to kustomize in the v1.33.0 release.
If you go to the release notes, it highlights the following change:
Breaking change: (kustomize/v2-alpha): kustomize/v2 is now stable and is the default version used in the plugin chain when using go/v4, ansible/v1, helm/v1, and hybrid/v1-alpha plugins. (operator-framework/operator-sdk#6613)
By accessing to that PR, we can see the following info:
(kustomize/v2): kustomize/v2 is now stable and is the default version used in the plugin chain when using go/v4, ansible/v1, helm/v1, and hybrid/v1-alpha plugins ... fixes operator-framework/operator-sdk#4975 (kustomize/v2 plugin uses v5.1.1)
So, my impression is that operator-sdk v1.33.0 then supports kustomize v5.1.1? Should we move to that version?

You want to look at the upgrading doc for v1.33.0 instead https://sdk.operatorframework.io/docs/upgrading-sdk-version/v1.33.0/

There is also a reference to kustomizev2, that explains that in some cases is should be used v4 and in others v5 of kustomize. The references made in there are for v5.0.0, but if you create an operator (golang based) from scratch, you will see it is using v5.0.1. So most likely you want to upgrade to v5.0.1 and not v5.1.1

❯ operator-sdk version
operator-sdk version: "v1.33.0", commit: "542966812906456a8d67cf7284fc6410b104e118", kubernetes version: "1.27.0", go version: "go1.21.5", GOOS: "linux", GOARCH: "amd64"

❯ operator-sdk init --domain example.com --repo example.com 2>/dev/null 2>&1 && grep ^KUSTOMIZE_VERSION Makefile
Writing kustomize manifests for you to edit...
Writing scaffold for you to edit...
Get controller runtime:
$ go get sigs.k8s.io/controller-runtime@v0.15.0
Update dependencies:
$ go mod tidy
Next: define a resource with:
$ operator-sdk create api
KUSTOMIZE_VERSION ?= v5.0.1

Thanks @tonyskapunk for all the information, as usual :) it makes sense to me, so yes, let's do that

@ramperher
Copy link
Collaborator Author

check dallas ocp-4.13-vanilla example-cnf

1 similar comment
@ramperher
Copy link
Collaborator Author

check dallas ocp-4.13-vanilla example-cnf

@ramperher
Copy link
Collaborator Author

Correct job tracking this installation is this one, and for the moment is working, it's in preflight testing: https://www.distributed-ci.io/jobs/a9decd42-0206-4a73-920c-e4c025a78996/jobStates?sort=date

@tkrishtop tkrishtop merged commit f14a236 into main Dec 22, 2023
1 check passed
@tkrishtop tkrishtop deleted the operator-sdk-v1.33 branch December 22, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants