vdsm-patches March 2015

vdsm-patches@lists.fedorahosted.org

57 participants
742 discussions

Change in vdsm[master]: api: make _should_keep_bridge robust against missing running...
by automation＠ovirt.org 01 Mar '15

01 Mar '15

automation(a)ovirt.org has posted comments on this change. Change subject: api: make _should_keep_bridge robust against missing running/persistent cfg ...................................................................... Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3']) -- To view, visit https://gerrit.ovirt.org/38239 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I13a225f406e6140463e1ded2008de8444fc4e9e0 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <osvoboda(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Ido Barkan <ibarkan(a)redhat.com> Gerrit-Reviewer: Ondřej Svoboda <osvoboda(a)redhat.com> Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No

1 0

Change in vdsm[master]: api: make _should_keep_bridge robust against missing running...
by ibarkan＠redhat.com 01 Mar '15

01 Mar '15

Ido Barkan has posted comments on this change. Change subject: api: make _should_keep_bridge robust against missing running/persistent cfg ...................................................................... Patch Set 1: Code-Review-1 (2 comments) https://gerrit.ovirt.org/#/c/38239/1/vdsm/network/api.py File vdsm/network/api.py: Line 750: return dict( Line 751: (k, v) for k, v in conf.iteritems() Line 752: if k not in ('bonding', 'nic', 'mtu')) Line 753: Line 754: # isn't the parameter order wrong?! the order is indeed wrong Line 755: def _bridge_reconfigured(required_conf, current_conf): Line 756: # we know nothing about the running/persistent configuration because Line 757: # it has been removed e.g. by shell_helper's emergency_net_cleanup Line 758: if current_conf is None or required_conf is None: Line 755: def _bridge_reconfigured(required_conf, current_conf): Line 756: # we know nothing about the running/persistent configuration because Line 757: # it has been removed e.g. by shell_helper's emergency_net_cleanup Line 758: if current_conf is None or required_conf is None: Line 759: return False but let's return True an throw away the bridge just in case, because we don't care what was there before anyway. Line 760: Line 761: return (_bridge_only_config(required_conf) != Line 762: _bridge_only_config(current_conf)) Line 763: -- To view, visit https://gerrit.ovirt.org/38239 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I13a225f406e6140463e1ded2008de8444fc4e9e0 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <osvoboda(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Ido Barkan <ibarkan(a)redhat.com> Gerrit-Reviewer: Ondřej Svoboda <osvoboda(a)redhat.com> Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

vdsm-patches March 2015