Breaking changes with active:active commit and chart version unchanged

[cdunford]

There was an update made to the server chart (specifically ce60ce8#diff-cf1e8c14e54505f60aa10ceb8d5d8ab3) that removed values that we were setting (specifically the audit DB parameters auditHost, auditUser and auditPassword). Now our console container will no longer start as it cannot connect to the audit DB and one of our environments is unusable.

The bigger problem here is that the chart version did not change whatsoever with this change, and was published to helm.aquasec.com replacing the previous revision of 4.6.0. We have no way to reference the previous, working version of the chart, as it appears it is completely replaced by this new version.

Whenever changes are made to the chart, at the very least the chart patch version should be changed (and possibly major/minor version depending on the scope/nature of the change) to avoid impacting anyone who has pinned to a specific version of the chart. Once a specific version of the chart is published, it should not be modified.

[agilgur5]

Saw 70ae917 which also makes a large amount of breaking changes...

I'm not sure if the Chart registry was updated with those changes, but these kinds of drastic breaking changes without bumping major and without so much as a changelog (not to mention these commits and PRs basically just say "updates"), let alone upgrade directions, pretty much make this repo unusable due to poor maintenance and lack of care...

Overwriting an existing pinned version as you wrote means it's not even possible to rely on exact version pinning... 😬 😬 😕

[cdunford]

@agilgur5 - I'm now using the helm git plugin to consume these charts direct from the github repo and I pin to a specific commit. Seems to be the only safe thing to do.

[agilgur5]

Yea but without a changelog and without SemVer updates are basically ??? 😕

[xB-2048]

@cdunford Thanks for reporting the issue.

We will manage the chart version and changelog going forward.