Bug 1779491 - dns-node-resolver sometimes exits with status 255 on upgrade
Summary: dns-node-resolver sometimes exits with status 255 on upgrade
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: DNS
Version: 4.3.0
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
: 4.3.0
Assignee: Dan Mace
QA Contact: Hongan Li
Depends On: 1779490
TreeView+ depends on / blocked
Reported: 2019-12-04 04:47 UTC by Clayton Coleman
Modified: 2020-01-23 11:18 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1779490
Last Closed: 2020-01-23 11:17:46 UTC
Target Upstream Version:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Github openshift cluster-dns-operator pull 149 0 'None' closed [release-4.3] Bug 1779491: Handle missing pids during sidecar teardown 2020-09-02 17:59:28 UTC
Red Hat Product Errata RHBA-2020:0062 0 None None None 2020-01-23 11:18:04 UTC

Description Clayton Coleman 2019-12-04 04:47:58 UTC
+++ This bug was initially created as a clone of Bug #1779490 +++

Dec 04 01:43:12.495 E ns/openshift-dns pod/dns-default-954mg node/ip-10-0-133-153.ec2.internal container=dns-node-resolver container exited with code 255 (Error): kill: usage: kill [-s sigspec | -n signum | -sigspec] pid | jobspec ... or kill -l [sigspec]\n

shows up fairly frequently in our upgrade logs.  This is caused when all the child pids exit before the trap command completes, causing kill to get no arguments.

The fix is fairly standard in similar scripts - use pipe to xargs -r kill to tolerate empty.

Comment 2 Hongan Li 2019-12-09 08:25:43 UTC
verified with 4.3.0-0.nightly-2019-12-08-215349 (fresh installed and upgraded) and issue has been fixed.

$ oc -n openshift-dns get pod dns-default-7s2nr -o yaml | grep trap
      trap 'jobs -p | xargs kill || true; wait; exit 0' TERM

Comment 4 errata-xmlrpc 2020-01-23 11:17:46 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.