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.

Sign up for GitHub

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

Jump to bottom

[ILM] Only allow custom suffixes for ILM aliases #3895

Closed
simitt opened this issue Jun 17, 2020 · 0 comments · Fixed by #3905
Closed

[ILM] Only allow custom suffixes for ILM aliases #3895

simitt opened this issue Jun 17, 2020 · 0 comments · Fixed by #3905
Assignees
@simitt

Comments

@simitt
Copy link
Contributor

simitt commented Jun 17, 2020

In #3779 we implemented customizable ILM rollover aliases. Doing some research on integrating with index templates and data streams I suggest to only make a suffix customizable and always prefix rollover_aliases with apm-{version}-{event_type}. This would be aligned with other indexing strategies and make it easier to manage the indices, e.g. set auto_create_index setting for all apm indices, etc. while still providing flexibility to users.
@simitt simitt added the v7.9.0 label Jun 17, 2020
@zube zube bot added [zube]: Ready and removed v7.9.0 labels Jun 17, 2020
@zube zube bot assigned simitt Jun 17, 2020
simitt added a commit to simitt/apm-server that referenced this issue Jun 18, 2020
@simitt
ILM: restrict customization of alias to suffix.
50b22e1 
Change how rollover aliases can be configured to only allow
customizingi a suffix.This change intends to make the ILM
setup less error prone and to make the transition to the
new index_templates and data streams easier.

Follow up on elastic#3826
closes elastic#3895
simitt added a commit to simitt/apm-server that referenced this issue Jun 18, 2020
@simitt
ILM: restrict customization of alias to suffix.
3b72f27 
Change how rollover aliases can be configured to only allow
customizingi a suffix.This change intends to make the ILM
setup less error prone and to make the transition to the
new index_templates and data streams easier.

Follow up on elastic#3826
closes elastic#3895
@simitt simitt mentioned this issue Jun 18, 2020
Merged
4 tasks
simitt added a commit that referenced this issue Jun 23, 2020
@simitt @bmorelli25
ILM: restrict customization of alias to suffix. (#3905)
fc3b62a 
Change how rollover aliases can be configured to only allow
customizing a suffix. This change intends to make the ILM
setup less error prone and to make the transition to the
new index_templates and data streams easier.

Follow up on #3826
closes #3895

Co-authored-by: Brandon Morelli <[email protected]>
simitt added a commit to simitt/apm-server that referenced this issue Jun 23, 2020
@simitt @bmorelli25
ILM: restrict customization of alias to suffix. (elastic#3905)
19fc838 
Change how rollover aliases can be configured to only allow
customizing a suffix. This change intends to make the ILM
setup less error prone and to make the transition to the
new index_templates and data streams easier.

Follow up on elastic#3826
closes elastic#3895

Co-authored-by: Brandon Morelli <[email protected]>
simitt added a commit that referenced this issue Jun 23, 2020
@simitt @bmorelli25
ILM: restrict customization of alias to suffix. (#3905) (#3911)
ac90126 
Change how rollover aliases can be configured to only allow
customizing a suffix. This change intends to make the ILM
setup less error prone and to make the transition to the
new index_templates and data streams easier.

Follow up on #3826
closes #3895

Co-authored-by: Brandon Morelli <[email protected]>
@axw axw removed the [zube]: Done label Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simitt simitt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ILM: restrict customization of alias to suffix. simitt/apm-server
2 participants
@axw @simitt