Skip to content

관리자 그룹 소속의 terraform 프로필 예외 추가 - #144

Merged
m-a-king merged 2 commits into
mainfrom
docs/piki-tf-admin-group
Oct 15, 2026
Merged

m-a-king merged 2 commits into
mainfrom
docs/piki-tf-admin-group

Conversation

@m-a-king

Copy link
Copy Markdown
Contributor

한눈에

관리자 그룹 소속의 terraform 프로필 예외를 규약에 추가

  • 이전: 규약이 terraform 프로필을 관리자 역할 기준으로만 적음
    • 관리자 그룹 소속은 평소 로그인이 이미 관리자 권한
    • 결과: 규약과 실제 노트북 설정이 어긋남
  • 관리자 그룹 소속은 역할을 바꾸지 않는다는 예외를 적음
  • 그 경우 terraform 프로필이 평소 로그인을 그대로 쓰게 안내함
  • 머지 직후 바뀌는 서비스 없음

Situation

규약과 실제 설정의 차이

  • 서버 접속 규약 갱신 #134 가 terraform 프로필을 관리자 역할(Touch ID 로그인 뒤 20분 안에만 전환)로 고정함
  • PiKi-Developer 그룹(AdministratorAccess) 소속은 그 그룹에 남기로 결정
    • 평소 프로필만으로 terraform 을 돌릴 수 있음
  • 이 그룹은 core terraform/iam_people.tf 에 없음. terraform 밖에서 관리

Task

5절에 그룹 소속 예외 추가

Action

5절 갱신

대상 이전 이후
권한 두 층 설명 예외 없음 PiKi-Developer 소속은 역할 전환 없음
노트북 준비의 terraform 프로필 관리자 역할 고정 그룹 소속이면 평소 프로필로 바꾸라는 주석
사용 예의 20분 안내 무조건 관리자 역할 경로에만 해당
  • 비용: 이 그룹 소속에는 Touch ID 뒤 20분 제한이 걸리지 않음
    • 로그인 세션이 살아 있는 동안 계속 관리자 권한

연관 이슈

- PiKi-Developer(AdministratorAccess) 소속은 관리자 역할로 바꾸지 않는다는 예외 추가
  - 원인: 평소 권한이 곧 관리자라 역할 전환이 필요 없음
  - 이 그룹은 terraform 밖에서 관리함 (iam_people.tf 에 없음)
- 노트북 준비의 piki-tf 블록에 그 경우 --profile piki 로 둔다는 주석 추가
- 사용 예의 20분 제한 주석을 관리자 역할 경로로 한정
@m-a-king m-a-king added the docs 문서만 만지는 작업 label Oct 15, 2026
@m-a-king m-a-king self-assigned this Oct 15, 2026
@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: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d10cbd4d-28a4-448a-af99-94ee67cdb024
  • 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 a58d265 into main Oct 15, 2026
4 checks passed
@m-a-king
m-a-king deleted the docs/piki-tf-admin-group branch October 15, 2026 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs 문서만 만지는 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant