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

lma: prometheus: fix kubeetcd endpoints #77

Merged
merged 1 commit into from
Mar 19, 2022
Merged

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Mar 18, 2022

kube-prometheus-stack 차트는 POD로 실행되는 etcd의 exporter 아이피를 서비스 셀렉터를 통해 자동 구성하는 방법을 지원합니다. 이를 위해 kubeEtcd.endpoints 리스트가 비어 있어야 하기 때문에 해당 내용으로 정리하였습니다.

@bluejayA
Copy link
Contributor

이거는 @intelliguy 님도 아셔야 하는 내용일까요?

@zugwan
Copy link
Contributor Author

zugwan commented Mar 18, 2022

이거는 @intelliguy 님도 아셔야 하는 내용일까요?

먼저 작업을 하셨던 분이니 확인을 해주시면 좋습니다.

@@ -24,8 +24,7 @@ charts:

- name: prometheus
override:
kubeEtcd.endpoints:
- 172.27.1.222
kubeEtcd.endpoints: [] # If your etcd is not deployed as a pod, specify IPs it can be found on
Copy link
Contributor

@intelliguy intelliguy Mar 19, 2022

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

반영하였습니다.

@zugwan zugwan force-pushed the fix_prometheus_kubeetcd branch from 9a22792 to 32b1cf0 Compare March 19, 2022 02:42
@zugwan zugwan requested a review from intelliguy March 19, 2022 02:43
Copy link
Contributor

@intelliguy intelliguy left a comment

Choose a reason for hiding this comment

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

서비스로 띄우던 것을 pod로 띄우면서 간편해졌네요.
사용자 클러스터 생성전 이부분 넣어주는 부분이 사라져서 좋네요.

@bluejayA bluejayA merged commit d8f8b7b into main Mar 19, 2022
@bluejayA bluejayA deleted the fix_prometheus_kubeetcd branch March 19, 2022 02:48
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.

3 participants