Skip to content

Conversation

pstibrany
Copy link
Contributor

@pstibrany pstibrany commented Dec 9, 2020

What this PR does: This PR adds memberlist status page to Cortex. Status page shows memberlist cluster members, and some details about KV store that members are gossiping about. There is also possibility to inspect values in JSON or Go struct format, or download for offline inspection.

Screenshot 2020-12-09 at 13 42 22

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@jtlisi jtlisi assigned jtlisi and unassigned jtlisi Dec 9, 2020
Copy link
Contributor

@jtlisi jtlisi left a comment

Choose a reason for hiding this comment

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

LGTM

CHANGELOG.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

nit(optional): It may be good to clarify the mechanism used to view/download.

Suggested change
* [ENHANCEMENT] Memberlist: add status page with available details about memberlist-based KV store and memberlist cluster. It's also possible to view KV values in Go struct or JSON format, or download for inspection. #3575
* [ENHANCEMENT] Memberlist: add status page with available details about memberlist-based KV store and memberlist cluster. Additional query parameters can be added to view KV values in either Go struct or JSON format, or to download KV values for inspection. #3575

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think people will mostly just click on the links in the page (see screenshot). I'm not sure this is worth having in changelog.

Copy link
Contributor

@ranton256 ranton256 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for doing this.

Copy link
Contributor

@pracucci pracucci left a comment

Choose a reason for hiding this comment

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

Amazing, fantastic job! 👏

Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
@pstibrany pstibrany merged commit 61122a4 into cortexproject:master Dec 10, 2020
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.

4 participants