Skip to content

이관 워크플로의 사이드 배포 호출 입력 갱신 - #1232

Merged
m-a-king merged 1 commit into
devfrom
infra/migrate-side-dispatch
Oct 15, 2026
Merged

m-a-king merged 1 commit into
devfrom
infra/migrate-side-dispatch

Conversation

@m-a-king

Copy link
Copy Markdown
Collaborator

한눈에

계정 이관 때 extractor·renderer 배포를 부르는 방법 갱신

  • 이전: 이관 작업이 두 서비스 배포에 새 서버 주소를 넘김
    • 두 서비스 배포에는 서버 주소 입력이 이미 없음
    • 결과: 이관 작업이 두 서비스 배포를 켜면 실행 요청이 거부됨
  • 이관 작업이 서버 주소 대신 새 계정의 역할을 넘김
    • 두 서비스 배포가 그 역할의 계정에서 서버를 찾음
  • 머지 직후 바뀌는 서비스 없음
    • 이관 작업은 사람이 직접 실행할 때만 돎

Situation

사이드 배포 호출이 SSM 전환 뒤 낡음

  • 계정 이관 워크플로의 서버 접속 SSM 전환 #1219 머지 뒤 이관 워크플로를 다시 보다 발견
  • extractor·renderer 배포 워크플로의 이관 입력은 migration_role_arn·request_id 둘 (두 레포 main 실측)
    • 서버는 넘겨받은 역할의 계정에서 Name 태그로 찾음
  • 이관 워크플로의 두 호출
호출 넘기는 입력 결과
extractor migration_host·migration_role_arn·request_id 없는 입력이라 dispatch 거부 (422)
renderer migration_host·request_id 없는 입력이라 dispatch 거부 (422)
  • 위험: renderer 는 입력 오류만 고치면 역할이 빠진 채 실행됨
    • 결과: 기본 역할(구 계정)로 배포해 신 서버 대신 구 서버를 덮어씀

Task

두 호출의 입력을 두 레포 계약에 맞춤

  • 조건: 사이드 배포 옵션의 기본값은 꺼짐. 켰을 때만 이 경로를 탐

Action

호출 입력 전환

호출 이전 이후
extractor 주소·역할·요청 키 역할·요청 키
renderer 주소·요청 키 역할·요청 키
  • 역할은 이관용 역할
    • 이 역할의 신뢰 조건에 extractor·renderer 레포가 이미 들어 있음 (부트스트랩 스크립트 확인)

사이드 서버 확인 job 단순화

  • 서버 IP 출력 대신 두 서버의 존재만 확인
    • 원인: 넘길 주소가 없어짐
    • 서버가 없으면 배포 트리거 전에 멈추는 역할은 그대로

주석 갱신

Result

검증 범위

  • 두 레포 main 의 dispatch 입력 목록과 이관 워크플로가 넘기는 입력을 대조함
  • 이관 워크플로 검증 스크립트·actionlint 새 지적 0건
  • 위험: 실제 dispatch 는 이관 때만 돌아 미리 실행해 보지 못함

연관 이슈

- extractor·renderer 배포 호출에서 migration_host 제거
  - 원인: 두 레포가 SSM 전환 때 그 입력을 지움
    - 결과: 사이드 배포를 켜면 dispatch 가 Unexpected inputs(422)로 거부됨
- renderer 호출에 migration_role_arn(이관용 역할) 추가
  - 원인: 두 레포는 넘겨받은 역할의 계정에서 Name 태그로 박스를 찾음
    - 결과: 역할이 빠지면 기본 역할(구 계정)로 배포해 구 박스를 덮어씀
- side_hosts 는 IP 출력 대신 두 박스 존재 확인만 함
- core 배포 job 주석에서 cert 복사 언급 삭제 (#1219 로 provision 이관)
@m-a-king m-a-king added the infra 운영 환경 (IaC·클라우드 리소스·secret·배포 workflow) label Oct 15, 2026
@m-a-king m-a-king self-assigned this Oct 15, 2026
@github-actions

Copy link
Copy Markdown

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

@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: TeamPiKi/core/.coderabbit.yml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d0e22d89-0a71-4120-a93e-12905b05c830
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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 2fa9652 into dev Oct 15, 2026
8 checks passed
@m-a-king
m-a-king deleted the infra/migrate-side-dispatch branch October 15, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra 운영 환경 (IaC·클라우드 리소스·secret·배포 workflow)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant