Change in vdsm[master]: BZ#845830 confNet: Remove most of_validateNetworkSetup

danken at redhat.com danken at redhat.com
Tue Aug 7 11:44:58 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#845830 confNet: Remove most of_validateNetworkSetup
......................................................................


BZ#845830 confNet: Remove most of_validateNetworkSetup

All the dropped validations already exist in _addNetworkValidation.
Having them written twice is begging for bugs.

Change-Id: Id74a6418a669d45f11a9e9bd816e9400054860a9
Signed-off-by: Igor Lvovsky <ilvovsky at redhat.com>
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/configNetwork.py
1 file changed, 0 insertions(+), 113 deletions(-)

Approvals:
  Dan Kenigsberg: Verified
  Igor Lvovsky: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id74a6418a669d45f11a9e9bd816e9400054860a9
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Meni Yakove <myakove at redhat.com>


More information about the vdsm-patches mailing list