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

Generic implementation of CRD upgrade preflight check #661

Closed
Tracked by #657
everettraven opened this issue Feb 27, 2024 · 1 comment
Closed
Tracked by #657

Generic implementation of CRD upgrade preflight check #661

everettraven opened this issue Feb 27, 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 Feb 27, 2024

This focuses on creating the initial CRD Upgrade Safety preflight check. This should be a minimal implementation focused on:

  • Adding the preflight check to the preflight check registry
  • Creating an extensible interface for adding checks to be run against CustomResourceDefinition resources

The work done to resolve this issue will be a foundation that allows for future work to be done in parallel and iteratively add functionality to this preflight check.

If the proposal for adding this functionality to carvel-dev/kapp has been accepted prior to starting this work, all changes should be made against carvel-dev/kapp, otherwise the changes should be made against https://github.com/everettraven/kapp/tree/feature/crd-upgrade-safety-preflight

@everettraven everettraven changed the title Generic implementation of CRD upgrade preflight check. Should have a way to easily add new checks that should be run against a CustomResourceDefinition resource Generic implementation of CRD upgrade preflight check Feb 27, 2024
@everettraven everettraven added the epic/crd-upgrade-safety Marks an issue as part of the CRD Upgrade Safety epic label Mar 13, 2024
@everettraven everettraven self-assigned this Mar 13, 2024
@everettraven
Copy link
Contributor Author

Done in carvel-dev/kapp as of carvel-dev/kapp#901 - closing

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