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

do not monitor etcd on eks #137

Merged
merged 1 commit into from
May 15, 2023
Merged

do not monitor etcd on eks #137

merged 1 commit into from
May 15, 2023

Conversation

intelliguy
Copy link
Contributor

eks에서 etcd를 삭제하기 위해서는 추후 사용하는 부분까지 삭제해야함

@intelliguy intelliguy force-pushed the remove-etcd-on-eks branch from b3c4b08 to a1fe2a9 Compare May 15, 2023 04:36
@robertchoi80
Copy link
Contributor

@intelliguy 님~ eks-msa-reference 외에 eks-reference 쪽도 수정해야 하지 않나요??

@intelliguy
Copy link
Contributor Author

@intelliguy 님~ eks-msa-reference 외에 eks-reference 쪽도 수정해야 하지 않나요??

맞습니다. 그런데 현재 그런 내역을 적용하지는 않아서 (msa쪽만 현행화 상태)
최종적으로 변경을 반영하는 작업을 수행예정입니다.

@robertchoi80
Copy link
Contributor

robertchoi80 commented May 15, 2023

@intelliguy 님~ eks-msa-reference 외에 eks-reference 쪽도 수정해야 하지 않나요??

맞습니다. 그런데 현재 그런 내역을 적용하지는 않아서 (msa쪽만 현행화 상태) 최종적으로 변경을 반영하는 작업을 수행예정입니다.

네. 일단 msa 쪽만 쭉 수정하고, 나중에 eks-ref 쪽에도 한꺼번에 다 반영하신다는 거죠?? 그럼 지금 테스트를 msa 템플릿으로 해야겠네요.

추가로, validation 등 테스트 2개 실패한 걸로 보이는데 확인 부탁드립니다.

@intelliguy intelliguy added this pull request to the merge queue May 15, 2023
Merged via the queue into main with commit 628d513 May 15, 2023
@intelliguy intelliguy deleted the remove-etcd-on-eks branch May 15, 2023 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants