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

Adding cluster version to telemetry elastic #130

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

paigerube14
Copy link
Contributor

Missing cluster version when posting to elastic search that is printed in the general telemetry

This is a key metric where we will be able to more look at regressions

@paigerube14
Copy link
Contributor Author

@tsebastiani is there an easier way for the elastic and telemetry models to match? We'll always want these to be the same

@paigerube14
Copy link
Contributor Author

@chaitanyaenr @tsebastiani any final thoughts here? I can maybe update to use the telemetry model instead of having its own elastic model in a future pr. We need this for prow runs to be able to compare the pod recovery time among the different versions

@chaitanyaenr
Copy link
Contributor

@chaitanyaenr @tsebastiani any final thoughts here? I can maybe update to use the telemetry model instead of having its own elastic model in a future pr. We need this for prow runs to be able to compare the pod recovery time among the different versions

Merging this in favor of evaluating and creating PR for leveraging telemetry model for indexing if it makes sense.

@chaitanyaenr chaitanyaenr merged commit 547f29b into main Nov 4, 2024
3 checks passed
@tsebastiani tsebastiani deleted the cluster_version branch January 8, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants