Add more custom script hooks about containerpilot HOT 6 CLOSED

From @justenwalker in #22 (comment):

In my mind, OnStop makes more sense to happen after the application exits rather than before - so that we know the service isn't interacting with anything before we send cleanup commands (like Finally block)

That raises questions about the inconsistency with of the language compared to onStart. onStart is actually executed before the application starts. The following is a crappy solution, in my mind, but it avoids changing any existing code:

• onStart before application start
• onStarted after application start
• onStop before application stop
• onStopped after application stop

The following is more clear, in my mind, but does require changing existing code:

• preStart before application start, the current behavior of onStart
• postStart after application start
• preStop before application stop
• postStop after application stop

...though I hasten to add that I don't really see a need for postStart, I'm just enumerating it for clarity.

from containerpilot.

@justenwalker the title of the ticket may be too specific given our similar, but slightly different interests. Would you be okay with retitling it in a way that fits the conversation about pre and post stopped execution?

from containerpilot.

onStarted after application start

This is going to be difficult to actually implement because we block execution flow on the application. Even if we could figure out a way around that, the application can "start" (as in fork) but not actually be "running" for any useful purpose. e.g. slow Java startup times.

onStop before application stop

Services deregister and then stop. Does this happen before or after the service is deregistered?

from containerpilot.

So do we need something like:

• onStart - Unchanged
• preStop - Runs before Containerbuddy terminates the application - blocks
• postStop - Runs after the application terminates

It's not symmetric - but I'm not convinced it needs to be. Does it even make sense to run an application after the App starts? We are already polling the application, which is semantically similar to postStart

from containerpilot.

It's not symmetric

That's probably fine.

but I'm not convinced it needs to be. Does it even make sense to run an application after the App starts? We are already polling the application, which is semantically similar to postStart

I'm in agreement there.

from containerpilot.

Released with https://github.com/joyent/containerbuddy/releases/tag/0.0.4

from containerpilot.