Skip to content

Conversation

@mosche
Copy link
Contributor

@mosche mosche commented Apr 17, 2025

Using the Index (that contains a uuid) rather than just the index name is important to avoid accidental collisions if an index is recreated with the same name.

Relates to ES-10144, ES-10384

@mosche mosche requested a review from a team as a code owner April 17, 2025 07:30
@mosche mosche added >non-issue :Core/Infra/Core Core issues without another label :Core/Infra/Metrics Metrics and metering infrastructure and removed :Core/Infra/Core Core issues without another label labels Apr 17, 2025
@mosche mosche removed the request for review from a team April 17, 2025 07:30
@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team v9.1.0 labels Apr 17, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Apr 17, 2025
Copy link
Contributor

@prdoyle prdoyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense

Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mosche mosche merged commit e222e5c into main Apr 29, 2025
18 checks passed
@mosche mosche deleted the metering/keep_index_uuid branch April 29, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Metrics Metrics and metering infrastructure >non-issue serverless-linked Added by automation, don't add manually Team:Core/Infra Meta label for core/infra team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants