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

Migrate from helm operator to helm-controller and source-controller #372

Closed
seaneagan opened this issue Oct 19, 2020 · 2 comments
Closed
Assignees
Labels
enhancement New feature or request priority/critical Items critical to be implemented, usually by the next release ready for review Change related to the issue is ready for review
Milestone

Comments

@seaneagan
Copy link
Contributor

Problem description (if applicable)

Flux v1 and the Helm Operator are currently in maintenance mode, and will shortly be entering a 6 month migration period before support is removed, now that the GitOps Toolkit (includes helm-controller) is reaching feature parity. The new APIs (e.g. HelmRelease, HelmRepository, HelmChart) have reached beta status and any any further breaking changes to these APIs will include a conversion mechanism. See here for more info.

Proposed change

Add helm-controller, and source-controller which is a dependency, support to the airshipctl manifests and remove helm operator support.

Potential impacts

Any treasuremap or other downstream HelmRelease v1 documents will need to be replaced with HelmRelease v2beta1, and HelmRepository v1beta1 or GitRepository v1beta1 for chart sourcing information, and the versions catalog format will need to be updated to support this. In order to do this migration, the Helm Operator must be disabled (as proposed above) first, in order to avoid Helm releases being deleted by the Helm Operator.

@seaneagan seaneagan added enhancement New feature or request triage Needs evaluation by project members labels Oct 19, 2020
@jezogwza jezogwza added this to the v2.0 milestone Oct 21, 2020
@jezogwza jezogwza added priority/critical Items critical to be implemented, usually by the next release and removed triage Needs evaluation by project members labels Oct 21, 2020
@seaneagan seaneagan added the ready for review Change related to the issue is ready for review label Oct 21, 2020
@airshipbot
Copy link

airshipbot commented Oct 21, 2020

Related Change #758615

Subject: Add helm-controller and source-controller
Link: https://review.opendev.org/758615
Status: MERGED
Owner: Sean Eagan (sean.eagan@att.com)

This change will close this issue when merged.

Approvals

Code-Review
+2 Kostyantyn Kalynovskyi
+2 Dmitry Ukov
Verified
+2 Zuul
Workflow
+1 Kostyantyn Kalynovskyi

Last Updated: 2020-11-06 12:45:17 CST

@airshipbot
Copy link

The Change that closes this issue was merged.

@airshipbot airshipbot added ready for review Change related to the issue is ready for review and removed ready for review Change related to the issue is ready for review labels Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority/critical Items critical to be implemented, usually by the next release ready for review Change related to the issue is ready for review
Projects
None yet
Development

No branches or pull requests

3 participants