Bug 1815957: fix(dns): fix slow dns resolution of localhost for grpc health probe

[ecordell]

Description of the change:
Adds an nsswitch.conf to prefer local files over dns for resolution.

Motivation for the change:
see: golang/go#35305

Reviewer Checklist

• Implementation matches the proposed design, or proposal is updated to match implementation
• Sufficient unit test coverage
• Sufficient end-to-end test coverage
• Docs updated or added to /docs
• Commit messages sensible and descriptive

[njhale]

Cool! Here's what I found:

1. I think we should add the file for all generated Dockerfiles, since we don't know if the given base image contains it.
2. Looks like image builds are failing on chmod to /dev. I don't think the failure is related to any changes here. Since the blame is months old, my primary suspect is that the base image was swapped out under us in CI again.

[ecordell]

I think we should add the file for all generated Dockerfiles, since we don't know if the given base image contains it.

This is tricky, since the source image can vary on user input and there's no guarantee we'll have echo

[njhale]

@ecordell, then do you think opm should just attempt to generate the file at runtime?

[ecordell]

@njhale Updated with your suggestion.

In the downstream image, nsswitch is already populated correctly. I tested with the upstream image as well:

/go # cat /etc/nsswitch.conf
cat: can't open '/etc/nsswitch.conf': No such file or directory
/go # /registry-server &
/go # cat /etc/nsswitch.conf
hosts: files dns/go

[ecordell]

/retest

[ecordell]

/retest

[exdx]

/lgtm

[dinhxuanvu]

/lgtm

[awgreene]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: awgreene, dinhxuanvu, ecordell, exdx

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 [dinhxuanvu,ecordell]

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

[openshift-ci-robot]

@ecordell: Some pull requests linked via external trackers have merged: . The following pull requests linked via external trackers have not merged:

• operator-framework/operator-lifecycle-manager#1407 is closed
  Bugzilla bug 1815957 has been moved to the MODIFIED state.

In response to this:

Bug 1815957: fix(dns): fix slow dns resolution of localhost for grpc health probe

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[ecordell]

/cherry-pick release-4.4

[openshift-cherrypick-robot]

@ecordell: new pull request created: #269

In response to this:

/cherry-pick release-4.4

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.