Comment 7 for bug 1844724

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to metal (master)

Reviewed: https://review.opendev.org/687028
Committed: https://git.openstack.org/cgit/starlingx/metal/commit/?id=8b757de303489715a0c8c63478fb30d601f60c5f
Submitter: Zuul
Branch: master

commit 8b757de303489715a0c8c63478fb30d601f60c5f
Author: Eric MacDonald <email address hidden>
Date: Mon Oct 7 09:14:33 2019 -0400

    Make pmon-restart honour startuptime config option

    Each monitored process's config file contains a startuptime
    label that specifies how many seconds it takes for that newly
    started process to stabalize and produce its pidfile.

    The pmon-restart feature needs to delay monitoring
    newly restarted process for 'startuptime' seconds.

    Failing to do so can cause it to fail the restarted
    process to early if there is pidfile creation delay.

    Test Plan:

    PASS: Verify collectd pmon-restart function with soak ;
          > 5000+ collectd pmon-restarts.
    PASS: Verify pmond regression test suite (test-pmon-action.sh)
          > restart command ; graceful restart all monitored processes. (5 loops)
          > kill command ; kill and recover all monitored processes. (5 loops)

    Regression:

    PASS: Verify pmon-stop command/function
    PASS: Verify pmon-start command/function also honors the startuptime.
    PASS: Verify pmon-stop auto start after auto-start timeout
    PASS: Verify System Install
    PASS: Verify Patching (soak)

    Change-Id: I9fd7bba8e49fe4c28281539ab4930bdac370ef11
    Closes-Bug: #1844724
    Signed-off-by: Eric MacDonald <email address hidden>