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

Installing fresh with esRollover fails due to init Job hook #276

Open
dpkirchner opened this issue Aug 13, 2021 · 1 comment
Open

Installing fresh with esRollover fails due to init Job hook #276

dpkirchner opened this issue Aug 13, 2021 · 1 comment
Labels
bug Something isn't working

Comments

@dpkirchner
Copy link

Describe the bug
When you install the jaeger helm chart with esRollover enabled, the chart attempts to run the es-rollover-init Job, and due to the use of pre-install, pre-upgrade, the chart will fail to install because Elasticsearch is not yet installed.

https://github.com/jaegertracing/helm-charts/blob/main/charts/jaeger/templates/es-rollover-hook.yml

To Reproduce
Steps to reproduce the behavior:

  1. helm repo add jaegertracing https://jaegertracing.github.io/helm-charts
  2. helm install jaeger jaegertracing/jaeger --set=provisionDataSource.elasticsearch=true --set=esRollover.enabled=true

I use terraform so I'm synthesizing the command, I might have the flags wrong.

Expected behavior
I expected the chart to install successfully and, if it's actually necessary, for the es rollover job to run after ES has been installed and is live (which takes a few minutes).

Version (please complete the following information):

  • OS: [e.g. Linux] n/a (whatever the version is in the containerS)
  • Jaeger version: [e.g. 1.8] 1.22
  • Deployment: [e.g. bare metal, Docker, Kubernetes, ...] Kubernetes, GKE

What troubleshooting steps did you try?
None -- although removing the esRollover.enabled flag temporarily is an effective workaround.

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

No branches or pull requests

2 participants