supervisord don't really monitor cobbler container

[Sergey Vasilenko]

supervisord don't really monitor cobbler container

Container may be dead, but supervisord don't know about it.

6.0 ISO #109, #110

screenshot:

https://www.dropbox.com/s/5sjc1fgvj8o83pt/Screenshot%202014-11-17%2012.18.59.png?dl=0

[Sergey Vasilenko (xenolog)]

Diagnostic snapshop can't be attached, because:
https://www.dropbox.com/s/yku7ohf3ovn9e15/Screenshot%202014-11-17%2012.29.02.png?dl=0

[Matthew Mosesohn (raytrac3r)]

It's just the workflow of dockerctl start --attach containername. It should only attach to process and skip all the check steps that halt it. The container does restart after all 4 checks pass. It's just that it takes 32 minutes for all checks to time out.
I'll tackle this on two fronts:
1 - combine checks down to 2
2 - skip automatic "check_ready" when we run with --attach option.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/134905

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/137005

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/134905
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=68ef000b8cab70f56944040afcf7d5d0e5f34bbf
Submitter: Jenkins
Branch: master

commit 68ef000b8cab70f56944040afcf7d5d0e5f34bbf
Author: Matthew Mosesohn <email address hidden>
Date: Mon Nov 17 14:15:28 2014 +0400

    Fix behavior when running dockerctl start --attach

    --attach option skips check_ready steps, which would only
    run after a container was killed. This led to unnecessary
    checks that were run when the container is stopped. Now the
    program exits quickly so supervisor can restart the container
    as intended.

    Additionally, consolidated cobbler checks so that it fails in
    16 minutes intead of 32.

    Change-Id: I4d3c6e20b8dec0d11af46080fe6941b6b34cae4a
    Closes-Bug: #1393342

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/137005
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=3c7855027192dbcfa330a664174830d2dae0953b
Submitter: Jenkins
Branch: stable/5.1

commit 3c7855027192dbcfa330a664174830d2dae0953b
Author: Matthew Mosesohn <email address hidden>
Date: Mon Nov 17 14:15:28 2014 +0400

    Fix behavior when running dockerctl start --attach

    --attach option skips check_ready steps, which would only
    run after a container was killed. This led to unnecessary
    checks that were run when the container is stopped. Now the
    program exits quickly so supervisor can restart the container
    as intended.

    Additionally, consolidated cobbler checks so that it fails in
    16 minutes intead of 32.

    Change-Id: I4d3c6e20b8dec0d11af46080fe6941b6b34cae4a
    Closes-Bug: #1393342
    (cherry picked from commit 68ef000b8cab70f56944040afcf7d5d0e5f34bbf)