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

[refactor] sql 위치 변경 및 ecr 이미지 사용 관련 #309

Merged

Conversation

linglong67
Copy link
Collaborator

💡 Motivation and Context

1) sql 위치 변경 2) ecr 이미지 사용


🔨 Modified

Flyway sql 파일 위치 변경 (repackage)

  • module-api 에서 module-domain 으로 이동

ecr 이미지 추가

  • 개발 api, batch 모두 ecr 레포지토리로 이미지 푸시 경로 변경함
    • docker hub에 올라간 이미지는 이번 PR merge된 후 삭제 예정
  • 운영 batch도 ecr 레포지토리에 푸시하도록 변경함

🌟 More

  • ...


📋 커밋 전 체크리스트

  • 추가/변경에 대한 단위 테스트를 완료하였습니다.
  • 컨벤션에 맞게 작성하였습니다.

🤟🏻 PR로 완료된 이슈

closes #307
closes #308

@linglong67 linglong67 added 🖥️ BackEnd 서버 관련 🔨 Refactor 이 코드는 아주 약간 더 클린코드에 가까워졌습니다... ⚙️ Settings 프로젝트 설정, 혹은 기타 환경설정 labels Mar 18, 2024
@linglong67 linglong67 self-assigned this Mar 18, 2024
run: |
docker build -t ${{ secrets.DOCKERHUB_USERNAME }}/${{ secrets.MODULE_API_NAME }} . --platform=linux/amd64
docker push ${{ secrets.DOCKERHUB_USERNAME }}/${{ secrets.MODULE_API_NAME }}
docker build -t $REGISTRY/$REPOSITORY:$IMAGE_TAG .
Copy link
Collaborator

Choose a reason for hiding this comment

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

ecr로 이관 확인하였습니다

Copy link
Collaborator

@HyunJunSon HyunJunSon left a comment

Choose a reason for hiding this comment

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

sql 파일 위치변동, ECR login 변동 등 설정변경 확인하였습니다.

@linglong67 linglong67 merged commit bb78a12 into Kernel360:develop Mar 18, 2024
@linglong67 linglong67 deleted the refactor/sql-repack-and-add-ecr-upload branch March 18, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖥️ BackEnd 서버 관련 🔨 Refactor 이 코드는 아주 약간 더 클린코드에 가까워졌습니다... ⚙️ Settings 프로젝트 설정, 혹은 기타 환경설정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ECR Repo 관련 작업 Flyway sql 파일 리팩키징 (모듈 변경)
3 participants