-
Notifications
You must be signed in to change notification settings - Fork 1.2k
UI: Display system VM count in hosts listing #10791
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
UI: Display system VM count in hosts listing #10791
Conversation
|
@bernardodemarco a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 4.20 #10791 +/- ##
=========================================
Coverage 16.13% 16.13%
- Complexity 13215 13216 +1
=========================================
Files 5649 5649
Lines 496692 496692
Branches 60175 60175
=========================================
+ Hits 80135 80146 +11
+ Misses 407633 407622 -11
Partials 8924 8924
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
UI build: ✔️ |
|
great improvement @bernardodemarco , I have one question; by the look of the code this would apply to the normal list view and to the metrics list view, but in QA I only see the extra column in the metrics view: Is this
tnx |
|
@DaanHoogland, yes, it should be displayed by default both on the normal list view and on the metrics view. Maybe, could this be related to browser caching? I accessed the QA environment in a private tab, and this is the default host list view that appears for me: 2025-04-30.08-14-45.mp4The same behavior also occurs in my local environment. |
|
thanks @bernardodemarco , I am using duckduckgo browser and thought it would not cache anything. I was so wrong 🤦 . |
Pearl1594
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM


Description
Currently, the hosts listing displays the number of end-user VMs allocated on each host. This PR proposes to also display the number of system VMs allocated on the hosts.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Screenshots (if appropriate):
How Has This Been Tested?