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

configHotReload of FluentbitAgent ignores imagePullSecrets #1771

Closed
sebhoss opened this issue Jul 4, 2024 · 3 comments · Fixed by #1772
Closed

configHotReload of FluentbitAgent ignores imagePullSecrets #1771

sebhoss opened this issue Jul 4, 2024 · 3 comments · Fixed by #1772
Labels
bug Something isn't working

Comments

@sebhoss
Copy link

sebhoss commented Jul 4, 2024

Describe the bug:

I created a FluentbitAgent like this:

apiVersion: logging.banzaicloud.io/v1beta1
kind: FluentbitAgent
metadata:
  name: default
  namespace: logging
spec:
  configHotReload:
    image:
      imagePullSecrets:
        - name: some-pull-secret
      repository: internal.registry.example/ghcr/kube-logging/config-reloader

The configured pull secret was not part of the pod spec for fluentbit and thus the config-reloader image could not be pulled.

Expected behaviour:

I expected that the configured pull secret appears as part of the pod spec for fluentbit pods and that the config-reloader image is pulled through our internal image registry.

Steps to reproduce the bug:

  • Apply FluentbitAgent with configHotReload using at least one image.imagePullSecrets
  • Look at spec of running fluentbit pods whether the pull secret appears

Additional context:

none

Environment details:

  • Kubernetes version (e.g. v1.15.2): 1.29
  • Cloud-provider/provisioner (e.g. AKS, GKE, EKS, PKE etc): kubeadm
  • logging-operator version (e.g. 2.1.1): 4.8.0
  • Install method (e.g. helm or static manifests): helm
  • Logs from the misbehaving component (and any other relevant logs): none
  • Resource definition (possibly in YAML format) that caused the issue, without sensitive data: see above

/kind bug

@aslafy-z
Copy link
Collaborator

aslafy-z commented Jul 5, 2024

Hello @sebhoss
This was fixed and merged. It will be included in the next release.
In the meantime, you can pass your imagepullsecret for configReloader in the fluentbit one at FluentbitAgent.spec.image.imagePullSecrets.

@sebhoss
Copy link
Author

sebhoss commented Jul 5, 2024

@aslafy-z thanks a lot for the quick response!

@pepov
Copy link
Member

pepov commented Jul 5, 2024

thx @aslafy-z !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants