Issue 241: Chart changes and CRD refactoring

[SrishT]

Signed-off-by: SrishT [email protected]

Change log description

Helm upgrade of the zookeeper operator and zookeeper cluster from 0.2.8 to 0.2.9-rc0 is failing. Also the additionalPrinterColumns from the ZookeeperCluster CRD have been removed.

Purpose of the change

Fixed #241

What the code does

Changes the location of the CRD from the crds/ since that was causing the operator upgrade from 0.2.8 to 0.2.9-rc0 to fail, and adds the additionalPrinterColumns back into the ZookeeperCluster CRD. Also adds default value for the storageType and kubernetesClusterDomain fields so that helm upgrade of the zookeeper cluster using the reuse-values flag does not fail.

How to verify it

• Helm upgrade of zookeeper operator from 0.2.8 to 0.2.9-rc0 should succeed.
• Helm upgrade of zookeeper cluster from 0.2.8 0.2.9-rc0 should succeed both with and without the reuse-values flag.

[codecov-commenter]

Codecov Report

Merging #242 into master will decrease coverage by 0.65%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master     #242      +/-   ##
==========================================
- Coverage   81.81%   81.16%   -0.66%     
==========================================
  Files          11       11              
  Lines        1248     1258      +10     
==========================================
  Hits         1021     1021              
- Misses        162      170       +8     
- Partials       65       67       +2     

Impacted Files	Coverage Δ
...g/apis/zookeeper/v1beta1/zookeepercluster_types.go	98.30% <ø> (ø)
...kg/apis/zookeeper/v1beta1/zz_generated.deepcopy.go	82.03% <0.00%> (-4.19%)	⬇️
pkg/zk/generators.go	98.31% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5502fb...ea02263. Read the comment docs.

[SrishT]

Closing this PR as it has been replaced with #245