Skip to content

fix(rules): 알림 라벨에서 __name__ 을 뗀다 - #19

Merged
eightynine01 merged 1 commit into
mainfrom
fix/alert-metric-name
Aug 12, 2026
Merged

fix(rules): 알림 라벨에서 __name__ 을 뗀다#19
eightynine01 merged 1 commit into
mainfrom
fix/alert-metric-name

Conversation

@eightynine01

Copy link
Copy Markdown
Contributor

클러스터 실증에서 드러난 결함입니다. 실 GPU 알림이 이렇게 나왔습니다:

{"labels":{"__name__":"nodevitals_hw_gpu_temperature_celsius",
            "alertname":"GpuHot","node":"e22","severity":"warning"},
 "state":"firing","value":"59"}

알림의 정체성은 alertname 과 대상 라벨이지 어떤 메트릭에서 나왔는가가 아닙니다. __name__ 이 남으면 지문에 섞여 그룹화·중복제거가 메트릭 이름에 묶이고, 같은 대상의 알림이 식만 바뀌어도 다른 알림으로 취급됩니다. Prometheus 도 뗍니다.

단위 테스트로는 잡히지 않았습니다 — 테스트가 node·severity 존재만 확인하고 없어야 할 것은 보지 않았기 때문입니다. 회귀 테스트를 추가했습니다.

클러스터 실증에서 드러났다 — 실제 GPU 알림에 __name__=
nodevitals_hw_gpu_temperature_celsius 가 그대로 실려 나왔다.

알림의 정체성은 alertname 과 대상 라벨이지 어떤 메트릭에서 나왔는가가 아니다.
남겨 두면 지문에 섞여 그룹화·중복제거가 메트릭 이름에 묶이고, 같은 대상의
알림이 식만 바뀌어도 다른 알림으로 취급된다. Prometheus 도 뗀다.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@eightynine01
eightynine01 merged commit 4dd4669 into main Aug 12, 2026
1 check passed
@eightynine01
eightynine01 deleted the fix/alert-metric-name branch August 12, 2026 15:11
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