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: 젠킨스 to 깃헙액션 #430

Merged
merged 4 commits into from
Jun 29, 2024
Merged

feat: 젠킨스 to 깃헙액션 #430

merged 4 commits into from
Jun 29, 2024

Conversation

hocaron
Copy link
Collaborator

@hocaron hocaron commented Jun 22, 2024

PR 타입

개요

  • 젠킨스 to 깃헙액션

변경사항

script: |
aws ecr get-login-password --region ap-northeast-2 | docker login --username AWS --password-stdin 321148231533.dkr.ecr.ap-northeast-2.amazonaws.com
cd /home/ec2-user/mashup-server/docker/deploy/admin/dev
sh ./deploy-admin.sh $TAG
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어떤건 TAG가 없는데 어떤 차인가요

Copy link
Collaborator Author

@hocaron hocaron Jun 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

지금은 없지만, dev 나중에 develop 에 머지되어야지만 develop 으로 태그만들어지는 방식외에도 별도의 브렌치에서 태그 만드는 방식으로 가려면 스크립트에 태그를 추가하려고 넣어줬습니다!

현재는 태그 넣어도 develop 으로만 되기는 합니당~! sh, docer-compose 수정 필요합니다.

prod 는 master 에 머지되어야지만 태그 만들어지는 방식으로만 현상 유지할 것 같아서, 태그 없애놓은 건데 덜 지웠네요! prod 는 다 지워놓을게욧!

@hocaron
Copy link
Collaborator Author

hocaron commented Jun 23, 2024

@kh0712 리얼 환경 배포는 관리자만 배포할 수 있도록 권한 체크하는 코드 추가해놓았어용! 요것도 같이 체크 부탁드립니다🙇‍♀️

(저 다시 어드민으로 권한 수정 부탁드립니다ㅎㅎ🥲)

https://github.com/mash-up-kr/mashup-server/actions/runs/9631230964/job/26562761086

Copy link
Collaborator

@kh0712 kh0712 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kh0712 kh0712 merged commit 2001d8e into develop Jun 29, 2024
8 of 9 checks passed
@hocaron hocaron deleted the feature/cd-github-action branch September 19, 2024 14:04
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