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

CLOUDP-238915: Move to operator-sdk v1.34.1 #1473

Merged
merged 3 commits into from
Mar 27, 2024

Conversation

helderjs
Copy link
Collaborator

All Submissions:

  • Have you signed our CLA?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
  • Update docs/release-notes/release-notes.md if your changes should be included in the release notes for the next release.

I'm anticipating this task once I'd like to have it to be able to ship some code with generics at #1466

As I have been using operator-sdk in the latest version for a while locally, I checked what was necessary to migrate/update it and it seems to be as simple as this PR shows. No changes in base manifests.
The next release, bundle will reflect the update of controller gen in the manifests and the new project layout, go.kubebuilder.io/v4 in the CSV:

The warning triggered was because PROJECT config had v1beta1 API set for backup CRDs (not sure why) but both CRDs were already in v1(along with all others) and didn't have changes in them.

@helderjs helderjs added the cloud-tests Run expensive Cloud Tests: Integration & E2E label Mar 25, 2024
Copy link
Contributor

github-actions bot commented Mar 25, 2024

@helderjs helderjs force-pushed the CLOUDP-238915-update-operator-sdk branch from de6646d to 98782b9 Compare March 27, 2024 09:29
@helderjs helderjs merged commit 3bcfce3 into main Mar 27, 2024
47 checks passed
@helderjs helderjs deleted the CLOUDP-238915-update-operator-sdk branch March 27, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cloud-tests Run expensive Cloud Tests: Integration & E2E
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants