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

Spring 환경설정 #405

Merged
merged 12 commits into from
Feb 15, 2024
Merged

Spring 환경설정 #405

merged 12 commits into from
Feb 15, 2024

Conversation

sk000801
Copy link
Member

Issue

Overview

Spring을 위한 환경설정 노션 글에 고민 과정과 실행 과정을 자세히 기록해놓았습니다!

To Reviewers

도커 컴포즈로 nest 서버랑 한꺼번에 관리하면 좋을 것 같은데, 아직 잘 모르겠어서 일단 따로따로 배포하는 코드 작성했어요 ㅠㅠ
그리고 Spring CI/CD yml 파일도 따로 만들었는데, 이게 맞는지도 모르겠네요ㅠㅠ

읽어보시고 이상한 거 코멘트 달아주시고 이 브랜치에서 고쳐주세요!!

* 서버 폴더가 nest, spring으로 분리됨에 따라 /nest 경로 추가
* 서버 폴더가 nest, spring으로 분리됨에 따라 /nest 경로 추가
* server 하위의 spring 폴더 추가
* DB 스키마 싱크를 맞추기 위해 기존 설정인 true로 다시 변경 (변경 파일이 많아져 실수함)
* 기존의 ci/cd 코드가 node기준이므로 trigger 기준을 server/nest 내부의 코드로 변경
* spring 컨테이너 관련 Dockerfile도 필요하여 파일명 변경
* spring 서버 도커 이미지를 빌드하기 위한 dockerfile 작성
* 이후 서버에 정상적으로 띄워졌는지 확인하기 위한 API 작성
* dockerfile이 여러 개가 되어 docker build 경로 상세 지정
* 트리거 브랜치 수정
* 기존 shell script에서 포트 번호만 변경
* spring 프로젝트를 빌드하는  yml 파일 작성
* spring 테스트 코드를 실행하는 스크립트 작성
@sk000801 sk000801 added ✨ feat 기능 개발 🖥 server server labels Feb 14, 2024
@sk000801 sk000801 added this to the 🔧 etc milestone Feb 14, 2024
@sk000801 sk000801 self-assigned this Feb 14, 2024
@sk000801 sk000801 requested a review from khw3754 as a code owner February 14, 2024 11:48
@sk000801 sk000801 linked an issue Feb 14, 2024 that may be closed by this pull request
* checkout distribution 변경
* https://github.com/actions/setup-java 링크 참고
* checkout distribution 변경
* https://github.com/actions/setup-java 링크 참고
Copy link
Collaborator

@khw3754 khw3754 left a comment

Choose a reason for hiding this comment

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

우와 spring 초기 세팅부터 ci, cd 까지 하느라 고생하셨네요 🥲 노션 글 하나하나 꼼꼼히 읽어봤습니다!
아직 딱히 오류가 날 부분을 잘 몰라서 같이 개발하면서 맞춰야할 것 같아요

@khw3754
Copy link
Collaborator

khw3754 commented Feb 15, 2024

그리고 혹시 스프링을 서브도메인으로 배포를 하신 이유가 있나요? nest의 auth 모듈은 두고 spring 인증 인가가 잘 구성될 때까지 테스트하면서 분리를 하고 싶으셔서 그런거죠? 개발이 끝나면 나중에 도메인을 합쳐야하지 않을까 해서 여쭤봅니다!

@khw3754 khw3754 merged commit 4e4bf9f into develop Feb 15, 2024
2 checks passed
@khw3754 khw3754 deleted the server/feature/404 branch February 15, 2024 07:23
@sk000801
Copy link
Member Author

sk000801 commented Feb 15, 2024

@khw3754
auth 모듈 spring 환경에서 잘 돌아갈 때까지 분리하려구 한 것도 맞는데 사실 서브 도메인으로 둔 게 nginx에서 spring이랑 nest 컨테이너로 각자 포트 포워딩 시켜주려면 하나의 도메인으로 두는게 안되지 않나..? 싶어서 그랬어요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 개발 🖥 server server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

스프링 환경설정
2 participants