Skip to content

dev 앱 서버 보안 그룹을 추출 인바운드에 추가 - #114

Open
m-a-king wants to merge 1 commit into
mainfrom
infra/113-dev-app-sg
Open

m-a-king wants to merge 1 commit into
mainfrom
infra/113-dev-app-sg

Conversation

@m-a-king

Copy link
Copy Markdown
Collaborator

한눈에

extractor 가 dev 앱 서버의 새 보안 그룹에서 오는 호출을 받게 함

  • core 가 dev 앱 서버를 전용 보안 그룹으로 옮김 (dev 앱 서버 보안 그룹 분리 core#1214)
  • extractor 서버는 dev·prod 앱 서버의 추출 호출을 둘 다 받음
  • 그래서 추출 포트의 허용 대상에 dev 앱 서버의 새 그룹을 더함
  • 머지만으로는 아무것도 바뀌지 않음
    • 관리자가 terraform 을 적용해야 함
    • core 의 새 그룹 생성 뒤, dev 앱 서버의 그룹 교체 앞에 적용

Situation

추가

Task

범위

Action

추출 포트 인바운드

  • prod 앱 그룹(team3-dev-ec2-sg, 옛 이름)과 dev 앱 그룹(piki-dev-app-sg)을 허용
  • 새 그룹은 이름으로 data 조회

적용 순서 (세 PR 합본)

  1. core: terraform apply -target=aws_security_group.dev_app
  2. extractor: 이 PR terraform apply (보안 그룹 제자리 변경 1)
  3. core: 전체 terraform apply (dev 앱 서버 그룹 교체)
  • 원인: 1번 전에는 data 조회가 새 그룹을 못 찾아 plan 이 깨짐
  • 원인: 3번이 먼저면 dev 앱 서버의 추출 호출이 그 사이 거절됨

Result

검증

  • terraform validate 통과
  • plan 은 돌리지 않음. 관리자 역할(Touch ID)로만 도는 권한 모델을 따름
  • 위험: 보안 그룹 설명 문구는 그대로 둠. 바꾸면 그룹이 다시 만들어짐

연관 이슈

- 추출 포트 인바운드를 prod 앱 그룹과 dev 앱 그룹 둘로 넓힘
  - 원인: core#1213 이 dev 앱 서버를 전용 그룹 piki-dev-app-sg 로 옮김
  - 이 서버 한 대가 dev·prod 추출 호출을 둘 다 받음
- 변수 설명의 "세 인스턴스가 그룹 하나를 공유" 서술을 현행으로 고침
- 적용 순서: core 의 새 그룹 생성 뒤, dev 앱 서버의 그룹 교체 앞
  - 원인: data 조회가 새 그룹을 이름으로 찾음. 교체가 먼저면 그 사이 dev 추출 호출이 거절됨
@m-a-king m-a-king added the infra 운영 환경 (IaC·클라우드 리소스·secret·배포 workflow) label Oct 14, 2026
@m-a-king m-a-king linked an issue Oct 14, 2026 that may be closed by this pull request
@m-a-king m-a-king self-assigned this Oct 14, 2026
@github-actions

Copy link
Copy Markdown

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

@coderabbitai

coderabbitai Bot commented Oct 14, 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: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2938889c-9444-4d4f-9285-4804fbb7938a
  • 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.

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.

dev 앱 서버 보안 그룹을 추출 인바운드에 추가

1 participant