Bug 1700076 - [sig-storage] PVC tests should not be running on UPI metal installations
Summary: [sig-storage] PVC tests should not be running on UPI metal installations
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Storage
Version: 4.1.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 4.1.0
Assignee: Hemant Kumar
QA Contact: Liang Xia
URL:
Whiteboard:
: 1700098 1700120 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-04-15 18:30 UTC by Abhinav Dahiya
Modified: 2019-05-06 13:00 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-04-29 15:50:27 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Abhinav Dahiya 2019-04-15 18:30:18 UTC
PVC e2e tests are being run on e2e-metal CI clusters even though no k8s storage has been setup.

example:
https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_installer/1621/pull-ci-openshift-installer-master-e2e-metal/4/

```
[sig-storage] PVC Protection Verify "immediate" deletion of a PVC that is not in active use by a pod [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-storage] PVC Protection Verify that PVC in active use by a pod is not removed immediately [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-storage] PVC Protection Verify that scheduling of a pod that uses PVC that is being deleted fails and the pod becomes Unschedulable [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-storage] PersistentVolumes NFS with Single PV - PVC pairs create a PVC and non-pre-bound PV: test write access [Suite:openshift/conformance/parallel] [Suite:k8s]
```

Expected:

storage tests that require PV etc must be skipped when storage is not configured.

Comment 1 Hemant Kumar 2019-04-15 19:52:09 UTC
I have opened a PR to skip the tests if default SC can not be found - https://github.com/kubernetes/kubernetes/pull/76625

But NFS failure is unrelated to that test fwiw and I am investigating that separately.

Comment 2 Jan Safranek 2019-04-16 13:50:14 UTC
These probably depend on default storage class too:

[sig-apps] StatefulSet [k8s.io] Basic StatefulSet functionality [StatefulSetBasic] should adopt matching orphans and release non-matching pods [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-apps] StatefulSet [k8s.io] Basic StatefulSet functionality [StatefulSetBasic] should not deadlock when a pod's predecessor fails [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-apps] StatefulSet [k8s.io] Basic StatefulSet functionality [StatefulSetBasic] should perform rolling updates and roll backs of template modifications with PVCs [Suite:openshift/conformance/parallel] [Suite:k8s]
[sig-apps] StatefulSet [k8s.io] Basic StatefulSet functionality [StatefulSetBasic] should provide basic identity [Suite:openshift/conformance/parallel] [Suite:k8s]

Comment 3 Jan Safranek 2019-04-16 13:50:55 UTC
*** Bug 1700120 has been marked as a duplicate of this bug. ***

Comment 4 Jan Safranek 2019-04-16 15:22:39 UTC
*** Bug 1700098 has been marked as a duplicate of this bug. ***

Comment 5 Abhinav Dahiya 2019-04-16 17:46:41 UTC
This is blocking installer team's UPI metal and vsphere CI efforts. therefore I hope it is okay to bump to high for 4.1.0

Comment 6 Hemant Kumar 2019-04-17 16:09:44 UTC
Why will this block vsphere CI efforts? vsphere environment should have a default storageclass configured by default fwiw.

Comment 7 Vadim Rutkovsky 2019-04-17 16:18:28 UTC
(In reply to Hemant Kumar from comment #6)
> Why will this block vsphere CI efforts? vsphere environment should have a
> default storageclass configured by default fwiw.

This doesn't affect vSphere - the same tests are failing, but for a different reason - tracked in https://bugzilla.redhat.com/show_bug.cgi?id=1699820

Comment 8 Hemant Kumar 2019-04-22 18:19:30 UTC
Looks like https://github.com/kubernetes/kubernetes/pull/76625 was approved. I will open a backport request.

Comment 9 Hemant Kumar 2019-04-23 14:45:50 UTC
https://github.com/openshift/origin/pull/22626

Comment 12 Hemant Kumar 2019-04-29 15:50:27 UTC
I don't think this bug fix needs QE verification. I am going to close this as fixed.


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