Remove zombie process crontab

[yuzhiquan]

What type of PR is this?

/kind bug

What this PR does / why we need it:

Remove zombie process crontab, in some situation, this cron will kill dockershim and make pod recreated

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

Does this PR introduced a user-facing change?

NONE

Additional documentation, usage docs, etc.:

[pixiake]

Can you describe the situation in detail? Thanks
dockershim was killed by this cronjob, is it because the docker service is not normal?

[yuzhiquan]

Can you describe the situation in detail? Thanks dockershim was killed by this cronjob, is it because the docker service is not normal?

If some pod in cluster create zombie process,and doesnot handle it, then grep ppid will lead to the dockershim process, that's happening in our online cluster!

[pixiake]

Can we get around it by adding filter conditions to this command?
such as: ps -A -ostat,ppid,cmd | grep -v *shim* | grep -e '^[Zz]' | awk '{print $2}' | xargs kill -HUP > /dev/null 2>&1"

[yuzhiquan]

Can we get around it by adding filter conditions to this command? such as: ps -A -ostat,ppid,cmd | grep -v *shim* | grep -e '^[Zz]' | awk '{print $2}' | xargs kill -HUP > /dev/null 2>&1"

May be the fix can avoid this issue, but z process does not affect in many situation, so i'm not sure if flannel or dns pod make z process, for example, this cmd can handle these?

[pixiake]

Can we get around it by adding filter conditions to this command? such as: ps -A -ostat,ppid,cmd | grep -v *shim* | grep -e '^[Zz]' | awk '{print $2}' | xargs kill -HUP > /dev/null 2>&1"

May be the fix can avoid this issue, but z process does not affect in many situation, so i'm not sure if flannel or dns pod make z process, for example, this cmd can handle these?

Yes, some zombie processes may be killed by mistake.

[pixiake]

/lgtm
/approve

[ks-ci-bot]

LGTM label has been added.

Git tree hash: 3f041b2cecc2fbd64632f1c7172b35ff3c5b78b1

[pixiake]

/approve

[ks-ci-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pixiake, yuzhiquan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [pixiake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment