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] Docker 기반 ci/cd 구축 #8

Closed
wants to merge 54 commits into from
Closed

[feat] Docker 기반 ci/cd 구축 #8

wants to merge 54 commits into from

Conversation

sansan20535
Copy link
Contributor

변경사항

.gitignore

### macOS ###
# General
.DS_Store
### secret ###
application-secret.properties

  • .DS_STORE과 중요변수를 저장해 놓은 application-secret.properties를 추가했습니다.

CI

on:
push:
branches: [ "**" ]

  • 모든 브랜치에서 push를 하면 CI를 하도록 하였습니다. (현재 파트장님의 의견)
  • 이후 develop으로 변경하면 될 것 같습니다.

- name: create application-secret.properties
run: |
## create application-secret.properties
cd ./src/main/resources
# application-secret.properties 파일 생성
touch ./application-secret.properties
# GitHub-Actions 에서 설정한 값을 application-secret.properties 파일에 쓰기
echo "${{ secrets.PROPERTIES_APPLICATION }}" >> ./application-secret.properties
- name: create application.yml
run: |
## create application.yml
cd ./src/main/resources
# application.yml 파일 생성
touch ./application.yml
# GitHub-Actions 에서 설정한 값을 application.yml 파일에 쓰기
echo "${{ secrets.CI_APPLICATION }}" >> ./application.yml
shell: bash

  • 중요 변수는 PROPERTIES_APPLICATION에, 개발용 yaml파일은 CI_APPLICATION과 CD_APPLICATION에 저장했습니다.
  • 한 번에 불러오는 경우 오류가 발생해서, 우선 properties를 카피하고 다음에 CI, CD를 카피했습니다.

CD (Docker)

FROM amd64/amazoncorretto:21
WORKDIR /app
COPY ./build/libs/offloadserver-0.0.1-SNAPSHOT.jar /app/offroad.jar
CMD ["java", "-Duser.timezone=Asia/Seoul", "-jar", "-Dspring.profiles.active=dev", "offroad.jar"]

  • Dockerfile을 추가했습니다.

build.gradle

  • implementation 'org.springframework.boot:spring-boot-starter-actuator' 의존성을 추가했습니다.

Test

image

질문사항

  • scripts와 scripts/deploy.sh 는 이제 필요 없는 것 같은데 삭제를 해도 좋을까요??
  • 이상한 점이나 고쳐야되는 점이 보이시면 바로 말씀해 주시면 감사하겠습니다!!

@sansan20535 sansan20535 added Feature New feature or request Setting about setting labels Jun 24, 2024
@sansan20535 sansan20535 self-assigned this Jun 24, 2024
@sansan20535 sansan20535 requested a review from a team as a code owner June 24, 2024 13:21
@jun3327
Copy link
Contributor

jun3327 commented Jun 24, 2024

확인했습니다! scripts파일은 없어도 될 것 같아요. CI/CD 부분은 윤한님 확인 한 번 더 하시면 머지하면 좋을 것 같습니다

@sansan20535 sansan20535 changed the title Task/cicd [feat] Docker 기반 ci/cd 구축 Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request Setting about setting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants