Skip to content

[alerts] fix pod/container/namespace not rendering #11060

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

Merged
merged 1 commit into from
Jul 1, 2022
Merged

Conversation

jenting
Copy link
Contributor

@jenting jenting commented Jul 1, 2022

Description

Because every time series is uniquely identified by its metric name and a set of labels, every unique combination of key-value label pairs represents a new alert for this time series.

There is no common value for these metrics

  • kube_pod_container_status_restarts_total
  • gitpod_ws_manager_workspace_backups_failure_total

Related Issue(s)

Fixes #

How to test

Release Notes

None

Documentation

None

Werft options:

  • /werft with-preview

Because every time series is uniquely identified by its metric name
a set of labels, and every unique combination of key-value label pairs
represents a new alert for this time series.

There is no common value for these metrics
- kube_pod_container_status_restarts_total
- gitpod_ws_manager_workspace_backups_failure_total

Signed-off-by: JenTing Hsiao <[email protected]>
@jenting jenting requested a review from a team July 1, 2022 00:45
@github-actions github-actions bot added the team: workspace Issue belongs to the Workspace team label Jul 1, 2022
@roboquat roboquat merged commit 7800a21 into main Jul 1, 2022
@roboquat roboquat deleted the jenting/alerts branch July 1, 2022 00:53
@mads-hartmann
Copy link
Contributor

mads-hartmann commented Jul 1, 2022

@jenting / @sagor999 did any of you run the job to ensure this is deployed to Gen51 and Gen47? ☺️

@jenting
Copy link
Contributor Author

jenting commented Jul 1, 2022

@jenting / @sagor999 did any of you run the job to ensure this is deployed to Gen51 and Gen47? ☺️

No. But it's a minor change, we could consider keeping the Gen51 and Gen47 as is.

@roboquat roboquat added deployed: workspace Workspace team change is running in production deployed Change is completely running in production labels Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: workspace Workspace team change is running in production deployed Change is completely running in production release-note-none size/XS team: workspace Issue belongs to the Workspace team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants