Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update k8s-e2e-external-storage job to use Kubernetes 1.21 #2017

Closed
1 of 3 tasks
nixpanic opened this issue Apr 22, 2021 · 3 comments
Closed
1 of 3 tasks

Update k8s-e2e-external-storage job to use Kubernetes 1.21 #2017

nixpanic opened this issue Apr 22, 2021 · 3 comments
Labels
component/testing Additional test cases or CI work dependency/k8s depends on Kubernetes features keepalive This label can be used to disable stale bot activiity in the repo priority-3
Milestone
release-3.5.0

Comments

@nixpanic
Copy link
Member

nixpanic commented Apr 22, 2021 •
edited
Loading

Running the job with Kubernetes 1.21 fails (k8s-e2e-external-storage/11):

Summarizing 10 Failures:

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should support multiple inline ephemeral volumes 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (OnRootMismatch)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with same fsgroup skips ownership changes to the volume contents 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should support two pods which share the same volume 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (Always)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with different fsgroup applied to the volume contents 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (OnRootMismatch)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with different fsgroup applied to the volume contents 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (Always)[LinuxOnly], pod created with an initial fsgroup, new pod fsgroup applied to volume contents 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (Always)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with same fsgroup applied to the volume contents 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (OnRootMismatch)[LinuxOnly], pod created with an initial fsgroup, new pod fsgroup applied to volume contents 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should create read-only inline ephemeral volume 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should create read/write inline ephemeral volume 
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

Ran 36 of 5978 Specs in 6890.099 seconds
FAIL! -- 26 Passed | 10 Failed | 0 Pending | 5942 Skipped
--- FAIL: TestE2E (6890.14s)
FAIL

Ginkgo ran 1 suite in 1h54m53.465533491s
Test Suite Failed

Running on Kubernetes 1.20 works fine. We'll need to make sure that we are complaint with Kubernetes 1.21 too.

  • The CI job description k8s-e2e-external-storage.yaml needs to be updated so that it generates a 2nd job for Kubernetes 1.21. This would look similar to the project/template in mini-e2e.yaml
  • errors listed above should be addressed in PRs referencing Updates: #2017
  • additional errors are expected with the RBD testing, these have not been executed yet
@nixpanic nixpanic added dependency/k8s depends on Kubernetes features component/testing Additional test cases or CI work labels Apr 22, 2021
@nixpanic nixpanic mentioned this issue Apr 22, 2021
Open
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue Apr 29, 2021
@nixpanic
ci: provide k8s-e2e-external-storage jobs for different k8s versions
8a7a410 
The Kubernetes e2e external storage tests from v1.21 do not work yet
with Ceph-CSI. In order to address the issues, the job is now provided
and can be run with:

     /test ci/centos/k8s-e2e-external-storage/1.21

The job for v1.20 is enabled by default, and identified by the
ci/centos/k8s-e2e-external-storage/1.20 context in PRs.

Updates: ceph#2017
Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic mentioned this issue Apr 29, 2021
Merged
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue Apr 29, 2021
@nixpanic
ci: provide k8s-e2e-external-storage jobs for different k8s versions
7cc3e5f 
The Kubernetes e2e external storage tests from v1.21 do not work yet
with Ceph-CSI. In order to address the issues, the job is now provided
and can be run with:

     /test ci/centos/k8s-e2e-external-storage/1.21

The job for v1.20 is enabled by default, and identified by the
ci/centos/k8s-e2e-external-storage/1.20 context in PRs.

Updates: ceph#2017
Signed-off-by: Niels de Vos <[email protected]>
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue May 3, 2021
@nixpanic
ci: provide k8s-e2e-external-storage jobs for different k8s versions
d6d10f6 
The Kubernetes e2e external storage tests from v1.21 do not work yet
with Ceph-CSI. In order to address the issues, the job is now provided
and can be run with:

     /test ci/centos/k8s-e2e-external-storage/1.21

