Dan Kenigsberg has submitted this change and it was merged. Change subject: bonding defaults: for validation, we need to be aware of all options ......................................................................
bonding defaults: for validation, we need to be aware of all options In the commit b01a096a, we started to use our bonding defaults database to validate bonding options (by name). 'primary' was not recognized as a bonding option because its default value is empty. This patch adds the rest of valid, but initially empty, options, plus three new ones. ad_actor_sys_prio, ad_actor_system and ad_user_port_key have recently been added to the upstream kernel. primary and arp_ip_target are empty by default. Change-Id: Ie2bbd778bb151e4671cc5d3d70ce898e48125331 Bug-Url: https://bugzilla.redhat.com/1262496 Backport-To: 3.6 Signed-off-by: Ondřej Svoboda <osvob...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/48008 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/tool/dump_bonding_defaults.py M tests/functional/networkTests.py M vdsm/bonding-defaults.json 3 files changed, 46 insertions(+), 5 deletions(-) Approvals: Ondřej Svoboda: Verified Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/48008 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie2bbd778bb151e4671cc5d3d70ce898e48125331 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches