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

disable dns discovery of thanos sidecar service #160

Merged
merged 1 commit into from
May 25, 2022
Merged

Conversation

robertchoi80
Copy link
Contributor

https://github.com/openinfradev/tks-issues/issues/221

thanos에서 타 클러스터의 sidecar LB endpoint를 직접 참조하므로, 사용하지 않는 dns discovery 기능을 disable합니다.

@robertchoi80 robertchoi80 requested a review from intelliguy May 17, 2022 13:49
@github-actions
Copy link

This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days.

@github-actions github-actions bot added the Stale There has been no activity on this label May 21, 2022
@Jaesang
Copy link
Contributor

Jaesang commented May 25, 2022

dns discovery 기능이 활성화되면 "sidecar LB endpoint를 직접 참조"를 할 수없어서 비활성화하는건가요?
단순히 사용을 안하기에 비활성화하는건가요?

@github-actions github-actions bot removed the Stale There has been no activity on this label May 25, 2022
@zugwan
Copy link
Contributor

zugwan commented May 25, 2022

dns discovery 기능이 활성화되면 "sidecar LB endpoint를 직접 참조"를 할 수없어서 비활성화하는건가요? 단순히 사용을 안하기에 비활성화하는건가요?

https://github.com/thanos-io/thanos/blob/main/docs/service-discovery.md/ 중에서 Static Flags 방식을 사용하게 되는 것인지 궁금합니다.

@robertchoi80
Copy link
Contributor Author

@zugwan @Jaesang 현재 thanos-query에서 사용하는 flag가 다음과 같은데, 이 중 세번째 라인이 동일 클러스터의 prometheus sidecar 주소, 그리고 네번째 라인이 타 클러스터의 sidecar 주소입니다. 말씀하신 대로 static flag 방식을 사용하는 것이 맞으며, helm chart에서 자동으로 flag를 만들어줍니다.

    - --store=dnssrv+_grpc._tcp.thanos-storegateway.lma.svc.cluster.local
    - --store=dnssrv+_grpc._tcp.thanos-ruler.lma.svc.cluster.local
    - --store=prometheus-operated.lma.svc.cluster.local:10901
    - --store=a94131f6fac324a67a782cfe0103a2e1-1083702434.ap-northeast-2.elb.amazonaws.com:10901

dns discovery 방식은 사용할 필요가 없어서 삭제하는 것이기도 하고, 중복을 피하기 위함이기도 합니다.

@zugwan zugwan merged commit 02ed8c0 into main May 25, 2022
@zugwan zugwan deleted the disable_thanos_dns branch May 25, 2022 05:18
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.

3 participants