Hi Robert,<div><br></div><div>thanks for reporting this, it makes total sense. We will update the package for the next release.</div><div><br></div><div>thanks for the patch!</div><div><br></div><div>regards,</div><div>Jaime<br>

<br><div class="gmail_quote">On Thu, Jun 2, 2011 at 4:01 PM, Robert Schweikert <span dir="ltr"><<a href="mailto:rschweikert@novell.com">rschweikert@novell.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Hi,<br>
<br>
IMHO the "one" startup script should create the directory for the pid files if the directory is not present. Reason is that /var/run may be located on a tmpfs and thus it might go away with every boot. Expecting the sysadmin to create the directory every time or writing her/his own script appears to me not very user friendly.<br>


<br>
Here is the patch I am applying in OBS to build the SUSE package.<br>
<br>
--- share/scripts/one.orig<br>
+++ share/scripts/one<br>
@@ -17,6 +17,7 @@<br>
<br>
#--------------------------------------------------------------------------- #<br>
<br>
 if [ -z "$ONE_LOCATION" ]; then<br>
+    ONE_PIDDIR=/var/run/one<br>
     ONE_PID=/var/run/one/oned.pid<br>
     ONE_SCHEDPID=/var/run/one/sched.pid<br>
     ONE_CONF=/etc/one/oned.conf<br>
@@ -28,6 +29,7 @@ if [ -z "$ONE_LOCATION" ]; then<br>
<br>
     LOCK_FILE=/var/lock/one/one<br>
 else<br>
+    ONE_PIDDIR=$ONE_LOCATION/var<br>
     ONE_PID=$ONE_LOCATION/var/oned.pid<br>
     ONE_SCHEDPID=$ONE_LOCATION/var/sched.pid<br>
     ONE_CONF=$ONE_LOCATION/etc/oned.conf<br>
@@ -53,6 +55,10 @@ setup()<br>
         exit 1<br>
     fi<br>
<br>
+    if [ ! -d $ONE_PIDDIR ]; then<br>
+       mkdir $ONE_PIDDIR<br>
+    fi<br>
+<br>
     if [ -f $LOCK_FILE ]; then<br>
         if [ -f  $ONE_PID ]; then<br>
             ONEPID=`cat $ONE_PID`<br>
<br>
The directories required in /var/lock and /var/log are created by the package in %post. However, for general use of OpenNebula outside of a package install it might make sense to have the "one" startup script create these directories as well. Although, since neither /var/lock nor /var/log are expected on tmpfs, AFAIK, the creation is a one time setup task and thus may not be desired in the "one" startup script.<br>


<br>
<br>
Robert<br>
-- <br>
Robert Schweikert                           MAY THE SOURCE BE WITH YOU<br>
SUSE-IBM Software Integration Center                   LINUX<br>
Tech Lead<br>
<a href="mailto:rschweikert@novell.com" target="_blank">rschweikert@novell.com</a><br>
<a href="mailto:rschweikert@ca.ibm.com" target="_blank">rschweikert@ca.ibm.com</a><br>
<a href="tel:781-464-8147" value="+17814648147" target="_blank">781-464-8147</a><br>
_______________________________________________<br>
Users mailing list<br>
<a href="mailto:Users@lists.opennebula.org" target="_blank">Users@lists.opennebula.org</a><br>
<a href="http://lists.opennebula.org/listinfo.cgi/users-opennebula.org" target="_blank">http://lists.opennebula.org/listinfo.cgi/users-opennebula.org</a><br>
</blockquote></div><br></div><br clear="all"><br>-- <br>Jaime Melis, Cloud Technology Engineer/Researcher<br>Major Contributor<br>OpenNebula - The Open Source Toolkit for Cloud Computing<br><a href="http://www.OpenNebula.org" target="_blank">www.OpenNebula.org</a> | <a href="mailto:jmelis@opennebula.org" target="_blank">jmelis@opennebula.org</a><br>