Skip to content

Fixed delayed power state update after vm shutdown #4284

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
yadvr merged 2 commits into from
Sep 1, 2020
Merged

Fixed delayed power state update after vm shutdown #4284

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a2d2fa0
Fixed delayed power state update after vm shutdown
Spaceman1984 Aug 25, 2020
6725b56
Moved db update to after guru finalize stop method call
Spaceman1984 Aug 26, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Moved db update to after guru finalize stop method call
  • Loading branch information
@Spaceman1984
Spaceman1984 committed Aug 26, 2020
commit 6725b5688b7df023ce12d868863b7fff9f56f022
3 changes: 1 addition & 2 deletions engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1657,11 +1657,10 @@ protected boolean sendStop(final VirtualMachineGuru guru, final VirtualMachinePr
return false;
}

guru.finalizeStop(profile, answer);
final UserVmVO userVm = _userVmDao.findById(vm.getId());
userVm.setPowerState(PowerState.PowerOff);
_userVmDao.update(userVm.getId(), userVm);

guru.finalizeStop(profile, answer);
} else {
s_logger.error("Invalid answer received in response to a StopCommand for " + vm.getInstanceName());
return false;
Expand Down