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

receive: Store the most recent config hash and timestamp as metrics #1378

Merged
merged 1 commit into from
Aug 6, 2019

Conversation

kakkoyun
Copy link
Member

@kakkoyun kakkoyun commented Aug 6, 2019

This PR attempts to store the most recent config md5 checksum, and when it happens as metrics to catch any inconsistencies between current and desired state. The same strategy is used by prometheus/alertmanager#coodinator.

Changes

  • Exposes 3 new metrics: thanos_receive_config_hash, thanos_receive_config_last_reload_successful and thanos_receive_config_last_reload_success_timestamp_seconds.

Verification

make test, make lint, make build and manual verification of the metrics.

@kakkoyun
Copy link
Member Author

kakkoyun commented Aug 6, 2019

cc @brancz @squat @metalmatze

@brancz
Copy link
Member

brancz commented Aug 6, 2019

Nice!

@brancz brancz merged commit 477a720 into thanos-io:master Aug 6, 2019
paulfantom added a commit to paulfantom/thanos that referenced this pull request Aug 7, 2019
* master:
  iter.go: error message typo correction (thanos-io#1376)
  Fix usage of $GOPATH in Makefile (thanos-io#1379)
  Moved Prometheus 2.11.1 and TSDB to 0.9.1 (thanos-io#1380)
  Store latest config hash and timestamp as metrics (thanos-io#1378)
  pkg/receive/handler.go: log errors (thanos-io#1372)
  receive: Hash-ring metrics (thanos-io#1363)
  receiver: avoid race of hashring (thanos-io#1371)
  feat compact: added readiness Prober (thanos-io#1297)
  Add changelog entry for S3 option (thanos-io#1361)
  Multipart features (thanos-io#1358)
  Added katacoda.yaml (thanos-io#1359)
  Remove deprecated option from example (thanos-io#1351)
  Move suggestion about admin API to appropriate place (thanos-io#1355)
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.

None yet

2 participants