Change in vdsm[master]: net: set physical functions up before using them.

danken at redhat.com danken at redhat.com
Mon Sep 21 17:02:41 UTC 2015


Dan Kenigsberg 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
---
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/45639
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib4a5ad2e51bb8d5d09647455e14905c07d8c89cb
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list