Skip to content

bucket: Set state of status prober properly#2120

Merged
bwplotka merged 3 commits intothanos-io:masterfrom
kakkoyun:bucket_probe
Feb 11, 2020
Merged

bucket: Set state of status prober properly#2120
bwplotka merged 3 commits intothanos-io:masterfrom
kakkoyun:bucket_probe

Conversation

@kakkoyun
Copy link
Copy Markdown
Member

This PR adds missing instruction to set probe status for bucket web UI.

Fixes #2115

Signed-off-by: Kemal Akkoyun kakkoyun@gmail.com

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

  • Set state of status prober properly

Verification

  • make test-local
  • MINIO_ENABLED=1 ./scripts/quickstart.sh

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
@bwplotka
Copy link
Copy Markdown
Member

Thanks!

@bwplotka bwplotka merged commit 573eff4 into thanos-io:master Feb 11, 2020
@kakkoyun kakkoyun deleted the bucket_probe branch February 11, 2020 16:38
vankop pushed a commit to monitoring-tools/thanos that referenced this pull request Feb 28, 2020
* Set state of status prober proberly

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>

* Add bucket web to quickstart script

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>

* Add changelog

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
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.

readnessProbe checks failed of thanos bucket web

2 participants