Dan Kenigsberg has submitted this change and it was merged. Change subject: hostdev: remove physfn attribute in physfn ......................................................................
hostdev: remove physfn attribute in physfn Previously, detecting physfn was done via finding capabilities inside capabilities element, where address element pointed to physfn. This caused capabilities element in physfn with type 'virt_functions' to wrongly reference physfn in physfn. This patch fixes the bug, adding check for type attribute inside capabilities element, therefore removing the wrongful referencing. Change-Id: I9d2cbdb1c53eda9929b35596b6192b2ae6b1b344 Signed-off-by: Martin Polednik <[email protected]> Reviewed-on: http://gerrit.ovirt.org/37395 Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M tests/hostdevTests.py M vdsm/hostdev.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve Martin Polednik: Verified -- To view, visit http://gerrit.ovirt.org/37395 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d2cbdb1c53eda9929b35596b6192b2ae6b1b344 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik <[email protected]> Gerrit-Reviewer: Alona Kaplan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Martin Betak <[email protected]> Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
