From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: Centralize bond mode 'numerization'
......................................................................


net: Centralize bond mode 'numerization'

When handling the bond mode, at several location the numeric value is
required, therefore the conversion from a name value to a numeric value
is performed.

As this task is currently perform at several locations outside the bond
modules, this patch centralizes it under one of bond modules.

This change originated from the need to add another such numerization of
the bond mode value.

Change-Id: I3e40fe1b539a9844a3bb40266d99a87ff4d4f628
Signed-off-by: Edward Haas <edwa...@redhat.com>
---
M lib/vdsm/network/kernelconfig.py
M lib/vdsm/network/models.py
M lib/vdsm/network/netinfo/bonding.py
3 files changed, 7 insertions(+), 8 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Petr Horáček: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Edward Haas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3e40fe1b539a9844a3bb40266d99a87ff4d4f628
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgold...@redhat.com>
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to