-
Notifications
You must be signed in to change notification settings - Fork 61
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 to operator-sdk v0.16.2 #100
Upgrade to operator-sdk v0.16.2 #100
Conversation
Followed upgrade guide at: https://sdk.operatorframework.io/docs/migration/version-upgrade-guide/#v016x The reason I didn't go all the way to the latest 0.18.z is that after 0.16.z there were dependency conflicts that might best be solved by either updating dependencies in openshift/operator-custom-metrics, or by removing that as a dependency from this operator.
/cc @RiRa12621 ^since this was suggested at https://github.com/openshift/pagerduty-operator/pull/90/files#r448467970 |
/ok-to-test |
/lgtm |
Discussed with the team and we agree this is a Good Thing. But not on a Friday. I'm on PTO next week but will approve and put on hold for deployment and testing in int/stage (side effect of merging this) next week. /hold |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: grdryn, jewzaam, RiRa12621 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@jewzaam We could just merge this as it's going int and stage only without the promotion to prod, shouldn't be a big deal even on Friday or weekend |
/unhold |
Thanks @RiRa12621, I'll rebase my other PR now on top of the new master, which should make it much smaller. 👍 |
Followed upgrade guide at:
https://sdk.operatorframework.io/docs/migration/version-upgrade-guide/#v016x
The reason I didn't go all the way to the latest 0.18.z is that after
0.16.z there were dependency conflicts that might best be solved by
either updating dependencies in openshift/operator-custom-metrics, or
by removing that as a dependency from this operator.