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

init k8s-monitoring #100

Merged
merged 2 commits into from
Jul 9, 2023
Merged

init k8s-monitoring #100

merged 2 commits into from
Jul 9, 2023

Conversation

PFCJeong
Copy link
Member

@PFCJeong PFCJeong commented Jul 8, 2023

이전 PR 내리고 새로 올려요.

생각해보니 웹서버로 안띄울거라 메모리 250이면 충분할 듯.

그리고 templates 하위에는 안넣었습니다... 구속이 싫어... 괜찮을까요? 허락좀 부탁드립니다ㅎㅎ

@PFCJeong PFCJeong requested review from a team and davin111 as code owners July 8, 2023 02:24
Copy link
Member

@davin111 davin111 left a comment

Choose a reason for hiding this comment

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

넹 원하는 application 의 경우 수동 배포 하면 될 거 같습니다.

요게 정의된 Argo Project or Application 이 어디있을까요? 이걸 보고 역으로 찾아갈 수 있게 주석 정도는 달아주면 좋을 거 같아요.

@davin111
Copy link
Member

davin111 commented Jul 8, 2023

아 repo 아예 새로 파셨군요. 요거 배포는 어떤 방식으로 하시는 건가요? 배포를 위해 새로운 docker image 를 뽑으면… 그 후 일어나는 일이 궁금합니다. https://github.com/wafflestudio/k8s-monitoring 한 번만 설명해주세여 ㅋㅋ 제가 안 익숙해서.

@PFCJeong PFCJeong force-pushed the feature/k8s-monitoring branch 5 times, most recently from fb813f0 to 555df61 Compare July 9, 2023 04:01
name: k8s-monitoring-secret
namespace: k8s-monitoring
stringData:
# SLACK_TOKEN:
Copy link
Member

Choose a reason for hiding this comment

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

아… 이거 방금 메일 왔는데 public 으로 아까 push 되어서 deactivate 된듯… 순간의 실수엿죠?

Copy link
Member Author

Choose a reason for hiding this comment

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

와 ㅇㅇㅇ.. deactivate 바로되네?? 아오

Copy link
Member

Choose a reason for hiding this comment

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

슬랙 봇 권한 있으시던가요? 직접 기존 봇 재발급 가능?

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅇㅇㅇㅇ네네 트러플 slack token 재발급해서 secret 적용해두었음.
안되겠다 봇은 새로 만들게요 ㅎㅎ

Copy link
Member

Choose a reason for hiding this comment

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

Bot 자체를 새로 만들 필요가 있나여?

Copy link
Member Author

Choose a reason for hiding this comment

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

봇 이름이 truffle이라;; 그냥 트러플쓸까?

Copy link
Member

Choose a reason for hiding this comment

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

아아 truffle 봇이랑 구분하려고…. 어느 쪽이든 상관 없는데 분리해도 되긴 하겠네용

truffle 쪽은 기존 slack token SecretsManager 에 있었을 거 같은데 바꿨다면 pod 재시작 해줘야할듯! 이미 했을 수 있겠지만

Copy link
Member Author

Choose a reason for hiding this comment

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

맘 바뀜. 그냥 같이 쓸게요. truffle은 시크릿매니저 안쓰고 있어여. 제가 시크릿매니저를 싫어해서..

Copy link
Member

@davin111 davin111 Jul 9, 2023

Choose a reason for hiding this comment

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

아 맞다 그랬었지 그냥 k8s secret 에…

근데 별개로 #82 도입하면 AWS SecretsManager 에 값 넣어놓고 애플리케이션에서 코드로 SM 에 의존할 일은 없어지고 평범한 k8s secret 에서 읽게 할 수 있어서, 이렇게 하고 나면 쓰는 건 어떤가요?

요즘 우리 회사에서도 이렇게 하는 중.

실수할 일도 적고 kubectl 권한 없는 사람도 다 동일하게 사용 가능.

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅇㅇ 제발 좀 도입해주세요. 시크릿매니저를 안쓴느 저는 지금 개불편함 ㅠ

@davin111
Copy link
Member

davin111 commented Jul 9, 2023

별개로 수동 배포 방식 어케 하시는지 한 번 남겨놔주세요. #100 (comment)

@PFCJeong
Copy link
Member Author

PFCJeong commented Jul 9, 2023

@davin111 리뷰 반영했습니다. 확인 부탁드림!

@PFCJeong
Copy link
Member Author

PFCJeong commented Jul 9, 2023

도터 이미지 빌드, 푸시 다 제가 로컬에서 할거에요.

배포만 waffle-world에 버전 태그 올려서 하려구 합니다.

@davin111
Copy link
Member

davin111 commented Jul 9, 2023

ㅇㅎ… docker push 자체는 와플 ecr 에 하는 거죠?

먼가 docker build 시 특별한 환경변수 넣거나 하는 건 필요 없는 건가

@PFCJeong PFCJeong force-pushed the feature/k8s-monitoring branch 2 times, most recently from db6983e to 748537c Compare July 9, 2023 04:33
@PFCJeong PFCJeong merged commit 6f40898 into main Jul 9, 2023
@PFCJeong PFCJeong deleted the feature/k8s-monitoring branch July 9, 2023 04:38
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