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

danken at redhat.com danken at redhat.com
Fri Mar 20 12:59:59 UTC 2015


Dan Kenigsberg has posted comments on this change.

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


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/38884/2//COMMIT_MSG
Commit Message:

Line 6: 
Line 7: hostdev: add after_hostdev_list_by_caps hook
Line 8: 
Line 9: In order to allow oVirt deployment maintainers some customization over
Line 10: which host devices are hidden to oVirt or do modification/quirks for
> about the flow, which I;m ashamed I'm not familiar enough with.
crazy hook writers get what they deserve. Hooks are exactly for crazy use cases that we have not anticipated, with consequences that are equally difficult to anticipate.
Line 11: specific hardware, new hook, after_hostdev_list_by_caps is introduced.
Line 12: This hook is another _json_hook, meaning that it receives device json
Line 13: representation and is free to modify it.
Line 14: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4ab6b3fee49cea83266c77dbfcd8c08ed69c630e
Gerrit-PatchSet: 2
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
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list