Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1605#discussion_r69677623
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -2015,19 +2016,20 @@ protected StartAnswer execute(StartCommand cmd) {
}
/**
- * Sets video card memory to the one provided in detail svga.vramSize
(if provided).
+ * Sets video card memory to the one provided in detail svga.vramSize
(if provided) on {@code vmConfigSpec}.
* 64MB was always set before.
* Size must be in KB.
* @param vmMo virtual machine mo
* @param vmSpec virtual machine specs
+ * @param vmConfigSpec virtual machine config spec
*/
- protected void postVideoCardMemoryConfigBeforeStart(VirtualMachineMO
vmMo, VirtualMachineTO vmSpec) {
+ protected void videoCardMemoryConfig(VirtualMachineMO vmMo,
VirtualMachineTO vmSpec, VirtualMachineConfigSpec vmConfigSpec) {
--- End diff --
Although the method name is apt for current scenario, i.e. configuring
video RAM, this method could be used for different kinds of Video card
configurations. Probably it is better to rename this to videoCardConfig or
videoCardReconfig or so.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
