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

Create CRDUpgradeSafety pre-flight check that re-uses the CRD change validation logic contributed to carvel-dev/kapp #742

Closed
everettraven opened this issue Apr 4, 2024 · 1 comment
Assignees
Labels
epic/crd-upgrade-safety Marks an issue as part of the CRD Upgrade Safety epic

Comments

@everettraven
Copy link
Contributor

everettraven commented Apr 4, 2024

This will involve writing a pre-flight check that follows the pattern introduced by #741 . This will also need to include a transplant of the validation logic we contributed to carvel-dev/kapp and adhere to any of the conditions of their licensing to take that logic and put it into an internal operator-controller package.

@everettraven everettraven added the epic/crd-upgrade-safety Marks an issue as part of the CRD Upgrade Safety epic label Apr 4, 2024
@everettraven everettraven self-assigned this May 23, 2024
@everettraven
Copy link
Contributor Author

operator-framework/rukpak#873 merged. This added the functionality into rukpak. Work to transplant this logic into operator controller is blocked by #846 and will be tracked by #877

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic/crd-upgrade-safety Marks an issue as part of the CRD Upgrade Safety epic
Projects
Status: Done
Development

No branches or pull requests

1 participant