[DevOps] fix: FE, BE 배포 후 health check 코드 추가 - [merged] - Githubissues

qysim / DEVoca

DEVoca: 개발자를 위한 단어장

https://defiant-piccolo-ad9.notion.site/DEVoca-43b00aab7a2545bdbead9917bf7e96ba

0 stars 0 forks source link

[DevOps] fix: FE, BE 배포 후 health check 코드 추가 - [merged] #345

Closed qysim closed 7 months ago

qysim commented 7 months ago

In GitLab by @cheesecat47 on Feb 6, 2024, 14:57

_Merges DevOps/feat/78_healthcheck -> develop

이슈

78

이 MR의 목표

기존에 Jenkins로 FE, BE 빌드 배포를 자동화하고 성공 시 MM 알림을 보내는 작업은 해뒀으나, 서버 실행 후 health check 부분이 없었음. 때문에 서버 실행이 안 되어도 확인할 방법이 없었음.
그래서 서버 실행 후 health check를 하고 알림을 보내도록 변경.

변경점

FE health check는 main 사이트 주소로 확인.
BE는 health check용 엔드포인트를 추가.
Jenkinsfile 수정.

리뷰 주안점

BE에 엔드포인트 하나 추가했습니다. url이 겹치지는 않는 것 같습니다. 코드 컨벤션 맞는지 확인 부탁드릴게요.

참고 사항

@amysia @qysim

qysim commented 7 months ago

In GitLab by @cheesecat47 on Feb 6, 2024, 14:57

requested review from @amysia

qysim commented 7 months ago

In GitLab by @cheesecat47 on Feb 6, 2024, 14:58

added 14 commits

eea5a915...8c608edf - 13 commits from branch develop
a4fab472 - Merge remote-tracking branch 'origin/develop' into DevOps/feat/78_health_check

Compare with previous version

qysim commented 7 months ago

In GitLab by @amysia on Feb 6, 2024, 15:19

approved this merge request

qysim commented 7 months ago

In GitLab by @cheesecat47 on Feb 6, 2024, 16:12

mentioned in commit 69a84b6620054b2df3c927beed2efdbe06a00537