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

[EPM] Reassess default index template priority #90299

Closed
skh opened this issue Feb 4, 2021 · 6 comments
Closed

[EPM] Reassess default index template priority #90299

skh opened this issue Feb 4, 2021 · 6 comments
Labels
Feature:EPM Fleet team's Elastic Package Manager (aka Integrations) project Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@skh
Copy link
Contributor

skh commented Feb 4, 2021

Currently, we use the following priorities for built-in or generated index templates:

The priority 200 might conflict with what we suggest people do in the elasticsearch documentation linked above:

For example, if you don’t use the Elastic Agent and want to create a template for the logs-* index pattern, assign your template a priority of 200. This ensures your template is applied instead of the built-in template for logs--.

This issue is to decide if we want to use a different default priority, change the recommendation in the docs, or leave everything as it is now.

@skh skh added the Feature:EPM Fleet team's Elastic Package Manager (aka Integrations) project label Feb 4, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Feature:EPM)

@skh skh added the Team:Fleet Team label for Observability Data Collection Fleet team label Feb 4, 2021
@ph
Copy link
Contributor

ph commented Feb 4, 2021

@ruflin I would be tented to just update the doc to give us more room if we have more thing to changes, I compare this to z-index problems, maybe 500-1000 would be a good level.

@ruflin
Copy link
Contributor

ruflin commented Feb 5, 2021

I wonder if the ES team has an opinion on this @jpountz @dakrone ?

@dakrone
Copy link
Member

dakrone commented Feb 5, 2021

I don't have a strong opinion about this, but for the sake of priority consistency between versions, I'd prefer updating the documentation as the solution.

@skh
Copy link
Contributor Author

skh commented Feb 15, 2021

I've opened elastic/elasticsearch#68982 for a potential documentation change.

@skh
Copy link
Contributor Author

skh commented Feb 17, 2021

Docs were changed in elastic/elasticsearch#69006

@skh skh closed this as completed Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:EPM Fleet team's Elastic Package Manager (aka Integrations) project Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Development

No branches or pull requests

5 participants