fix(rules): 알림 라벨에서 __name__ 을 뗀다 - #19
Merged
Merged
Conversation
클러스터 실증에서 드러났다 — 실제 GPU 알림에 __name__= nodevitals_hw_gpu_temperature_celsius 가 그대로 실려 나왔다. 알림의 정체성은 alertname 과 대상 라벨이지 어떤 메트릭에서 나왔는가가 아니다. 남겨 두면 지문에 섞여 그룹화·중복제거가 메트릭 이름에 묶이고, 같은 대상의 알림이 식만 바뀌어도 다른 알림으로 취급된다. Prometheus 도 뗀다.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
클러스터 실증에서 드러난 결함입니다. 실 GPU 알림이 이렇게 나왔습니다:
{"labels":{"__name__":"nodevitals_hw_gpu_temperature_celsius", "alertname":"GpuHot","node":"e22","severity":"warning"}, "state":"firing","value":"59"}알림의 정체성은
alertname과 대상 라벨이지 어떤 메트릭에서 나왔는가가 아닙니다.__name__이 남으면 지문에 섞여 그룹화·중복제거가 메트릭 이름에 묶이고, 같은 대상의 알림이 식만 바뀌어도 다른 알림으로 취급됩니다. Prometheus 도 뗍니다.단위 테스트로는 잡히지 않았습니다 — 테스트가
node·severity존재만 확인하고 없어야 할 것은 보지 않았기 때문입니다. 회귀 테스트를 추가했습니다.