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

[IDLE-248] github actions workflow 수정 #84

Merged
merged 6 commits into from
Aug 31, 2023
Merged

Conversation

minnseong
Copy link
Member

@minnseong minnseong commented Aug 30, 2023

🧑‍💻 작업 사항

모든 작업을 @0112leesy 와 하였습니다!!

Book-Service, User-Service ECR-ECS work flow 분리

  • bootstrap 환경 변수 지정으로 인해 Test step과 Build step 분리

api-gateway 서버 의존성 추가 및 수정

  • config server busrefresh를 위해 bus-amqp 의존성 추가
  • config-server 의존성을 config-client 의존성으로 변경

Book-Service, User-Service Dockerfile 수정 및 shell script 추가

  • fargate 동작시, eureka 서버에 private ip가 아닌 메타데이터의 ip가 등록된다.
  • shell script를 통해 런타임 시점에서 메타데이터 ip에 등록된 private ip를 가져오도록 하여, 이슈 해결

Book-Service, User-Service Request 로깅 추가

  • 컨트롤러 요청 로깅시, 요청한 유저 ID도 함께 로깅



🔗 링크



🐰 시급한 정도

🐢 천천히 : 급하지 않습니다.



📖 참고 사항

@minnseong minnseong self-assigned this Aug 30, 2023
@minnseong minnseong added bug Something isn't working config labels Aug 31, 2023
Copy link
Member

@0112leesy 0112leesy left a comment

Choose a reason for hiding this comment

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

이틀 동안 정말 고생많았습니다 😭 운영 환경의 이미지 빌드 자동화가 잘 마무리 된 것 같아서 다행이네요
다음에는 ECS, Code Deploy 관련 workflow를 작성해서 무중단 배포에 도전해봅시다!! 💪

@@ -254,9 +254,9 @@ void searchBookListByCategory() throws IOException {
// when
BookListByCategorySearchResultResponse response = aladinBookSearchApiService.searchBookListByCategory(request);

// then
// then정
Copy link
Member

Choose a reason for hiding this comment

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

r3: 정신없이 코드 고치시다가 한 글자 더 추가된 것 같네용 🫢 다음에 레포 이사 가신 다음에 수정하셔도 될 것 같습니다!

- config-server에서 config-client로 수정
- fargate 동작시, eureka 서버에 private ip가 아닌 메타데이터의 ip가 등록된다.
- shell script를 통해 런타임 시점에서 메타데이터 ip에 등록된 private ip를 가져오도록 함.
- with @0112leesy
@minnseong minnseong merged commit 2a9e8ad into master Aug 31, 2023
5 checks passed
@minnseong minnseong deleted the cicd/IDLE-248 branch August 31, 2023 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working config
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants