<div dir="ltr"><div><div>Hi zigo, I agreed with Gustavo to use the EnvironmentFile to deal with the logging facility.<br></div>But after some failed tries I found it seems impossible to use EnvironmentFile to handle this.<br>

</div><div>The syntax EnvironmentFile using[1] is incompatible with bash(especially when dealing with<br>quotes and spaces), and <span style="color:rgb(67,67,67);font-family:Tahoma,Arial;font-size:12px;font-style:normal;font-variant:normal;font-weight:bold;letter-spacing:normal;line-height:19.1875px;text-align:left;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(242,242,242);display:inline!important;float:none"></span>seems have no control statements like if or while. So the following<br>

</div><div>code couldn't be done with systemd service file AFAIK.<br></div><div><br>[ "x$USE_SYSLOG" = "xyes" ] && DAEMON_ARGS="$DAEMON_ARGS --use-syslog"<br>[ "x$USE_LOGFILE" != "xno" ] && DAEMON_ARGS="$DAEMON_ARGS --log-file<br>

/var/log/nova/$NAME.log"<br><br></div><div>IMHO a workaround is to always enable logfile and offer some options in /etc/default<br>to decide whether to enable syslog.<br></div><div><br></div>_[1]: <a href="http://www.freedesktop.org/software/systemd/man/systemd.exec.html">http://www.freedesktop.org/software/systemd/man/systemd.exec.html</a><br>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 14, 2014 at 6:42 PM, stanzgy <span dir="ltr"><<a href="mailto:stan.zgy@gmail.com" target="_blank">stan.zgy@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Mon, Jul 14, 2014 at 5:20 PM, Thomas Goirand <span dir="ltr"><<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On 07/14/2014 12:23 PM, stanzgy wrote:<br>
</div><div>> BTW, I'm not very experienced at debian packaging. Could you waste a few<br>
> minutes reviewing my commit that adding dh-systemd support for glance<br>
> debian/juno branch to check if I has done it correctly?<br>
> Thanks very much.<br>
<br>
</div>Oh, and by the way, we shouldn't need PIDFile=/var/run/glance-api.pid.<br>
systemd is smart enough to not need that (it uses cgroups...).<br></blockquote><div><br></div></div><div>Got it. <br></div><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
Before doing anything more in other packages, shall we try to agree to<br>
have everything correct in both Glance and Keystone?<br>
<div><div><br></div></div></blockquote></div><div>Agreed.<br><br></div><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>

Thomas<br>
<br>
<br>
_______________________________________________<br>
Openstack-devel mailing list<br>
<a href="mailto:Openstack-devel@lists.alioth.debian.org" target="_blank">Openstack-devel@lists.alioth.debian.org</a><br>
<a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/openstack-devel" target="_blank">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/openstack-devel</a><br>
</div></div></blockquote></div></div><br><br clear="all"><div class=""><br>-- <br>Best Regards,<br><br>Gengyuan Zhang<br>NetEase Inc.
</div></div></div>
</blockquote></div><br><br clear="all"><br>-- <br>Best Regards,<br><br>Gengyuan Zhang<br>NetEase Inc.
</div>