Code review comment for lp:~jamesodhunt/upstart/bug-1357252

Revision history for this message
Stéphane Graber (stgraber) wrote :

I suspect there may be some unwanted complications when two jobs share the same cgroup depending on the order they each move between states, but I guess it's a tiny tiny corner case.

The current fix seems sane.

review: Approve

« Back to merge proposal