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

[FEAT] - Helm Controller recreate CRDs #391

Closed
3 tasks
Tracked by #537
IvoGoman opened this issue Jul 25, 2024 · 0 comments · Fixed by #577
Closed
3 tasks
Tracked by #537

[FEAT] - Helm Controller recreate CRDs #391

IvoGoman opened this issue Jul 25, 2024 · 0 comments · Fixed by #577
Assignees
Labels

Comments

@IvoGoman
Copy link
Contributor

IvoGoman commented Jul 25, 2024

Priority

(Medium) I'm annoyed but I'll live

Description

In the case that CRDs used by a Plugin's Helm Chart are deleted from the remote cluster, there is no mechanism to get these CRDs back.

The Helm Controller currently is only installing these CRDs via the helm install-action. Updates to a CRD are done when diffing/ upgrading the Helm Chart.

When a CRD cannot be found the logic skips these CRDs, as it assumes this to only happen if the release was never installed. This prevent's the helm controller from auto-healing such releases.

Acceptance Criteria:

  • CRDs are re-created if they are missing but the release was installed
  • CRDs are only replaced during an upgrade, not during a dry-run/ template
  • Test showing the recreation

Reference Issues

No response

@IvoGoman IvoGoman moved this to Sprint Backlog in Greenhouse Core Roadmap Jul 25, 2024
@Zaggy21 Zaggy21 self-assigned this Sep 6, 2024
@Zaggy21 Zaggy21 moved this from Sprint Backlog to In progress in Greenhouse Core Roadmap Sep 6, 2024
@Zaggy21 Zaggy21 linked a pull request Sep 17, 2024 that will close this issue
26 tasks
@github-project-automation github-project-automation bot moved this from In progress to Done in Greenhouse Core Roadmap Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants