weizhouapache commented on PR #11819: URL: https://github.com/apache/cloudstack/pull/11819#issuecomment-3390439828
> @erikbocks , sensible change but I would leave out the “Vm” bit from the new name. “CreateConditionForAutoScalingCmd” seems more appropriate as the VM is not scaled itself but the service is scaled by adding more (or less) VMs.What do you think? cc @weizhouapache @shwstppr . maybe use `CreateAutoScaleCondition` which consistent with other files: createAutoScalePolicy createAutoScaleVmProfile createAutoScaleVmGroup -- 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]
