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

Implement generic, extensible pre-flight check functionality based on the new Helm direction #741

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

Now that we have decided to shift away from building on top of the Carvel tool suite, we are going to have to roll our own preflight check implementation. Ideally this is a generic, extensible implementation that allows us to add additional pre-flight checks in the future.

Pre-flight checks should also have available to them the manifests which would be applied if the pre-flight checks (and potential later validations) are successful. This is particularly useful for the CRD Upgrade Safety pre-flight check because without this information we wouldn't know how the CRDs are changing.

@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