Alona Kaplan has submitted this change and it was merged. Change subject: core: add code setting storage pool id in DetachNetworkFromVdsInterfaceCommand ...................................................................... core: add code setting storage pool id in DetachNetworkFromVdsInterfaceCommand DetachNetworkFromVdsInterfaceCommand does not initialize storagePoolId property defined in CommandBase. So when method: org.ovirt.engine.core.bll.CommandBase#getMacPool is called, it will find: org.ovirt.engine.core.dal.dbbroker.auditloghandling.AuditLogableBase#getStoragePoolId to be null. Which causes MacPool not be found for null-valued datacenter. Change-Id: I2bd3d7bcb8a6bbd667d9bbde6658a8746f195692 Bug-Url: https://bugzilla.redhat.com/1299630 Signed-off-by: Martin Mucha <mmucha@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/DetachNetworkFromVdsInterfaceCommand.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Marcin Mirecki: Looks good to me, but someone else must approve Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/53051 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bd3d7bcb8a6bbd667d9bbde6658a8746f195692 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki@redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>