Change in vdsm[master]: hostdev: add after_hostdev_list_by_caps hooking point

danken at redhat.com danken at redhat.com
Fri Mar 20 14:20:51 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hostdev: add after_hostdev_list_by_caps hooking point
......................................................................


hostdev: add after_hostdev_list_by_caps hooking point

In order to allow oVirt deployment maintainers some customization over
which host devices are hidden to oVirt or do modification/quirks for
specific hardware, new hooking point, after_hostdev_list_by_caps is
introduced. This hooking point is another _json_hook, meaning that
it receives device json representation and is free to modify it.

This hook needs to be used with caution: modifying the device forest
(multiple trees) may lead to engine's inability to parse it and use
it correctly. Safe modification are removing leaf nodes and whole
trees. Although engine may not be able to parse broken tree correctly,
it should not explode or cause inconsistent database state.

Change-Id: I4ab6b3fee49cea83266c77dbfcd8c08ed69c630e
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38884
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
---
M AUTHORS
M debian/vdsm.dirs
M tests/hostdevTests.py
M vdsm.spec.in
M vdsm/hooks.py
M vdsm/hostdev.py
M vdsm/vdsmd.8.in
M vdsm_hooks/Makefile.am
8 files changed, 15 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 https://gerrit.ovirt.org/38884
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4ab6b3fee49cea83266c77dbfcd8c08ed69c630e
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan 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: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list