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

chore: 인프라 정리 #282

Merged
merged 35 commits into from
Jan 5, 2025
Merged

chore: 인프라 정리 #282

merged 35 commits into from
Jan 5, 2025

Conversation

clean2001
Copy link
Collaborator

📝 PR 타입

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 리팩토링
  • docs 작업, swagger 작업
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

📢 변경 사항

  • 헬스체크 api를 액추에이터로 변경하고, 다른 헬스체크 api(/greeting) 제거
  • Dockerfile-prod, Dockerfile-dev > Dockerfile로 변경
  • dev, prod 배포 방식 통일

❗️To Reviewer

  • docker-compose 파일을 다른 모듈에서도 사용하므로, 디렉토리 최상단으로 옮기는 것이 어떤 것에 대해 피드백 해주셨는데, github actions 파일 내용을 보면, layer-api/infra/{환경}를 scp로 ec2에 전송한 다음 그 안에 있는 도커 컴포즈 파일을 실행하는 거여서 layer-api/infra/{환경} 경로에 있는 것이 편할 것 같아 변경하지 않았습니다. (혹시 제가 피드백을 이해하지 못했거나, 더 좋은 방법이 있으시다면 알려주세요!)

⚙️ 테스트 결과

발생한 쿼리 첨부

👉 반영 브랜치

  • feat/
  • closed #

Copy link
Collaborator

@mikekks mikekks left a comment

Choose a reason for hiding this comment

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

폭풍커밋... 정말 고생하셨습니다!!!

액츄에이터 관련해서만 의견여쭤보고 싶습니다 !!

layer-batch/Dockerfile Outdated Show resolved Hide resolved
layer-api/infra/production/deploy.sh Show resolved Hide resolved
layer-api/Dockerfile Show resolved Hide resolved
Copy link
Collaborator

@mikekks mikekks left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !!

@clean2001 clean2001 merged commit 56028f0 into develop Jan 5, 2025
3 checks passed
@clean2001 clean2001 deleted the chore/LA-33 branch January 5, 2025 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants