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

ILM policy is not configurable #2967

Closed
jrRibeiro opened this issue Apr 29, 2021 · 2 comments · Fixed by #2971
Closed

ILM policy is not configurable #2967

jrRibeiro opened this issue Apr 29, 2021 · 2 comments · Fixed by #2971

Comments

@jrRibeiro
Copy link
Contributor

jrRibeiro commented Apr 29, 2021

Requirement - what kind of business use case are you trying to solve?

Following #2796 (great work, btw), we now have the ability to use ILM to manage indices. In multi-tenancy scenarios, when running several Jaeger instances using the same Elastic storage, we can use different indices for each instance by defining its own prefix. However, different indices might need different ILM policies (some indices need different rollover and retention configurations than others).

Problem - what in Jaeger blocks you from solving the requirement?

Right now, an ILM policy is hard-coded (named jaeger-ilm-policy) and it is not possible to change.

Proposal - what do you suggest to solve the problem or improve the existing situation?

Make the policy's name configurable using an env var (to use with USE_ILM). This way, we can set the ILM policy to use and change the mappings of the index to use different policies per index.

I can open a new PR if you agree with my proposal. 🙂

@yurishkuro
Copy link
Member

+1

@albertteoh
Copy link
Contributor

Sounds good to me @jrRibeiro and a PR is very welcome!

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.

3 participants