Dan Kenigsberg has submitted this change and it was merged. Change subject: iproute2, pyroute2: models.Bond checks for .unifiedPersistence, so add it ......................................................................
iproute2, pyroute2: models.Bond checks for .unifiedPersistence, so add it Commit 30a3fc25 fixed a condition in Bond.configure for ifcfg configurator that has a .runningConfig member only when using unified persistence, but broke the other two configurators that use unified persistence implicitly. The simplest fix (and even self-documentation) is to add .unifiedPersistence to them. Change-Id: I5dccdd8ea585f4fcbf327f233e68e8fcc5f5133b Signed-off-by: Ondřej Svoboda <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45212 Reviewed-by: Petr Horáček <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Continuous-Integration: Jenkins CI --- M vdsm/network/configurators/iproute2.py M vdsm/network/configurators/pyroute_two.py 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Ondřej Svoboda: Verified Jenkins CI: Passed CI tests Petr Horáček: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45212 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5dccdd8ea585f4fcbf327f233e68e8fcc5f5133b Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