The job for v1.20 is enabled by default, and identified by the
ci/centos/k8s-e2e-external-storage/1.20 context in PRs.

Updates: ceph#2017
Signed-off-by: Niels de Vos <[email protected]>
mergify bot pushed a commit that referenced this issue May 3, 2021
@nixpanic @mergify
ci: provide k8s-e2e-external-storage jobs for different k8s versions
2fb24c3 
The Kubernetes e2e external storage tests from v1.21 do not work yet
with Ceph-CSI. In order to address the issues, the job is now provided
and can be run with:

     /test ci/centos/k8s-e2e-external-storage/1.21

The job for v1.20 is enabled by default, and identified by the
ci/centos/k8s-e2e-external-storage/1.20 context in PRs.

Updates: #2017
Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic mentioned this issue May 7, 2021
Merged
@Madhu-1 Madhu-1 added this to the release-3.4.0 milestone Jun 2, 2021
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue Jun 30, 2021
@nixpanic
ci: skip Ephemeral-volume kubernetes external-storage tests
dedc842 
Ceph-CSI does not suport (inline) Ephemeral-volumes. Testing this will
continue to fail. The driver configuration can not be used to disable
testing of this feature, so it is done by skipping the tests by pattern
matching.

Updates: ceph#2017
Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic mentioned this issue Jun 30, 2021
Merged
mergify bot pushed a commit that referenced this issue Jun 30, 2021
@nixpanic @mergify
ci: skip Ephemeral-volume kubernetes external-storage tests
432ff08 
Ceph-CSI does not suport (inline) Ephemeral-volumes. Testing this will
continue to fail. The driver configuration can not be used to disable
testing of this feature, so it is done by skipping the tests by pattern
matching.

Updates: #2017
Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic modified the milestones: release-3.4.0, release-3.5.0 Jul 14, 2021
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the wontfix This will not be worked on label Aug 26, 2021
@github-actions
Copy link

github-actions bot commented Sep 3, 2021

This issue has been automatically closed due to inactivity. Please re-open if this still requires investigation.

@github-actions github-actions bot closed this as completed Sep 3, 2021
@Rakshith-R Rakshith-R removed the wontfix This will not be worked on label Sep 6, 2021
@Rakshith-R Rakshith-R reopened this Sep 6, 2021
Madhu-1 added a commit to Madhu-1/ceph-csi that referenced this issue Sep 22, 2021
@Madhu-1
ci: disable fsGroupChangePolicy change test for cephfs
4b7be1c 
disabling fsGroupChangePolicy test suite for
cephfs.

updates: ceph#2017

Signed-off-by: Madhu Rajanna <[email protected]>
@Madhu-1 Madhu-1 added the keepalive This label can be used to disable stale bot activiity in the repo label Sep 22, 2021
@Madhu-1 Madhu-1 mentioned this issue Sep 22, 2021
Merged
Madhu-1 added a commit to Madhu-1/ceph-csi that referenced this issue Sep 22, 2021
@Madhu-1
ci: disable fsGroupChangePolicy change test for cephfs
6b2f6fe 
disabling fsGroupChangePolicy test suite for
cephfs.

updates: ceph#2017

Signed-off-by: Madhu Rajanna <[email protected]>
mergify bot pushed a commit that referenced this issue Sep 22, 2021
@Madhu-1 @mergify
ci: disable fsGroupChangePolicy change test for cephfs
2a8eb95 
disabling fsGroupChangePolicy test suite for
cephfs.

updates: #2017

Signed-off-by: Madhu Rajanna <[email protected]>
@nixpanic
Copy link
Member Author

#2532 makes the tests pass. #2015 can be used to enable fsGroupChangePolicy again (once fixed).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component/testing Additional test cases or CI work dependency/k8s depends on Kubernetes features keepalive This label can be used to disable stale bot activiity in the repo priority-3
Projects
None yet
Milestone
release-3.5.0
Development

No branches or pull requests
3 participants
@nixpanic @Madhu-1 @Rakshith-R