<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv=Content-Type content="text/html; charset=us-ascii"> <meta name=Generator content="Microsoft Word 11 (filtered medium)"> <style>  </style> </head> <body lang=EN-US link=blue vlink=purple> <div class=Section1> Hello all,<o:p></o:p> I was trying to fix a reboot problem in upstart and ran into an issue in upstart for which there is a bug open [Bug 301715].<o:p></o:p> The reason this does not work is as follows.<o:p></o:p> When we do a proweoff "shutdown -P now", in the code in shutdown.c, we set environment variable INIT_HALT="POWEROFF"<o:p></o:p> When rc0 is executed, the environment variables come in the order INIT_HALT=POWEROFF RUNLEVEL=0 PREVLEVEL=3<o:p></o:p> <o:p> </o:p> In the upstart job file for rc0, we expect RUNLEVEL to be the first environment variable. (start on runlevel 0). Since RUNLEVEL is not the first variable, your rc0 job file is not executed.<o:p></o:p> Simply modifying the job file rc0's start on condition to <o:p></o:p> start on runlevel RUNLEVEL=0<o:p></o:p> will make this work.<o:p></o:p> Alternatively, you can modify the shutdown.c file function shutdown_now()<o:p></o:p> to do the following<o:p></o:p> <o:p> </o:p>     NIH_MUST (e = nih_sprintf (NULL, "RUNLEVEL=%s", runlevel));<o:p></o:p>     NIH_MUST (nih_str_array_addp (&env, NULL, NULL, e));<o:p></o:p>     NIH_MUST (e = nih_sprintf (NULL, "PREVLEVEL=%s", prev_level()));<o:p></o:p>     NIH_MUST (nih_str_array_addp (&env, NULL, NULL, e));<o:p></o:p>    /* set this environment variable after RUNLEVEL and PREVLEVEL are set */<o:p></o:p>     if (init_halt) {<o:p></o:p>         NIH_MUST (e = nih_sprintf (NULL, "INIT_HALT=%s", init_halt));<o:p></o:p>         NIH_MUST (nih_str_array_addp (&env, NULL, NULL, e));<o:p></o:p>     }<o:p></o:p> instead of<o:p></o:p>     if (init_halt) {<o:p></o:p>         NIH_MUST (e = nih_sprintf (NULL, "INIT_HALT=%s", init_halt));<o:p></o:p>         NIH_MUST (nih_str_array_addp (&env, NULL, NULL, e));<o:p></o:p>     }<o:p></o:p> <o:p> </o:p>     NIH_MUST (e = nih_sprintf (NULL, "RUNLEVEL=%s", runlevel));<o:p></o:p>     NIH_MUST (nih_str_array_addp (&env, NULL, NULL, e));<o:p></o:p>     NIH_MUST (e = nih_sprintf (NULL, "PREVLEVEL=%s", prev_level()));<o:p></o:p>     NIH_MUST (nih_str_array_addp (&env, NULL, NULL, e));<o:p></o:p> <o:p> </o:p> This would pass the environment variable in the following order RUNLEVEL, PREVLEVEL, INIT_HALT<o:p></o:p> <o:p> </o:p> The same fix applies when you do "shutdown -H now"<o:p></o:p> <o:p> </o:p> I have updated the bug with the same info I have provided here.<o:p></o:p> <o:p> </o:p> Thanks,<o:p></o:p> Sandeep<o:p></o:p> </div> </body> </html>