Skip to content

dev 앱 서버 보안 그룹 분리 - #1214

Merged
m-a-king merged 2 commits into
devfrom
infra/1213-dev-app-sg
Oct 18, 2026
Merged

m-a-king merged 2 commits into
devfrom
infra/1213-dev-app-sg

Conversation

@m-a-king

Copy link
Copy Markdown
Collaborator

한눈에

dev 앱 서버를 전용 보안 그룹으로 옮김

  • 이전: prod 앱 서버와 dev 앱 서버가 AWS 보안 그룹 하나를 같이 씀
    • prod DB 서버는 그 그룹에서 오는 DB 연결을 허용함
    • 그래서 dev 앱 서버도 네트워크상으로 prod DB 에 닿을 수 있음
  • dev 앱 서버만 새 보안 그룹으로 옮김
    • prod 앱 서버와 prod DB 서버의 보안 그룹은 바뀌지 않음
  • 결과: prod DB 는 prod 앱 서버에서만 닿음
  • 머지만으로는 아무것도 바뀌지 않음
    • 관리자가 terraform 을 적용해야 함. 순서는 아래 "적용 순서"

Situation

추가

  • dev 앱 서버 보안 그룹 분리 #1213
  • 추가
    • 이 그룹을 참조하는 다른 보안 그룹은 셋: prod DB·extractor·renderer (실측)
    • core 는 renderer 를 직접 부르지 않음. renderer 의 이 그룹 참조는 파싱 이관 전의 잔재

Task

범위

Action

dev 전용 보안 그룹

  • piki-dev-app-sg 를 새로 만들고 piki-dev-app 에 붙임
    • 입구는 prod 와 같은 80·443. 22 번 없음
    • 나가는 방향 전체 (extractor 호출·이미지 pull·SSM)
  • 인스턴스 보안 그룹 교체는 제자리 변경. 서버를 다시 만들지 않음
선택지 prod 영향 비용
dev 를 새 그룹으로 이동 (채택) 없음 prod 그룹 이름의 team3-dev 가 옛 이름으로 남음
prod 를 새 그룹으로 이동 prod DB·extractor·renderer 참조를 함께 바꿔야 함. 순서가 어긋나면 prod 파싱·DB 연결이 끊김 이름은 깔끔해짐

적용 순서

  1. core: terraform apply -target=aws_security_group.dev_app (새 그룹만 만듦)
  2. extractor: 새 그룹을 인바운드에 추가하는 PR 적용 (TeamPiKi/extractor PR)
  3. core: 전체 terraform apply (dev 앱 서버의 그룹 교체)
  • 원인: 2번 전에 3번을 하면 dev 앱 서버의 extractor 호출이 그 사이 거절됨

Result

검증

  • terraform validate 통과
  • plan 은 돌리지 않음
    • 원인: plan 도 관리자 역할(Touch ID)로만 돌게 한 권한 모델을 따름
    • 예상 plan: 보안 그룹 생성 1·dev 인스턴스 제자리 변경 1. prod 리소스 변경 0
  • 위험: dev 앱 서버의 다른 인바운드 경로가 콘솔에서 이 그룹에 추가돼 있었다면 사라짐
    • 실측: 현재 그룹 인바운드는 80·443 둘뿐

후속

  • 관리자 적용(위 순서)
  • 적용 뒤 dev 상품 등록으로 extractor 호출 확인

연관 이슈

- dev 전용 보안 그룹 piki-dev-app-sg 를 만들고 piki-dev-app 을 옮김
  - 입구는 prod 와 같은 80·443, 나가는 방향 전체
- prod 앱 서버·prod DB 보안 그룹은 그대로
  - 결과: prod DB 의 MySQL·Redis 인바운드가 prod 앱 서버에서만 닿음
  - 이전: 같은 그룹을 쓰는 dev 앱 서버도 닿음
  - 원인: dev 는 자기 서버의 MySQL·Redis 를 씀 (SSM db-host·redis-host 172.17.0.1). prod DB 연결이 필요 없음
- dev 를 옮기고 prod 를 남김
  - 대안: prod 전용 그룹을 새로 만들고 prod 를 옮김
    - 기각: prod DB·extractor·renderer 참조를 함께 바꿔야 하고 순서가 어긋나면 prod 가 끊김
  - 비용: prod 그룹 이름의 team3-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: Repository: TeamPiKi/core/.coderabbit.yml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 546d5680-6cad-4a6f-837b-293e7e0e5d7f
  • 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 c4a1368 into dev Oct 18, 2026
5 checks passed
@m-a-king
m-a-king deleted the infra/1213-dev-app-sg branch October 18, 2026 21:36
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