Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Github Action 수행 시 AWS System Manager에서 config 파일 가져옴 #57

Merged
merged 1 commit into from
Dec 24, 2021

Conversation

umi0410
Copy link
Member

@umi0410 umi0410 commented Dec 24, 2021

🏋️‍♀️ PR 요약

Github Action 수행 시 ssm parameter에서 config 파일 가져옵니다.

기존에는 credential한 내용들을 모두 Github action secret으로 관리하다보니 현재 어떤 값이 설정되어있는지. 어떤 내용이 바뀌었는지. 각자가 갖고 있는 설정은 어떠한지를 알 수가 없어 불편했는데, AWS System Manager를 통해 설정들을 가져옴으로써 편리하게 이용 가능합니다.

  • 변경 내역 추적(버져닝) 가능
  • 누가 수정했는지 기록 남음
    • 이 기능뿐만 아니라 이제 출시도 했으니 저희끼리도 AWS의 root 계정을 최대한 이용 안하는 게 좋을 것 같습니다.

📌 변경 사항

  • Github Action에서 deploy하는 action이 조금 바뀌었어요.

📸 ScreenShot

image
AWS SystemManager - Parameter store 가시면 볼 수 있습니다!
이름은 가리는 게 좋을 것 같아 가리고 찍었어요.

✅ PR check list
- 적절한 branch로 요청했는지 확인해주세요.
- Assignees, Label, Reviewer를 설정해주세요.
- 관련된 Issue Link 해주세요.
- PR이 승인된 경우 해당 브랜치는 삭제해주세요.

Linked Issue

close #

@umi0410 umi0410 requested review from ktyu and atg0831 December 24, 2021 10:01
@umi0410 umi0410 self-assigned this Dec 24, 2021
@umi0410 umi0410 merged commit f2f4f5e into main Dec 24, 2021
@umi0410
Copy link
Member Author

umi0410 commented Dec 24, 2021

Spring은 Vault나 Consul 같은 외부 시크릿 서비스들이랑 연동되는 경우도 봤던 것 같은데, AWS SystemManager + Go의 조합에서는 그런 기능을 제공하는 믿을만한 오픈소스는 딱히 없는 것 같네요.

현재 저희가 configuration 관리/적용에 사용하는 viper의 경우 #877 "Is it possible to add support for AWS System Manager parameter store?" 이런 이슈가 열리기도 했더라구요.

일단은 그래서 프로그램 내부에서 SystemManager로부터 컨피그를 땡겨오는 것보단 배포할 때 Github Action으로 넣어주는 게 간단할 것 같아서 이렇게 구현해봤습니다.

umi0410 added a commit that referenced this pull request Dec 24, 2021
feat: Github Action 수행 시 AWS System Manager에서 config 파일 가져옴
@atg0831
Copy link
Member

atg0831 commented Jan 6, 2022

Good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants