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

vmui: update information about tsdb usage in cluster version #5004

Merged
merged 7 commits into from Sep 29, 2023
Merged

Conversation

dmitryk-dk
Copy link
Contributor

@dmitryk-dk dmitryk-dk commented Sep 13, 2023

This improvement adds explanations and expectations to the number of label values on Cardinality Explorer. This explanation is only visible for cluster installation because only in the cluster version the same label could have different values on each node.
Information was added to the documentation module.

Screenshot 2023-09-19 at 12 18 02

Related issue: #3070

Signed-off-by: dmitryk-dk d.kozlov@victoriametrics.com

@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (cbcfbaf) 60.40% compared to head (f84038e) 60.34%.
Report is 52 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5004      +/-   ##
==========================================
- Coverage   60.40%   60.34%   -0.06%     
==========================================
  Files         388      388              
  Lines       72732    72821      +89     
==========================================
+ Hits        43932    43945      +13     
- Misses      26402    26474      +72     
- Partials     2398     2402       +4     

see 19 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dmitryk-dk dmitryk-dk marked this pull request as ready for review September 15, 2023 07:35
Copy link
Contributor

@Loori-R Loori-R left a comment

Choose a reason for hiding this comment

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

a few comments

Copy link
Contributor

@Loori-R Loori-R left a comment

Choose a reason for hiding this comment

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

LGTM!

@f41gh7 f41gh7 merged commit f0e3370 into master Sep 29, 2023
9 of 10 checks passed
@f41gh7 f41gh7 deleted the issue-3070 branch September 29, 2023 09:47
valyala pushed a commit that referenced this pull request Oct 1, 2023
* vmui: update information about tsdb usage in cluster version

* vmui: cleanup

* vmui: add CHANGELOG.md

* vmui: cleanup

* vmui: update logic, move information to the visible place

* app/vmui: remove values fetch, update documentation for cardinality explorer

* app/vmui: update CHANGELOG.md
AndrewChubatiuk pushed a commit to AndrewChubatiuk/VictoriaMetrics that referenced this pull request Oct 3, 2023
…aMetrics#5004)

* vmui: update information about tsdb usage in cluster version

* vmui: cleanup

* vmui: add CHANGELOG.md

* vmui: cleanup

* vmui: update logic, move information to the visible place

* app/vmui: remove values fetch, update documentation for cardinality explorer

* app/vmui: update CHANGELOG.md
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

3 participants