Apache PIDs should be monitored more precisely on restart

[Andrey Grebennikov]

When config-changed hooks are running, the way keystone-charm monitors that Apache has been restarted is to execute "pgrep apache2".

In case the bundle defines Keystone on the host and any other service running under Apache inside LXD container - the command will always return PIDs of all Apache processes running on the host, consequently juju agent will be waiting for all them to be restarted.
Instead of "pgrep apache2" it should be executing "pgrep -f keystone", which only returns Keystone-related PIDs.

[Ryan Beisner (1chb1n)]

We should look at all os-charms and evaluate for this impact.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/608346
Committed: https://git.openstack.org/cgit/openstack/charm-keystone/commit/?id=f0ee6692792a7a2ad7dafa7f9914251b9ba4f338
Submitter: Zuul
Branch: master

commit f0ee6692792a7a2ad7dafa7f9914251b9ba4f338
Author: Andrey Grebennikov <email address hidden>
Date: Fri Oct 5 14:51:59 2018 -0500

    Improve Apache PID check on service restart

    In case there are containers with Apache running on the same
    host, the charm will only watch PIDs within the same namespace.

    Change-Id: I9cdd7d5b1f53fce748e7fafab538676e4e55dd54
    Closes-Bug: #1795918

[Corey Bryant (corey.bryant)]

I've targeted some of the charms that I know are affected by this bug, but this bug needs a full audit of charms to determine which ones are definitely affected.

[Aurelien Lourot (aurelien-lourot)]

Maybe related to lp:1952414

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Billy Olsen <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/charm-ceilometer/+/639631
Reason: Change has not been updated to address feedback in > 180 days

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Billy Olsen <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/charm-cinder/+/641003
Reason: Change has not been updated to address feedback in > 180 days

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Alex Kavanagh <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/charm-keystone/+/641043
Reason: Although it seems a reasonable change, it appears this has been also fixed via https://review.opendev.org/c/openstack/charm-keystone/+/608346, but also this change has charm-helper's changes that aren't in the charm-helper's library (there may be a PR for them). If this is still current, then please re-open the change, add a link to the charm-helpers PR (and cross link that PR back to this gerrit review) and then we can see to getting it landed. Thanks.