Change in vdsm[master]: hooks: Relocate hooking (api) module under lib

edwardh at redhat.com edwardh at redhat.com
Sun Apr 3 13:20:41 UTC 2016


Edward Haas has posted comments on this change.

Change subject: hooks: Relocate hooking (api) module under lib
......................................................................


Patch Set 4:

(1 comment)

I tried to use softlink, unfortunately it does not work: Unable to import a softlink without having the target on the search path.

https://gerrit.ovirt.org/#/c/55544/4/vdsm.spec.in
File vdsm.spec.in:

Line 1097: %{python_sitelib}/%{vdsm_name}/dmidecodeUtil.py*
Line 1098: %{python_sitelib}/%{vdsm_name}/exception.py*
Line 1099: %{python_sitelib}/%{vdsm_name}/executor.py*
Line 1100: %{python_sitelib}/%{vdsm_name}/health.py*
Line 1101: %{python_sitelib}/%{vdsm_name}/hooking.py*
> please add it to debian/vdsm-python.install as well
Ok, I will
Line 1102: %{python_sitelib}/%{vdsm_name}/hooks.py*
Line 1103: %{python_sitelib}/%{vdsm_name}/host.py*
Line 1104: %{python_sitelib}/%{vdsm_name}/hostdev.py*
Line 1105: %{python_sitelib}/%{vdsm_name}/ipwrapper.py*


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5f2e0ed06d5f17885b5892bb8659c52f4542a9d9
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list