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

So1S-207 argo cd image updater 적용 #12

Merged
merged 15 commits into from
Jul 26, 2022

Conversation

NEM-NE
Copy link
Member

@NEM-NE NEM-NE commented Jul 25, 2022

ArgoCD image updater 적용 - private Repository

Flow

image

latest에 대해서 반영이 됩니다.

Tasks

  • ArgoCD Image Updater 설치
  • Refactor Code (Pod Resource 변경, 파일 이름 변경, 불필요한 코드 삭제)
  • ArgoCD Image Updater Private Repository 반영
  • ArgoCD Image Updater App of Apps 패턴 적용

Discussion

Secret 적용 주의

@DPS0340 Private Repository에 접근하기 위해서 PAT(Personal Access Token)이 필요합니다. 따라서 해당 정보가 저장된 Secret 2개가 있습니다.(deployment imagePullSecret 용도, image-updater 프라이빗 레포지토리 접근 용도) 추후에 sealed-secret을 적용하실 때 참고 부탁드립니다~

https://github.com/So1S/deploy/blob/SO1S-207-argo-cd-image-updater-%EC%A0%81%EC%9A%A9/charts/argocd/image-updater-pull-secret.yaml

https://github.com/So1S/deploy/blob/SO1S-207-argo-cd-image-updater-%EC%A0%81%EC%9A%A9/charts/backend/templates/pull-secret.yaml

Jira

  • SO1S-207

@NEM-NE NEM-NE added Feat 기능이 추가됐습니다. Configuration 설정 파일이 변경됐습니다. Refactoring 코드를 개선 및 정리했습니다. Structure 프로젝트 구조가 변경됐습니다. (삭제 및 생성) labels Jul 25, 2022
@NEM-NE NEM-NE self-assigned this Jul 25, 2022
@NEM-NE NEM-NE force-pushed the SO1S-207-argo-cd-image-updater-적용 branch from 74c8dbf to 4fac7e1 Compare July 26, 2022 04:25
@DPS0340 DPS0340 merged commit 03d5e39 into main Jul 26, 2022
@DPS0340 DPS0340 deleted the SO1S-207-argo-cd-image-updater-적용 branch December 7, 2022 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Configuration 설정 파일이 변경됐습니다. Feat 기능이 추가됐습니다. Refactoring 코드를 개선 및 정리했습니다. Structure 프로젝트 구조가 변경됐습니다. (삭제 및 생성)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants