Fix Helm chart packaging #18
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It seems like
quarkus.helm.create-tar-file=truedoes not package the additional files that are generated underoperator/build/helm/kubernetes/postgresql-operator.See the marked files here, which are missing in the automatically packaged

*.tgzfile:See the files from the last release:
https://github.com/aboutbits/postgresql-operator/releases/download/v0.2.1/postgresql-operator-0.2.1.tgz
This archive contains a consolidated

clusterrolebinding.yaml, whereas the folderoperator/build/helm/kubernetes/postgresql-operatorwithquarkus.helm.create-tar-file=falsecontains a different set of role bindings, as shown in the previous screenshot.Also the content is not the same:

quarkus.helm.create-tar-file=trueAs the namespace is missing in the role bindings from the automatically created archive, I get this error when I install the Helm chart:
The content of, for example file

operator/build/helm/kubernetes/postgresql-operator/databasereconciler-crd-role-binding.yamlDue to this, I'm disabling
quarkus.helm.create-tar-file=falseand do the*.tgzpackaging myself.