Skip to content

관측 프로비저닝을 공용 블록 호출로 축소 - #41

Merged
m-a-king merged 1 commit into
mainfrom
refactor/40-alloy-ssm-block
Aug 15, 2026
Merged

관측 프로비저닝을 공용 블록 호출로 축소#41
m-a-king merged 1 commit into
mainfrom
refactor/40-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

  • SSM 조회 로직을 걷어내고 provision-alloy-ssm.sh 호출로 대체했다.
  • 블록이 형제 경로의 provision-alloy.sh 를 호출하므로, 블록 3개가 같은 디렉터리에 놓이도록 했다.
  • 동작·실패 정책은 그대로다 — 필수 5종 실패는 즉시 중단, traces 2종은 빈 값 허용. 같은 계약이 블록으로 옮겨졌고, 이제 그 계약은 블록의 self-test 가 고정한다.

Result

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

연관 이슈

- SSM 자격 조회 로직(약 45줄)을 걷어내고 TeamPiKi/infra#50 의 공용 블록 provision-alloy-ssm.sh 호출로 대체. 같은 로직이 core·renderer 에도 복제돼 있어, 경로나 실패 정책이 바뀌면 세 곳을 함께 고쳐야 하고 하나만 빠뜨리면 그 박스만 조용히 어긋났다
- 이 스크립트에 남는 것은 이 박스의 값(--box piki-extractor·--environment prod)뿐이다
- fetch 대상에 provision-alloy-ssm.sh 를 더했다 - 블록이 형제 경로의 provision-alloy.sh 를 부르므로 셋을 같은 디렉터리에 받는다
- 동작·실패 정책은 그대로다(필수 5종 중단, traces 빈 값 허용) - 같은 계약이 블록으로 옮겨졌고, 그 계약은 블록의 self-test 15케이스가 고정한다
- 검증: fetch 3종 성공 + 블록이 SSM 조회까지 진입하는 것을 로컬에서 확인(자격이 없어 의도대로 중단)
@m-a-king m-a-king added the refactor 구조 개선, 외부 동작 불변 label Aug 15, 2026
@m-a-king m-a-king self-assigned this Aug 15, 2026
@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: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6712951d-f34c-4b23-baff-7c65dfc8419a


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 e9d1e73 into main Aug 15, 2026
2 checks passed
@m-a-king
m-a-king deleted the refactor/40-alloy-ssm-block branch August 15, 2026 13:42
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 로드 중복 제거

1 participant