-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[SLO] Rename indices without dot prefix #196340
Comments
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
Thanks for logging this, @kdelemme -- for right now, I don't know if we want to change SLO index names because dot-prefixed indices makes logical sense for these indices. So for now, I think we won't move forward on THIS, but we'll need to check in with platform to see how they are going to enable this for us if the dot prefix restriction is turned on again. What would be GREAT is if the secondary auth split ONLY used the stored user's credentials for reading from source indices, and used the internal user for everything else including the write/create side of the transform and ingest pipelines. |
@jasonrhodes you mentioned that dot-prefixed indices make sense for these — is there a plan to eventually make these system indices? |
@dakrone I don't think I have a great understanding of what counts as a system index. Could you clarify? Our component templates are indeed marked as hidden. It's not clear to me what designates an index as a system index. Some setting in elasticsearch? Marking your component template as hidden? Naming it with a dot prefix? Thanks, I really appreciate the context. |
A system index is one where the user is not intended to perform actions directly on the index itself. Rather, the user would interact with Elasticsearch through specialized APIs designed to modify the index in some way. This prevents the user from doing something that could break the functionality of an area. As an example, the @dominiqueclarke I'm not familiar enough with what data goes into the SLO indices to be able to make a recommendation here, perhaps you can elaborate on how a user interacts with these indices? @jasonrhodes mentioned dot prefixes make logical sense for these indices, can you explain more on why you think these indices should begin with a dot? They seem to be internal, but a dot prefix is not a requirement for an index to be hidden, so I'd like to understand the motivation there as well. |
Technically, the So we cannot really make them system indices. When we began working on SLO, we were under the impression that dot-prefixed indices were treated as hidden index by default. And since we didn't want customer to delete the index by mistake, we went with that. |
Okay, it sounds like it would be best if these were indices that were hidden, but without a dot-prefix. |
@dakrone based on the original email announcing this change, we believe that the SLO app's using of dot prefix indices falls squarely within the intended usage. Without specifically quoting that internal email in a public issue I'll enumerate my points
|
+1 to everything @dominiqueclarke said, this is exactly why it seems like the SLO indices are a prime example of the way dot-prefixed indices are being described in the most recent email communications about them. |
Note: I've created an issue to test out a possible solution to this situation - #200953 |
🌮 Summary
Since https://github.com/elastic/serverless-gitops/pull/4615, users cannot create dot-prefixed indices, and therefore SLO transforms and ingest pipelines creating dot-prefixed indices for the rollup data are broken.
The current SLO version (v3.3) uses dot-prefixed indices for both the sli and summary data.
date_index_name
processor to create new monthly indices:.slo-observability.sli-v${SLO_RESOURCES_VERSION}.YYYY-MM-DD/M
destination
source
and the dot-prefixed summary index asdestination
We can't reset the SLOs because we need the user's API key to create the rollup transforms with the correct permission on the source index.
🎯 Acceptance criteria
Plan
to be defined
The text was updated successfully, but these errors were encountered: