From Arik Hadas <ahadas(a)redhat.com>:
Arik Hadas has submitted this change and it was merged. (
https://gerrit.ovirt.org/79258
)
Change subject: engine: handle NPE for guest info data of a powered-off vm
......................................................................
engine: handle NPE for guest info data of a powered-off vm
In case of powering-off a running VM with guest info data (i.e.
guest agent installed) and guestOsType == OsType.Windows, then
displaying 'guest info' sub-tab raises a NPE.
This is becuase powering off a VM, sets vm.guestOs field to null in
ResourceManager.resetVmAttributes(), while other Os fields like
guestOsType are not set to null.
That causes VmGuestInfoModel.setGuestOsNamedVersion() to fail while
trying to access that field.
This patch fixes it by avoid setting vm.guestOs to null.
Bug-Url:
https://bugzilla.redhat.com/1448650
Change-Id: I4f84fbe6abbc74bae54de80d8f5348646b8df3b4
Signed-off-by: Sharon Gratch <sgratch(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuestInfoModel.java
2 files changed, 1 insertion(+), 2 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Sharon Gratch: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/79258
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4f84fbe6abbc74bae54de80d8f5348646b8df3b4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>