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

PDB does not use right set of labels as selector #232

Closed
arthur-leclerc opened this issue Jan 15, 2024 · 1 comment · Fixed by #230
Closed

PDB does not use right set of labels as selector #232

arthur-leclerc opened this issue Jan 15, 2024 · 1 comment · Fixed by #230

Comments

@arthur-leclerc
Copy link

arthur-leclerc commented Jan 15, 2024

Hi,

The PDB deployed by the Helm Chart does not use the right set of labels as selector, currently the PDB uses authentik.labels when I think it should use authentik.labels.selectorLabels, the consequence is that the PDB watch on too many labels and can't find any pods.

server-pdb.yaml

  selector:
    matchLabels:
      {{- include "authentik.labels" $ | nindent 6 }}
      app.kubernetes.io/component: server

server-deployment.yaml

  template:
    metadata:
      labels:
        {{- include "authentik.labels.selectorLabels" $ | nindent 8 }}
        app.kubernetes.io/component: "server"
        app.kubernetes.io/version: "{{ trunc 63 $.Values.image.tag }}"

Edit: I just saw #230, I'm leaving this issue open to provide information that the PDBs are not currently working, the Chart rework will fixes this problem.

@rissson
Copy link
Member

rissson commented Jan 15, 2024

Hi, thanks for the report, it's indeed fixed in the rework

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 a pull request may close this issue.

2 participants