Change in vdsm[master]: netconf: privatize inRollback

danken at redhat.com danken at redhat.com
Thu Dec 5 16:02:37 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: netconf: privatize inRollback
......................................................................


netconf: privatize inRollback

inRollback is not part of the external setupNetworks api. Thus, it
is wise to be coherent and prefix it with '_'.

Change-Id: I6bcf98cb587c99366173d0a2994444b6370c9778
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21868
Reviewed-by: Assaf Muller <amuller at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Tested-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/configNetwork.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Assaf Muller: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6bcf98cb587c99366173d0a2994444b6370c9778
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list