Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: Mapping bonding option value names to numerics
......................................................................


net: Mapping bonding option value names to numerics

Several bonding options have values that may be inputted
either by name or by their numeric equivalent.
When read, for comparing the actual settings with the
requested configuration, only one format should be used.

This change creates a map, outputted to a JSON file.
[bonding-name2numeric.json]

Change-Id: I1f4c16b1523822a2d53d4525841ff8741af6296c
Signed-off-by: Edward Haas <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/49390
Continuous-Integration: Jenkins CI
Reviewed-by: Ido Barkan <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M debian/vdsm-python.install
M init/vdsmd_init_common.sh.in
M lib/vdsm/netinfo/bonding.py
M lib/vdsm/tool/Makefile.am
D lib/vdsm/tool/dump_bonding_defaults.py
A lib/vdsm/tool/dump_bonding_opts.py
M tests/Makefile.am
M tests/netinfoTests.py
A tests/toolBondingTests.py
M vdsm.spec.in
M vdsm/Makefile.am
A vdsm/bonding-name2numeric.json
12 files changed, 461 insertions(+), 83 deletions(-)

Approvals:
  Ido Barkan: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Edward Haas: Verified



-- 
To view, visit https://gerrit.ovirt.org/49390
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1f4c16b1523822a2d53d4525841ff8741af6296c
Gerrit-PatchSet: 27
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Edward Haas <[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: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to