Change in vdsm[master]: vdsm: allow hooks to pass down dictionaries in json format
danken at redhat.com
danken at redhat.com
Fri Jan 17 16:09:36 UTC 2014
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm: allow hooks to pass down dictionaries in json format
......................................................................
Patch Set 9:
(2 comments)
http://gerrit.ovirt.org/#/c/20330/9/vdsm/configNetwork.py
File vdsm/configNetwork.py:
Line 525:
Line 526:
Line 527: def _buildSetupHookDict(req_networks, req_bondings, req_options):
Line 528:
Line 529: hook_dict = {'request': {'networks': dict(req_networks),
ok, ok, let this extra level of dictionary be my concession to the idea of passing something *but* 'request'.
Line 530: 'bondings': dict(req_bondings),
Line 531: 'options': dict(req_options)}}
Line 532:
Line 533: return hook_dict
http://gerrit.ovirt.org/#/c/20330/9/vdsm/vdsmd.8.in
File vdsm/vdsmd.8.in:
Line 82: which holds a pointer to a file with the network parameters that vdsm is
Line 83: setting up (
Line 84: .B request
Line 85: )
Line 86: , the request may modified by the before_network_setup hook as thus affect
may --> may be
Line 87: the operation ultimately taken place by Vdsm..
Line 88:
Line 89: The JSON format of this file has one section: request, this section
Line 90: contains networks, bondings and options, those parameters are specified
--
To view, visit http://gerrit.ovirt.org/20330
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie07c511e9740fd19a1c27baf87e91f9a427d0dcd
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Miguel Angel Ajo Pelayo <majopela 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: Miguel Angel Ajo Pelayo <majopela at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list