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

app-serving: 파이프라인 상의 버튼 위치 조정 #68

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

robertchoi80
Copy link
Contributor

@robertchoi80 robertchoi80 requested a review from Siyeop June 1, 2023 00:17
Copy link
Member

@Siyeop Siyeop left a comment

Choose a reason for hiding this comment

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

사용 시나리오에 영향을 주게 되는 요소이니 Jira TC도 같이 수정해주시면 좋을 것 같습니다.
고생하셨습니다. 👍

Siyeop

This comment was marked as duplicate.


action = domain.ActionResponse{
} else if stage.Status == "PROMOTE_WAIT" && strategy == "blue-green" {
action := domain.ActionResponse{
Name: "PROMOTE",
Uri: fmt.Sprintf(internal.API_PREFIX+internal.API_VERSION+
"/organizations/%v/app-serve-apps/%v", app.OrganizationId, app.ID),
Copy link
Member

Choose a reason for hiding this comment

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

@ktkfree @robertchoi80
단순 궁금증입니다.

백앤드 코드에 fmt.Sprintf 로 되어있는 로그 출력이 종종 보이는데요.
로그 관리해주는 라이브러리는 사용하지 않고 있나요?
로그 파일로 떨구거나.. 로그레벨 등을 조절하려면 필요할 것 같아 보여서 질문드리빈다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

음.. 이 부분은 사실 제가 작성한 부분은 아니지만 대신 답변 드리자면,

  • 이 라인에 한정해서 말씀드리면 Sprintf 라서 단순히 string을 구성하는 거고 로그에 찍히는 부분은 아닌듯 합니다.
  • 그런데, 다른 곳곳에서도 log 라이브러리 대신 fmt.Printf 를 쓰는 경우가 있는데, 이유는 Printf와 %v 또는 %+v 포맷을 사용하면, struct나 map 등의 구조체 전체 내용을 출력해줘서 (디버깅에도 용이하다보니) 그럴때만 한정적으로 사용하고 있습니다.
  • 로그 라이브러리가 해당 기능을 커버하도록 확장된다면 그때는 Printf 사용이 없어질 것 같습니다.

Copy link
Member

Choose a reason for hiding this comment

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

아아.. Sprintf 는 단순 스트링 구성을 위한 함수였군요.
설명 감사합니다 😇

Copy link
Member

Choose a reason for hiding this comment

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

log 라이브러리를 하면 구조체 출력할 때
[Object object] 뭐 이런식으로 찍히거나 주소값만 출력되어서 그런것 같네요.

이런 상황을 해결하기 위해 아래 2가지 방법을 사용했던 것 같은데요. 참고삼아 공유드립니다.

  • VO 클래스 구성할 때는 toString을 구현해놓은 클래스를 상속받게 한 적이 있었고요.
  • Helper 함수를 만들어서 아래처럼 사용한 적도 있었던 것 같아요.
logger.info(
   LoggerHelper.toString(구조체)
)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ktkfree 님도 위 내용 참고하시면 좋을 듯 하네요. 전에 비슷한 질문하신 적 있으셔서요.

@robertchoi80 robertchoi80 merged commit edc58ed into develop Jun 1, 2023
@ktkfree ktkfree deleted the fix-stage-actions branch June 19, 2023 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants