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

[모니터링] 배포 자동화 #423

Merged
merged 1 commit into from
Nov 10, 2024

Conversation

jyoo0515
Copy link
Member

  • helm pre-install 훅으로 secret 배포하려고 했는데,
  • deployment.yaml 이 아닌 values.yaml 에서 그 값을 읽어가는게 까다로워서 일단 gh action 자동화만 해둡니다
    • 또 secret 만들때는 ("/") 가 illegal character 라고 뭐라고 하는데 otel 설정에서는 "/" 가 없으면 안된다고 뭐라고 해서;;

Checklist

  • 충분한 양의 자동화 테스트를 작성했는가?
    • 계단정복지도 서비스는 사이드 프로젝트로 진행되는 만큼 충분한 QA 없이 배포되는 경우가 많습니다. 따라서 자동화 테스트를 꼼꼼하게 작성하는 것이 서비스 품질을 유지하는 데 매우 중요합니다.

@jyoo0515 jyoo0515 requested a review from a team as a code owner November 10, 2024 07:06
@jyoo0515 jyoo0515 merged commit 9f978d9 into main Nov 10, 2024
1 check passed
@jyoo0515 jyoo0515 deleted the jason/20241107-monitoring-cd-automation branch November 10, 2024 07:47
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.

1 participant