Change in vdsm[ovirt-3.6]: net: set physical functions up before using them.

fromani at redhat.com fromani at redhat.com
Tue Oct 20 10:46:25 UTC 2015


Francesco Romani has submitted this change and it was merged.

Change subject: net: set physical functions up before using them.
......................................................................


net: set physical functions up before using them.

Before using virtual functions, it is not enough to create and
configure them. If the underlying physical function is not up, the
virtual functions are unusable. Furthermore, the engine needs the
physical function to be up in order to let the user configure it.

Change-Id: Ib4a5ad2e51bb8d5d09647455e14905c07d8c89cb
Bug-Url: https://bugzilla.redhat.com/1229691
Signed-off-by: Ido Barkan <ibarkan at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45639
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-on: https://gerrit.ovirt.org/47110
---
M vdsm/hostdev.py
M vdsm/network/api.py
M vdsm/supervdsmServer
M vdsm/vdsm-restore-net-config
4 files changed, 39 insertions(+), 12 deletions(-)

Approvals:
  Ido Barkan: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib4a5ad2e51bb8d5d09647455e14905c07d8c89cb
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list