Skip to content

feat(application-template): hook에서 ArgoCD hook annotation 지원 추가#45

Merged
atobaum merged 3 commits intomainfrom
feat/MD-23431-MD-23568-argocd-hooks-support
Mar 19, 2026
Merged

feat(application-template): hook에서 ArgoCD hook annotation 지원 추가#45
atobaum merged 3 commits intomainfrom
feat/MD-23431-MD-23568-argocd-hooks-support

Conversation

@atobaum
Copy link
Copy Markdown
Member

@atobaum atobaum commented Mar 18, 2026

Summary

  • hook.jobs[].argocd 블록 설정 시 ArgoCD annotation 사용, 미설정 시 기존 Helm hook 동작 유지 (하위 호환)
  • hook.jobs[].vault.argocd 블록으로 vault Secret에도 동일하게 적용
  • vault 미설정 시 nil pointer 버그 수정: .vault 대신 (.vault) 사용하면 js의 ?? 처럼 nil일때도 오류 안난다고 합니다.
  • helm-unittest 유닛 테스트 26개 추가
    • helm-unittest 이 제일 많이 쓰는 것 같아서 추가했습니다.

Usage

hook:
  enabled: true
  jobs:
    - name: db-migration
      argocd:
        hook: PreSync
        syncWave: "-1"
        hookDeletePolicy: BeforeHookCreation
      vault:
        enabled: true
        path: argo-system:myapp-stage
        argocd:
          hook: PreSync
          syncWave: "-2"
          hookDeletePolicy: HookSucceeded
        secrets:
          MY_SECRET: <MY_SECRET>

Jira: MD-23431 MD-23568

Helm hook annotation만 지원하던 hook 템플릿에 ArgoCD hook annotation을
선택적으로 사용할 수 있도록 수정. 각 job에 argocd 블록을 설정하면
argocd.argoproj.io/hook, sync-wave, hook-delete-policy annotation이
적용되며, 미설정 시 기존 helm.sh/hook 동작을 유지함 (하위 호환).

vault Secret에도 동일하게 vault.argocd 블록으로 ArgoCD annotation 지원.
nil pointer 접근 버그(vault 미설정 시 vault.enabled 참조) 함께 수정.
@atobaum atobaum self-assigned this Mar 18, 2026
atobaum added 2 commits March 18, 2026 23:20
helm-unittest을 사용하여 hooks 템플릿의 ArgoCD/Helm hook annotation
렌더링을 검증하는 테스트 스위트 추가. 총 26개 테스트 케이스로 Job,
Secret, ServiceAccount의 annotation 동작을 커버.

테스트 파일은 helm package 아티팩트에서 제외되도록 .helmignore에
tests/ 경로 추가. README에 테스트 설치 및 실행 방법 문서화.
@atobaum atobaum requested a review from hschoi1104 March 18, 2026 14:40
@atobaum atobaum merged commit 28a2a0f into main Mar 19, 2026
@atobaum atobaum deleted the feat/MD-23431-MD-23568-argocd-hooks-support branch March 19, 2026 02:22
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