helm: Make chart manage VolSync CRDs
Describe what this PR does Helm will only install CRDs if they are not present. It will not upgrade or remove them. This change switches the CRDs to be managed as a part of the templating mechanism so that they can be upgraded (and removed if necessary).
Is there anything that requires special attention? This is a breaking change for Helm users as noted in the Changelog. In order for a chart upgrade to succeed, users will need to label/annotate the CRDs
Related issues: Fixes #261
/retest
/retest
/hold Investigate leaving CRDs on uninstall
/unhold /cc @tesshuflower The CRD is now left in the cluster on uninstall
SonarCloud Quality Gate failed. 
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
8.6% Duplication
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: JohnStrunk, tesshuflower
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [JohnStrunk,tesshuflower]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
/override "SonarCloud Code Analysis"
@JohnStrunk: Overrode contexts on behalf of JohnStrunk: SonarCloud Code Analysis
In response to this:
/override "SonarCloud Code Analysis"
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.