Change in vdsm[master]: vdsm_reg: Make vdsm-reg persist the management bridge it cre...

asegurap at redhat.com asegurap at redhat.com
Fri Sep 26 19:09:17 UTC 2014


Antoni Segura Puimedon has posted comments on this change.

Change subject: vdsm_reg: Make vdsm-reg persist the management bridge it creates
......................................................................


Patch Set 1:

(3 comments)

@Douglas: I'll push a new version after Dan answers to the commit message question I made.

http://gerrit.ovirt.org/#/c/33411/1//COMMIT_MSG
Commit Message:

Line 21: 
Line 22: In order to solve this issue, the best course of action is to
Line 23: persist RunningConfig as PersistentConfig as soon as the management
Line 24: network has been created by vdsm-reg (and strictly before the
Line 25: registration request is sent to the engine).
> Could you explain why we'd better add a new vdsm-tool command instead of us
I didn't want vdsm-reg to depend on lib/vdsm/config.py for getting which persistence mode is set that's all. If you think that it is okay to import from vdsm in vdsm-reg, I'll simplify.
Line 26: 
Line 27: Change-Id: I9d35da853df485a9196ad0657f832b77ae65d24e
Line 28: Bug-Url: https://bugzilla.redhat.com/1144639


http://gerrit.ovirt.org/#/c/33411/1/vdsm_reg/deployUtil.py.in
File vdsm_reg/deployUtil.py.in:

Line 995: 
Line 996: 
Line 997: def setSafeVdsmNetworkConfig():
Line 998:     """consider current network configuration as safe and remove its backup"""
Line 999:     out, err, ret = _logExec([os.path.join(P_USR_BIN, 'vdsm-tool'),
> puritanically speaking, this module is supposed to be shipped to rhev-m-3.0
Okay.
Line 1000:                              'store-nets'])
Line 1001:     if ret:
Line 1002:         raise Exception('Failed to store network. Stderr: %s' % err)
Line 1003: 


Line 1106:             )
Line 1107: 
Line 1108:     # Save current config by removing the undo files:
Line 1109:     try:
Line 1110:         if fReturn:
> ... so maybe it's safer to limit the new logic to fIsOvirt==True
Yeah, I considered that as well, I'll do it.
Line 1111:             setSafeVdsmNetworkConfig()
Line 1112:     except:
Line 1113:         logging.debug(traceback.format_exc())
Line 1114: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9d35da853df485a9196ad0657f832b77ae65d24e
Gerrit-PatchSet: 1
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: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list