operator icon indicating copy to clipboard operation
operator copied to clipboard

Helm Chart ignores tenant.configSecret.existingSecret=true

Open coffee-time-design opened this issue 1 year ago • 0 comments

In a recent release, tenant seems to not accept an existing secret for the MinIO root credentials.

Expected Behavior

I provide Helm the following values file:

tenant:
  configSecret:
    existingSecret: true

Create the following file: echo -e "export MINIO_ROOT_USER=${RANDOM_USER}\nexport MINIO_ROOT_PASSWORD=${RANDOM_PASSWORD}" > config.env Create the following secret: kubectl create secret generic myminio-env-configuration --from-file=config.env=config.env -n tenant

I expect helm chart to install and these to be the new credentials of the root user.

Current Behavior

I get the following error: Error: UPGRADE FAILED: Unable to continue with update: Secret "myminio-env-configuration" in namespace "tenant" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "tenant"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "tenant"

If i make those changes (add annotations), the root user is now as per the default chart values as it changes the secret:

minio
minio123

Possible Solution

Something must have changed in a previous release that causes this to no longer work, as it was working, perhaps this change could be rolled back?

Steps to Reproduce (for bugs)

as above

Your Environment

OS Ubuntu 22.04 K3S Latest versions of Operator and Tenant using Helm.

coffee-time-design avatar Sep 24 '24 05:09 coffee-time-design