mlsorensen opened a new pull request, #7878:
URL: https://github.com/apache/cloudstack/pull/7878

   ### Description
   
   This PR publishes an action event for VM.STOP when the power state processor 
detects a VM is gone from hypervisor. Currently only a power state event is 
published on the message bus. This allows events to be seen and processed when 
VM is detected to be stopped out of band.
   
   Additionally, it was discovered that the existing missing VM code is 
triggered when a VM is taking awhile to start. For example if we are waiting on 
the router VM to come up, the report can possibly see no VM when one is 
expected. The VM is assigned to the host, but doesn't exist yet, and triggers 
the missing VM code.  A check was added to ignore the VM if it is still in 
"Starting" state.
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [x] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [x] Minor
   
   #### Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [ ] Minor
   - [x] Trivial
   
   
   ### Screenshots (if appropriate):
   <img width="1051" alt="Screenshot 2023-08-17 at 10 37 38 AM" 
src="https://github.com/apache/cloudstack/assets/1047709/fe0dc636-d2cd-4a09-a797-1e81dfabbd47";>
   
   ### How Has This Been Tested?
   
   Tested out of band stop by shutting down guest within the VM, confirmed new 
event triggered.
   
   Tested startup of VM where power state is processed while waiting on router 
to come up, confirmed events no longer triggered detecting a "missing VM" when 
VM is in starting state.
   
   Tested live migration, confirmed we processed power state reports for both 
source and destination hypervisor hosts and did not issue the new VM.STOP event.
   <!-- Please read the 
[CONTRIBUTING](https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md) 
document -->
   


-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to