Skip to content

feat: add memberlist content size (bytes) to web page #889

Merged
pracucci merged 2 commits intomainfrom
add-memberlist-value-size-to-web-page
Jan 30, 2026
Merged

feat: add memberlist content size (bytes) to web page #889
pracucci merged 2 commits intomainfrom
add-memberlist-value-size-to-web-page

Conversation

@pracucci
Copy link
Copy Markdown
Contributor

@pracucci pracucci commented Jan 29, 2026

What this PR does:

Based on a feedback from @pstibrany, in this PR I'm adding the "Size" (in bytes) column to the memberlist web page. It can be useful to quickly have an idea of how big is an entry.

See grafana/mimir#14200 to see a preview in Mimir.

Which issue(s) this PR fixes:

N/A

Checklist

  • Tests updated

Copy link
Copy Markdown
Contributor

@pstibrany pstibrany left a comment

Choose a reason for hiding this comment

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

Thank you ❤️

@pracucci pracucci merged commit 3948c25 into main Jan 30, 2026
12 checks passed
@pracucci pracucci deleted the add-memberlist-value-size-to-web-page branch January 30, 2026 08:21
pracucci added a commit to grafana/mimir that referenced this pull request Jan 30, 2026
#### What this PR does

In this PR I'm updating dskit to get
grafana/dskit#889 and adding the "Size" (in
bytes) column to the memberlist web page. It can be useful to quickly
have an idea of how big is an entry.

Draft because it's based on grafana/dskit#889
that has not been merged yet.

**Preview:**
<img width="1370" height="680" alt="Screenshot 2026-01-29 at 15 38 42"
src="https://github.com/user-attachments/assets/182f79c6-39e8-43ad-a3ba-11aa26e473f1"
/>

#### Which issue(s) this PR fixes or relates to

N/A

#### Checklist

- [x] Tests updated.
- [ ] Documentation added.
- [ ] `CHANGELOG.md` updated - the order of entries should be
`[CHANGE]`, `[FEATURE]`, `[ENHANCEMENT]`, `[BUGFIX]`. If changelog entry
is not needed, please add the `changelog-not-needed` label to the PR.
- [ ]
[`about-versioning.md`](https://github.com/grafana/mimir/blob/main/docs/sources/mimir/configure/about-versioning.md)
updated with experimental features.

---------

Signed-off-by: Marco Pracucci <[email protected]>
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