-
Notifications
You must be signed in to change notification settings - Fork 381
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
Update the operator-sdk to the newest version #565
Conversation
- removes marked passages - changes build of operator image to the usage of makefile
- updates scorecard - updated label for metrics builder - updates dockerfile
Thanks for taking the time to address this! great work, I'll take a look at reviewing and verifying this later on today! 👍 @Voigtus |
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.
Looks good to me, tested it out on minikube, looks like things are being reconciled properly, there isn't a lot of functional logic that was changed between the versions, which is also a good sign.
Thanks for contributing!
@pb82 @NissesSenap let's coordinate on how and when to merge this
@Voigtus can you rebase this PR? The reason why we haven't merged it was that we wanted to make sure that we didn't get any issues with the 4.0.1 release. Thanks once again |
Yes yes, I can |
# Conflicts: # Makefile
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.
Re-approving to enable CI, LGTM
Description
This PR updates the whole operator from operator-sdk v1.3.0 up to the current version (v1.13.1)
Relevant issues/tickets
#545
Type of change
Update of operator-sdk
Verification steps
Spun up a kind cluster and created the examples from the deploy/examples folder
Disclaimer
I have not touched the openshift-ci folder, since I cant verify the results there.