Dan Kenigsberg has submitted this change and it was merged. Change subject: net: consolidate _BRIDGING_OPT_PATH and BRIDGING_OPT ......................................................................
net: consolidate _BRIDGING_OPT_PATH and BRIDGING_OPT Those 2 constants were the same. only netinfo.bridges.BRIDGING_OPT was left. bridge_stp_state also used an almost identical value for the briddge stp path. Change-Id: I4f54fe8391c9368a658b21b2e40db7383998a246 Signed-off-by: Ido Barkan <ibar...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/49622 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Continuous-Integration: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/netinfo/bridges.py M lib/vdsm/network/configurators/iproute2.py 2 files changed, 3 insertions(+), 4 deletions(-) Approvals: Ido Barkan: Verified Dan Kenigsberg: Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/49622 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f54fe8391c9368a658b21b2e40db7383998a246 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ido Barkan <ibar...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches