Igor Lvovsky has uploaded a new change for review.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Optimize _getConfigValue to return multiple values
Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Signed-off-by: Igor Lvovsky ilvovsky@redhat.com --- M vdsm/configNetwork.py 1 file changed, 19 insertions(+), 16 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/06/7306/1
diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py index 7e3379e..fb283ff 100755 --- a/vdsm/configNetwork.py +++ b/vdsm/configNetwork.py @@ -373,9 +373,11 @@ cfg += conf # Check whether we need to preserve prev ip/mask/gateway if not ipaddr and os.path.exists(cf): - ipaddr = self._getConfigValue(cf, 'IPADDR') - netmask = self._getConfigValue(cf, 'NETMASK') - gateway = self._getConfigValue(cf, 'GATEWAY') + confParams = self._getConfigValue(cf, ['IPADDR', 'NETMASK', + 'GATEWAY']) + ipaddr = confParams['IPADDR'] + netmask = confParams['NETMASK'] + gateway = confParams['GATEWAY']
if ipaddr: cfg = cfg + 'IPADDR=%s\nNETMASK=%s\n' % (pipes.quote(ipaddr), @@ -485,12 +487,12 @@ self._backup(self.NET_CONF_PREF + bridge) self._removeFile(self.NET_CONF_PREF + bridge)
- def _getConfigValue(self, conffile, entry): + def _getConfigValue(self, conffile, entries): """ - Get value from network configuration file + Get values from network configuration file
- :param entry: entry to look for (entry=value) - :type entry: string + :param entry: entries to look for (entry=value) + :type entry: list
:returns: value for entry (or None) :rtype: string @@ -498,13 +500,14 @@ Search for entry in conffile and return its value or None if not found """ + confParams = dict(zip(entries, len(entries)*[None])) with open(conffile) as f: - entries = [ line for line in f.readlines() - if line.startswith(entry + '=') ] - if len(entries) != 0: - value = entries[0].split('=', 1)[1] - return value.strip() - return None + for line in f.readlines(): + key, val = line.split('=', 1) + if key.strip() in confParams: + confParams[key.strip()] = val.strip() + + return confParams
def _updateConfigValue(self, conffile, entry, value, delete=False): """ @@ -551,7 +554,7 @@ """ for nic in nics: cf = self.NET_CONF_PREF + nic - mtuval = self._getConfigValue(cf, 'MTU') + mtuval = self._getConfigValue(cf, ['MTU'])['MTU'] if not mtuval is None: if int(mtuval) > mtu: mtu = mtuval @@ -570,7 +573,7 @@ """ _netinfo = netinfo.NetInfo() cf = self.NET_CONF_PREF + bridge - currmtu = self._getConfigValue(cf, 'MTU') + currmtu = self._getConfigValue(cf, ['MTU'])['MTU'] if currmtu is None: return
@@ -591,7 +594,7 @@ if vlan == delvlan: continue cf = self.NET_CONF_PREF + vlan - mtu = self._getConfigValue(cf, 'MTU') + mtu = self._getConfigValue(cf, ['MTU'])['MTU'] newmtu = max(newmtu, mtu)
if newmtu != currmtu:
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 1:
Build Successful
http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/513/ : SUCCESS
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 2:
Build Successful
http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/518/ : SUCCESS
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 3:
Build Successful
http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/521/ : SUCCESS
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Dan Kenigsberg has posted comments on this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 3: I would prefer that you didn't submit this
(1 inline comment)
.................................................... File vdsm/configNetwork.py Line 486: subprocess.call([constants.EXT_BRCTL, 'delbr', bridge]) Line 487: self._backup(self.NET_CONF_PREF + bridge) Line 488: self._removeFile(self.NET_CONF_PREF + bridge) Line 489: Line 490: def _getConfigValue(self, conffile, entries): now this function has become way to similar to
netinfo.getIfaceCfg()
please consider using it all over the place. Line 491: """ Line 492: Get values from network configuration file Line 493: Line 494: :param entry: entries to look for (entry=value)
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 4:
Build Successful
http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/526/ : SUCCESS
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Dan Kenigsberg has posted comments on this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 4: I would prefer that you didn't submit this
reinstating my previous score
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Igor Lvovsky has abandoned this change.
Change subject: Optimize _getConfigValue to return multiple values ......................................................................
Patch Set 4: Abandoned
-- To view, visit http://gerrit.ovirt.org/7306 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon Gerrit-Change-Id: I15ff5542095d51f33f5f1970b81a4a0cbfb30632 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Livnat Peer lpeer@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
vdsm-patches@lists.fedorahosted.org