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

bugfix: thanos-sidecars handle raw data within 3h #83

Merged
merged 1 commit into from
Jun 20, 2021

Conversation

intelliguy
Copy link
Contributor

  1. thanos 질의가 수행되지 못하게 되었던 버그 수정
  2. prometheus내부의 thanos-sidecar에 대한 중복생성 이슈해소

@intelliguy intelliguy requested a review from estherk0 June 15, 2021 02:11
@@ -197,13 +197,16 @@ charts:
clusterDomain: $(clusterName)
existingObjstoreSecret: $(thanosObjstoreSecret)
query.nodeSelector: $(nodeSelector)
query.dnsDiscovery.sidecarsService: lma-thanos-discovery
Copy link
Contributor

Choose a reason for hiding this comment

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

sidecar service name이 lma-thanos-discovery로 생성되나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 promehtues 챠트에서 생성합니다.

@estherk0 estherk0 requested a review from robertchoi80 June 16, 2021 05:04
@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 label Jun 19, 2021
@intelliguy intelliguy merged commit 04c9100 into main Jun 20, 2021
@intelliguy intelliguy deleted the bugfix_thanos_sidecar branch July 9, 2021 07:27
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