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

Backend CI/CD 파이프라인 설정 #11 #12

Merged
merged 5 commits into from
May 11, 2024
Merged

Backend CI/CD 파이프라인 설정 #11 #12

merged 5 commits into from
May 11, 2024

Conversation

F-hiller
Copy link
Member

@F-hiller F-hiller commented May 5, 2024

close #11

구현 기능

  1. PR 생성 등의 이벤트 발생 시 gradlew test 진행
  2. PR이 닫히면 build 후 docker를 활용하여 배포용 서버에 업로드 진행

추가 사항

  1. 현재 데이터베이스를 h2만 사용하고 있으므로 따로 컨테이너 설정을 하지 않았습니다. 만약 변경이 필요하다면 알려주세요.
  2. 추후에 데이터베이스를 환경(Prod, Test)에 따라 변수를 분리하여 작성하도록 하겠습니다.

@F-hiller F-hiller added the feat New feature or request label May 5, 2024
@F-hiller F-hiller requested review from sim-mer and fanta4715 May 5, 2024 08:38
@F-hiller F-hiller self-assigned this May 5, 2024
@F-hiller F-hiller linked an issue May 5, 2024 that may be closed by this pull request
2 tasks
image: redis:latest
container_name: redis
hostname: redis
ports:
Copy link
Collaborator

@fanta4715 fanta4715 May 5, 2024

Choose a reason for hiding this comment

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

이렇게 되면, 서버의 6379 포트로 레디스 접근이 가능해요!
8080만 열어두고, 6379는 내부망을 이용해서 외부 포트는 감추는 게 좋을 거 같아요
관련 내용은 도커 네트워크 찾아보시면 바로 적용할 수 있을 것 같습니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

expose 키워드가 있던데 ports 대신 해당 키워드 사용하면 될까요?

Copy link
Contributor

@sim-mer sim-mer left a comment

Choose a reason for hiding this comment

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

개인적으로 pull request발생 시 수행하는 github action test yaml과 CD yaml은 분리하는게 어떨까 생각합니다.

또한

- name: Publish Test Results
        uses: EnricoMi/publish-unit-test-result-action@v2
        if: always()
        with:
          junit_files: '**/build/test-results/test/TEST-*.xml'

이런 코드를 사용하면 테스트 리포트도 생성할 수 있습니다.

Copy link
Collaborator

@fanta4715 fanta4715 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@F-hiller F-hiller merged commit db95abf into develop May 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI/CD 기본 기능 구현
3 participants