Skip to content

fix: 온콜 시험이 Slack 까지 실제 P0 와 같은 경로로 - #322

Merged
selee1012 merged 2 commits into
devfrom
fix/oncall-test-full-path
Oct 15, 2026
Merged

selee1012 merged 2 commits into
devfrom
fix/oncall-test-full-path

Conversation

@selee1012

Copy link
Copy Markdown
Contributor

test-trigger가 PagerDuty SNS 토픽에만 메시지를 직접 보내고 있었습니다. 그래서 시험할 때 Slack(#ruleup-alert, 긴급 채널 멘션)이 오지 않았습니다. 실제 P0 경보의 Slack 경로는 정상입니다(#ruleup-alert 최근 7일 64건 전달, 실패 0).

변경

  • 서울에 시험 경보 ruleup-prod-P0-ONCALL-TEST를 만듭니다.
    • AlarmActions·OKActions는 실제 P0와 같습니다: ruleup-prod-alerts, ruleup-prod-p0-alerts, ruleup-prod-pagerduty
    • 이 경보를 set-alarm-state로 ALARM 또는 OK로 바꿉니다.
  • 아무도 내지 않는 지표(RuleUp/Observe OnCallTest)를 missing=ignore로 봅니다. 그래서 다음 평가에서 OK로 돌아가지 않고, 10분 에스컬레이션까지 볼 수 있습니다.
  • test-resolve를 실행하면 Slack 복구 알림이 오고 PagerDuty 사건이 닫힙니다.
  • tests_pagerduty를 CI에 넣었습니다.

스택: #321 위에 올렸습니다.

🤖 Generated with Claude Code

selee1012 and others added 2 commits October 15, 2026 19:50
test-trigger 가 PagerDuty 토픽에만 메시지를 보내서 Slack(#ruleup-alert·긴급 채널 멘션)이 빠졌다.
시험 경보 ruleup-prod-P0-ONCALL-TEST 를 실제 P0 와 같은 AlarmActions 로 두고 set-alarm-state 로
상태만 바꾼다. 아무도 내지 않는 지표를 missing=ignore 로 봐서 다음 평가에서 OK 로 돌아가지 않으므로
10분 에스컬레이션도 그대로 볼 수 있다. tests_pagerduty 를 CI 에 넣는다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Test Coverage

There is no coverage information present for the Files changed

@github-actions

Copy link
Copy Markdown

Test Coverage

Overall Project 82.7% 🍏

There is no coverage information present for the Files changed

Base automatically changed from chore/pagerduty-ledger to dev October 15, 2026 10:59
@selee1012
selee1012 merged commit f60eaea into dev Oct 15, 2026
3 checks passed
@selee1012
selee1012 deleted the fix/oncall-test-full-path branch October 15, 2026 11:00
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.

1 participant