Change in ovirt-engine[master]: core: Plug status setting in vm device monitoring.

tnisan at redhat.com tnisan at redhat.com
Wed Aug 19 10:53:17 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: core: Plug status setting in vm device monitoring.
......................................................................


core: Plug status setting in vm device monitoring.

VM's devices monitoring did not set the isPlugged field of the device,
which cuased the device's plugged status to be unmatched with the actual
status recieved from vdsm.

This patch adds isPlugged settings when the device should be updated
to plugged or unplugged.

Change-Id: I04d927fe8b8d1557fb3640e052565fc143a5b989
Bug-Url: https://bugzilla.redhat.com/1206696
Bug-Url: https://bugzilla.redhat.com/1044466
Signed-off-by: Liron Aravot <laravot at redhat.com>
Signed-off-by: Amit Aviram <aaviram at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Amit Aviram: Verified
  Allon Mureinik: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/45066
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I04d927fe8b8d1557fb3640e052565fc143a5b989
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list