Change in ovirt-engine[master]: core: Ignore gateway for non-management net (#842137)

masayag at redhat.com masayag at redhat.com
Wed Jul 25 08:54:39 UTC 2012


Moti Asayag has submitted this change and it was merged.

Change subject: core: Ignore gateway for non-management net (#842137)
......................................................................


core: Ignore gateway for non-management net (#842137)

https://bugzilla.redhat.com/842137

The backend will not save the gateway value if it was reported for
non-management network, therefore the clients will receive null value
for it.

In addition, the "0.0.0.0" value for gateway is no longer permitted for
non-management network.

Change-Id: Ica928d0c774220339004e93110ecda742d6967f1
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/NetworkInterfaceValidator.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
2 files changed, 18 insertions(+), 4 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/6585
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ica928d0c774220339004e93110ecda742d6967f1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>



More information about the Engine-commits mailing list