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

Ensure ES index names are lowercase #4294

Closed
simitt opened this issue Oct 8, 2020 · 0 comments · Fixed by #4295
Closed

Ensure ES index names are lowercase #4294

simitt opened this issue Oct 8, 2020 · 0 comments · Fixed by #4295
Assignees
@simitt
Labels
bug

Comments

@simitt
Copy link
Contributor

simitt commented Oct 8, 2020

With elastic/beats#18854 libbeat changed the default behavior for not not lowercasing all outputs by default, but instead setting index and pipeline names to lowercase in the standard idxmgmt implementation.
APM Server diverges from beats idxmgmt implementation, therefore APM Server would have needed to also update to this behavior. By not applying any change in APM Server but ugrading to libbeat changes, we introduced a regression where customized index names are no longer guaranteed to be lower case.

Issue was reported in https://discuss.elastic.co/t/index-name-must-be-lowercase-after-upgrade-to-7-9-2/251433/7
@simitt simitt added the bug label Oct 8, 2020
@simitt simitt self-assigned this Oct 8, 2020
simitt added a commit to simitt/apm-server that referenced this issue Oct 8, 2020
@simitt
fix: ensure ES index names are lowercase
7e08c93 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes elastic#4294
@simitt simitt mentioned this issue Oct 8, 2020
Merged
2 tasks
simitt added a commit that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (#4295)
73e94cb 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes #4294
@zube zube bot reopened this Oct 12, 2020
simitt added a commit to simitt/apm-server that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (elastic#4295)
f1a15b6 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes elastic#4294
simitt added a commit to simitt/apm-server that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (elastic#4295)
4c5c3fe 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes elastic#4294
simitt added a commit to simitt/apm-server that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (elastic#4295)
59471b9 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes elastic#4294
simitt added a commit that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (#4295) (#4302)
a60f627 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes #4294
simitt added a commit that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (#4295) (#4303)
1c6ff2d 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes #4294
simitt added a commit that referenced this issue Oct 12, 2020
@simitt
fix: ensure ES index names are lowercase (#4295) (#4304)
617c743 
Ensure customized index names are always lowercase,
fixing regression introduced with https://github.com/elastic/beats/pull/18854/files

fixes #4294
simitt added a commit to simitt/apm-server that referenced this issue Oct 15, 2020
@simitt
fix: ensure ES index names are lowercase for ILM
42fef4d 
Ensure ILM custom index suffixes are also lowercased.

fixes elastic#4294
@simitt simitt mentioned this issue Oct 15, 2020
Merged
2 tasks
@simitt simitt closed this as completed Oct 15, 2020
@axw axw removed the [zube]: Done label Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simitt simitt

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: ensure ES index names are lowercase simitt/apm-server
2 participants
@axw @simitt