rg9975 commented on code in PR #8889:
URL: https://github.com/apache/cloudstack/pull/8889#discussion_r1628709982
##########
server/src/main/java/com/cloud/vm/UserVmManagerImpl.java:
##########
@@ -4367,14 +4368,7 @@ public boolean
checkIfDynamicScalingCanBeEnabled(VirtualMachine vm, ServiceOffer
*/
protected long configureCustomRootDiskSize(Map<String, String>
customParameters, VMTemplateVO template, HypervisorType hypervisorType,
DiskOfferingVO rootDiskOffering) {
verifyIfHypervisorSupportsRootdiskSizeOverride(hypervisorType);
- Long rootDiskSizeCustomParam = null;
Review Comment:
Appears to have been a refactor for a flagged potential NPE, but reverting
to avoid any regression as its not needed for the functional part of this PR.
Reverted in commit 9f8daa2878.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
