weizhouapache commented on PR #10602: URL: https://github.com/apache/cloudstack/pull/10602#issuecomment-2743244823
> @shwstppr, I think that we should also tackle this bug when starting VMs through group actions. > > I debugged the current behavior of the `AutogenView` component. On this block of code: > > https://github.com/apache/cloudstack/blob/6c40a7bebbc4c9d4dd9c96769834ad4b9c6cb6c7/ui/src/views/AutogenView.vue#L1443-L1447 > > The `paramsList` variable contains the `considerlasthost` parameter as `undefined` and, thus, it is not sent within the API request. To handle that, we could, in the following `groupMap` field: > > https://github.com/apache/cloudstack/blob/6c40a7bebbc4c9d4dd9c96769834ad4b9c6cb6c7/ui/src/config/section/compute.js#L116-L124 > > Check whether `values.considerlasthost` is `undefined`. If so, the `considerlasthost` field could be set as `false`. > > Edit: The bug when starting VMs through group actions occurs when the form is opened, and the `considerlasthost` field is not changed. If the field is set to `true` and switched back to `false`, then the UI sets the `considerlasthost` parameter correctly to `false`. good point. this is indeed a bug -- 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]
