Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bitnami/jaeger] Can't provide own secret for cassandra db via existingSecret #26359

Closed
Sebastian-Materna opened this issue May 23, 2024 · 6 comments · Fixed by #26563
Closed
Assignees
Labels
jaeger solved stale 15 days without activity tech-issues The user has a technical issue about an application

Comments

@Sebastian-Materna
Copy link
Contributor

Sebastian-Materna commented May 23, 2024

Name and Version

bitnami/jaeger 1.7,1

What architecture are you using?

amd64

What steps will reproduce the bug?

  1. Use this Helm chart as a dependency
  2. Create values
  3. Install/Upgrade the Helm release

Are you using any custom parameters or values?

We are using the Jaeger Helm chart as a dependency in our Helm chart. Part of our values.yaml:

jaeger:
  externalDatabase:
    existingSecret: jaeger-secret
    existingSecretPasswordKey: cassandra-password
  cassandra:
    persistence:
      size: 8Gi

What is the expected behavior?

I want to use an existing secret, which is created by external secrets operator named jaeger-secret. But this secret will not used, only the default jaeger-cassandra.

What do you see instead?

A new secret jaeger-cassandra gets created.

@Sebastian-Materna Sebastian-Materna added the tech-issues The user has a technical issue about an application label May 23, 2024
@github-actions github-actions bot added the triage Triage is needed label May 23, 2024
@carrodher
Copy link
Member

Are you able to reproduce the issue by deploying the Helm chart in a standalone way (without being a subchart)?

@Sebastian-Materna
Copy link
Contributor Author

Sebastian-Materna commented May 24, 2024

Yes, unfortunately:
Using the values-test.yaml:

externalDatabase:
  existingSecret: jaeger-secret
  existingSecretPasswordKey: cassandra-password

  cassandra:
    persistence:
      size: 24Gi

Running the command helm template -n test -f values-test.yaml jaeger bitnami/jaeger --version 1.7.1 > test.yaml
There are no matches for jaeger-secret, only the default from the template jaeger-cassandra. This happens also on the latest version (2.2.1).

            - name: CASSANDRA_PASSWORD
              valueFrom:
                secretKeyRef:
                  name: jaeger-cassandra
                  key: cassandra-password

I'm using this as a subchart, because I have added an Ingress resource and ExternalSecrets resource. Also the persistence can't be changed right now. I've set 24Gi, but it's still 8Gi

  volumeClaimTemplates:
    - apiVersion: v1
      kind: PersistentVolumeClaim
      metadata:
        name: data
        labels:
          app.kubernetes.io/instance: jaeger
          app.kubernetes.io/name: cassandra
      spec:
        accessModes:
          - "ReadWriteOnce"
        resources:
          requests:
            storage: "8Gi"

@github-actions github-actions bot removed the triage Triage is needed label May 24, 2024
@github-actions github-actions bot assigned migruiz4 and unassigned carrodher May 24, 2024
@migruiz4
Copy link
Member

Hi @Sebastian-Materna,

The values externalDatabase.existingSecret and externalDatabase.existingSecretPasswordKey are only taken into account when the Cassandra subchart is not used, which means cassandra.enabled=false.

To provide an existingSecret, you should use the value cassandra.dbUser.existingSecret.
If using Jaeger as a subchart: jaeger.cassandra.dbUser.existingSecret.

@migruiz4
Copy link
Member

While testing your use case, I found an issue where Jaeger would not use the cassandra.dbUser.existingSecret value, so you may need to update to the latest version once this fix is released: #26563

@migruiz4 migruiz4 changed the title Can't provide own secret for cassandra db via existingSecret [bitnami/jaeger] Can't provide own secret for cassandra db via existingSecret May 30, 2024
Copy link

This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback.

@github-actions github-actions bot added the stale 15 days without activity label Jun 15, 2024
Copy link

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Issue. Do not hesitate to reopen it later if necessary.

@bitnami-bot bitnami-bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jaeger solved stale 15 days without activity tech-issues The user has a technical issue about an application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants