Skip to content

Conversation

@ThoSap
Copy link
Contributor

@ThoSap ThoSap commented Jan 30, 2026

It seems like quarkus.helm.create-tar-file=true does not package the additional files that are generated under operator/build/helm/kubernetes/postgresql-operator.

See the marked files here, which are missing in the automatically packaged *.tgz file:
image

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 folder operator/build/helm/kubernetes/postgresql-operator with quarkus.helm.create-tar-file=false contains a different set of role bindings, as shown in the previous screenshot.
image

Also the content is not the same:
quarkus.helm.create-tar-file=true
image

As the namespace is missing in the role bindings from the automatically created archive, I get this error when I install the Helm chart:

> helm install --namespace postgresql-cluster-configuration-operator postgresql-operator https://github.com/aboutbits/postgresql-operator/releases/download/v0.2.1/postgresql-operator-0.2.1.tgz
Error: INSTALLATION FAILED: 12 errors occurred:
        * ClusterRoleBinding.rbac.authorization.k8s.io "clusterconnectionreconciler-cluster-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "schemareconciler-crd-validating-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "databasereconciler-cluster-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "rolereconciler-crd-validating-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "clusterconnectionreconciler-crd-validating-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "rolereconciler-cluster-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "databasereconciler-crd-validating-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "schemareconciler-cluster-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "defaultprivilegereconciler-crd-validating-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "grantreconciler-cluster-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "defaultprivilegereconciler-cluster-role-binding" is invalid: subjects[0].namespace: Required value
        * ClusterRoleBinding.rbac.authorization.k8s.io "grantreconciler-crd-validating-role-binding" is invalid: subjects[0].namespace: Required value

The content of, for example file operator/build/helm/kubernetes/postgresql-operator/databasereconciler-crd-role-binding.yaml
image


Due to this, I'm disabling quarkus.helm.create-tar-file=false and do the *.tgz packaging myself.

@ThoSap ThoSap requested a review from Copilot January 30, 2026 19:28
@ThoSap ThoSap self-assigned this Jan 30, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses an issue where the Quarkus Helm chart packaging feature (quarkus.helm.create-tar-file=true) was not including all necessary files in the generated .tgz archive. The solution disables automatic packaging and implements manual Helm chart packaging in the release workflow.

Changes:

  • Disabled automatic Helm chart packaging in Quarkus configuration
  • Added manual Helm chart packaging step using tar command in the release workflow
  • Updated release upload command to reference the manually packaged chart with explicit versioning

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
operator/src/main/resources/application.yml Disabled automatic Helm chart packaging by setting create-tar-file to false
.github/workflows/release.yml Added manual tar packaging step and updated release upload to use the explicitly named chart file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ThoSap ThoSap merged commit 4d642d4 into main Jan 30, 2026
4 checks passed
@ThoSap ThoSap deleted the fix-helm-chart-packaging branch January 30, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants