[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 5/5] tools/hotplug: support XENSTORED_TRACE in systemd
On Fri, Dec 12, Olaf Hering wrote: > On Fri, Dec 12, Ian Campbell wrote: > > > Seems ok. I wonder if the wrapper ought to source > > @CONFIG_DIR@/@CONFIG_LEAF_DIR@/xencommons to obtain XENSTORED_* itself > > rather than relying on the initscript and unit file to do so. Especially > > in the initscript case it looks a bit ugly to have to manually propagate > > things. > > It seems all that wrapping is of no use because SELinux can not deal > with it. I will see if "ExecStart=/bin/ary --no-fork $ENVVAR" can be > used to pass additional arguments. If so, the current XENSTORED_TRACE > handling has to be removed in favour of XENSTORED_ARGS=. This works: ExecStart=@XENSTORED@ --no-fork $XENSTORED_ARGS This fails: ExecStart=$XENSTORED --no-fork $XENSTORED_ARGS Dez 12 13:06:21 optiplex systemd[1]: [/usr/lib/systemd/system/xenstored.service:16] Executable path is not absolute, ignoring: $XENSTORED --no-fork $XENSTORED_ARGS Looks like variables are not expanded for the executable itself. If that really has to be supported a wrapper is required. Maybe that new wrapper script just needs some special SELinux handling? No idea. Olaf _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |