Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1663327

Summary: Openshift hostpath e2e tests are failing and blocking merge queue
Product: OpenShift Container Platform Reporter: Hemant Kumar <hekumar>
Component: StorageAssignee: Hemant Kumar <hekumar>
Status: CLOSED CURRENTRELEASE QA Contact: Liang Xia <lxia>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 4.1.0CC: aos-bugs, aos-storage-staff, jsafrane, wking
Target Milestone: ---   
Target Release: 4.1.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-01-04 15:25:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Hemant Kumar 2019-01-03 20:22:09 UTC
Our hostpath e2e tests appear to be failing and blocking merge queue:

https://testgrid.k8s.io/redhat-openshift-release-blocking#redhat-release-openshift-origin-installer-e2e-aws-4.0&exclude-non-failed-tests=

Comment 1 Hemant Kumar 2019-01-03 22:42:18 UTC
fwiw this is because worker nodes don't have external IP anymore. There are two ways of fixing this:

a. We can use node's internal IP if external IP isn't available.
b. In kubernetes master, these tests have been migrated to use stdout of container with a special binary so as there is no need to SSH anymore. This is obviously cleaner but also messy to backport because it includes bunch of unrelated refactor.

Comment 2 Liang Xia 2019-01-04 06:21:30 UTC
I think you mean 4.0.0, rather than 1.0.0

Comment 3 Jan Safranek 2019-01-04 09:27:45 UTC
> a. We can use node's internal IP if external IP isn't available.

I remember some PRs, e.g. https://github.com/kubernetes/kubernetes/pull/68915 (not tested!)

Comment 4 Hemant Kumar 2019-01-04 15:25:03 UTC
@jan yeah I ended up backporting that one and PR that introduced GetNodeInternalIP.