Change in vdsm[ovirt-3.1]: BZ#838097 _addNetworkValidation: do not explode if STP/DNS1 ...

danken at redhat.com danken at redhat.com
Sat Jul 7 19:31:19 UTC 2012


Hello Igor Lvovsky,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/6017

to review the following change.

Change subject: BZ#838097 _addNetworkValidation: do not explode if STP/DNS1 option passed
......................................................................

BZ#838097 _addNetworkValidation: do not explode if STP/DNS1 option passed

http://gerrit.ovirt.org/5456 has started passing all optional parameters
to _addNetworkValidation, but no one is expecting them there.

Change-Id: I071e6a7a53279c24d45ebec3858d9b0c0d6294ae
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/5885
Reviewed-by: Igor Lvovsky <ilvovsky at redhat.com>
---
M vdsm/configNetwork.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/17/6017/1
--
To view, visit http://gerrit.ovirt.org/6017
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I071e6a7a53279c24d45ebec3858d9b0c0d6294ae
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.1
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list