Skip to content

관측 프로비저닝 SSM 로드를 공용 블록 호출로 교체 - #949

Merged
m-a-king merged 2 commits into
devfrom
refactor/948-alloy-ssm-block
Aug 15, 2026
Merged

관측 프로비저닝 SSM 로드를 공용 블록 호출로 교체#949
m-a-king merged 2 commits into
devfrom
refactor/948-alloy-ssm-block

Conversation

@m-a-king

Copy link
Copy Markdown
Collaborator

Situation

  • 이 repo 가 갖고 있던 "SSM 에서 Grafana 자격 7종을 읽어 provision-alloy.sh 를 호출한다"는 로직은 core·extractor·renderer 세 곳에 복제된 것이었다. 셋 다 같은 SSM 경로(/piki/observability/grafana-*)·같은 필수/선택 구분·같은 dockerized aws-cli 를 쓰고, 다른 건 --box·--environment 값뿐이었다.
  • 경로나 실패 정책이 바뀌면 세 곳을 함께 고쳐야 하고, 하나만 빠뜨리면 그 박스만 조용히 어긋난다. 이번에 겪은 사고(fs_types_exclude 가 기본값을 대체하는데 squashfs 한 항목이 빠져 /snap 시계열이 누수된 것)와 같은 종류다.
  • Alloy SSM 자격 로드를 공용 블록으로 추출 infra#50 으로 그 로직이 공용 블록 blocks/alloy/provision-alloy-ssm.sh 로 추출됐다 (self-test 15케이스 포함).

Task

  • 복제본을 지우고 공용 블록을 호출한다. 이 repo 에는 박스 값만 남긴다.

Action

  • provision-runtime.sh 4절의 SSM 조회 로직을 걷어내고 provision-alloy-ssm.sh 호출로 대체했다.
  • deploy.yml 업로드 목록에 그 블록을 더했다 — 블록이 형제 경로의 provision-alloy.sh 를 부르므로 같은 alloy/ 하위에 안착해야 한다.
  • 전환기 잔재 정리(구 team3-alloy 제거)는 core 고유라 이 스크립트에 남겼다.
  • 동작·실패 정책은 그대로다 — 필수 5종 실패는 즉시 중단, traces 2종은 빈 값 허용. 같은 계약이 블록으로 옮겨졌고, 이제 그 계약은 블록의 self-test 가 고정한다.

Result

  • 이 repo 에 남는 관측 관련 코드는 --box·--environment 값과 fetch 뿐이다.
  • 앞으로 SSM 경로·조회 방식·실패 정책이 바뀌면 infra 블록 한 곳만 고치면 되고, 세 박스에 각자의 다음 배포로 전파된다.

연관 이슈

- provision-runtime.sh 4절의 SSM 자격 조회(약 15줄)를 걷어내고 TeamPiKi/infra#50 의 공용 블록 provision-alloy-ssm.sh 호출로 대체. 같은 로직이 extractor·renderer 에도 복제돼 있어, 경로나 실패 정책이 바뀌면 세 곳을 함께 고쳐야 하고 하나만 빠뜨리면 그 박스만 조용히 어긋났다
- deploy.yml 업로드 목록에 그 블록을 더했다 - 블록이 형제 경로의 provision-alloy.sh 를 부르므로 같은 alloy/ 하위에 안착해야 한다
- 동작·실패 정책은 그대로다(필수 5종 중단, traces 빈 값 허용) - 같은 계약이 블록으로 옮겨졌고 그 계약은 블록의 self-test 15케이스가 고정한다
- 전환기 잔재 정리(구 team3-alloy 제거)는 core 고유라 이 스크립트에 남긴다
- AWSCLI_IMAGE 는 mysql SSM pull 이 계속 쓰므로 유지하되, alloy 가 더는 쓰지 않는다는 것을 주석에 반영
@m-a-king m-a-king added the refactor 구조 개선, 외부 동작 불변 label Aug 15, 2026
@m-a-king m-a-king linked an issue Aug 15, 2026 that may be closed by this pull request
@m-a-king m-a-king self-assigned this Aug 15, 2026
@github-actions

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: 976798fa-e3e1-4ff8-bf67-6e224502ea59


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@m-a-king
m-a-king merged commit 073cd73 into dev Aug 15, 2026
7 checks passed
@m-a-king
m-a-king deleted the refactor/948-alloy-ssm-block branch August 15, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 구조 개선, 외부 동작 불변

Projects

None yet

Development

Successfully merging this pull request may close these issues.

관측 프로비저닝 SSM 로드를 공용 블록 호출로 교체

2 participants