관측 프로비저닝을 공용 블록 호출로 축소 - #41
Merged
Merged
Conversation
- 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 조회까지 진입하는 것을 로컬에서 확인(자격이 없어 의도대로 중단)
|
Important Review available on request
Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 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. Comment |
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.
Situation
provision-alloy.sh를 호출한다"는 로직은 core·extractor·renderer 세 곳에 복제된 것이었다. 셋 다 같은 SSM 경로(/piki/observability/grafana-*)·같은 필수/선택 구분·같은 dockerized aws-cli 를 쓰고, 다른 건--box·--environment값뿐이었다.fs_types_exclude가 기본값을 대체하는데squashfs한 항목이 빠져/snap시계열이 누수된 것)와 같은 종류다.blocks/alloy/provision-alloy-ssm.sh로 추출됐다 (self-test 15케이스 포함).Task
Action
provision-alloy-ssm.sh호출로 대체했다.provision-alloy.sh를 호출하므로, 블록 3개가 같은 디렉터리에 놓이도록 했다.Result
--box·--environment값과 fetch 뿐이다.연관 이슈