Change in ovirt-engine[master]: core: Fix add/remove host error in the Label class

Code Review gerrit at ovirt.org
Mon Feb 27 14:31:15 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged.

Change subject: core: Fix add/remove host error in the Label class
......................................................................


core: Fix add/remove host error in the Label class

This patch corrects an error in Label.java where the addHost and
removeHost methods were modifying the vms field instead of the
hosts field.

Change-Id: Ia6d4f14705f110a40733f30e4ee3f5165e31cfcb
Signed-off-by: Phillip Bailey <phbailey at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Label.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Denis Chaplygin: Looks good to me, but someone else must approve
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Phillip Bailey: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia6d4f14705f110a40733f30e4ee3f5165e31cfcb
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list