-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[UI] Fix display of disk size and IOPS fields in the scale VM form #11553
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] Fix display of disk size and IOPS fields in the scale VM form #11553
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 4.19 #11553 +/- ##
=========================================
Coverage 15.18% 15.18%
- Complexity 11368 11370 +2
=========================================
Files 5415 5415
Lines 476073 476073
Branches 58125 58125
=========================================
+ Hits 72279 72301 +22
+ Misses 395706 395680 -26
- Partials 8088 8092 +4
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:
|
@bernardodemarco a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
UI build: ✔️ |
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.
clgtm and verified in qa
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.
coed lgtm
Merging based on approvals and manual verification by @DaanHoogland |
Description
When scaling VMs, it is possible to change the disk offering for the VMs' ROOT disk if the global setting
allow.diskoffering.change.during.scale.vm
is true. To change the disk offering, the new selected compute offering for the VM must have the disk offering attached to it.However, for custom disk offerings (custom size and/or IOPS), it is not possible to currently define the values of the size and IOPS in the UI form:
Thus, this PR proposes to fix the scale VM UI form to display the custom disk size and IOPS fields.
Types of changes
Feature/Enhancement Scale or Bug Severity
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?