Upstart Cookbook

Scott James Remnant scott at netsplit.com
Mon Mar 28 22:32:54 UTC 2011


More comments

On Thu, Mar 24, 2011 at 2:34 PM, James Hunt <james.hunt at canonical.com> wrote:

>  http://upstart.ubuntu.com/cookbook/
>
I've tried to set this up at:

  http://upstart.at/cookbook/

But I couldn't figure out how to make the bazaar keywords stuff work?


1.4 - the footnote here is [2], where is [1] ?

2.2 - the "indented block"s aren't indented

3.1.2 - should be "NIH Utility Library" (qv. http://launchpad.net/libnih)

4.1.1 & 4.1.2 - worth explaining the real difference between "task"
and "service" here, perhaps? or later?

4.2 - the footnote here is [7], where are [2] thru [6] ?! :p

    - you leap into "blocking" here, which is a very very core upstart
concept, without explaining it

4.2.1 - signal events are created with the --no-wait option to emit,
there is no --emit option

5.3 - this is basically all completely wrong ...

5.4 - I wouldn't use the word "started" here, because it's easily
confused with the event and the started event is emitted when a task
is running not when it's finished

    - and your example is wrong

5.6 - one of the things you can do in "pre-start" is cancel the start!
worth mentioning?

5.8 - and the big feature of "pre-stop" is you can cancel the stop!
worth mentioning?

5.9 seems to run-in with the next paragraph

8.2.3 - worth a | sort in there?

9.15 - "start on stopped mydb EXIT_SIGNAL=SEGV" ... ;-)

9.39 - really script/exec ?  I'd just use exec there

17 - that should read "Scott James Remnant (Google)" no?

Is there anything in /usr/share/doc/upstart/README.Debian worth adding to this?

Scott



More information about the ubuntu-devel mailing list