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

e2e: remove unsupported kube versions check and enablement of tests in RBD and CephFS #2617

Merged
merged 6 commits into from
Nov 18, 2021
Merged

e2e: remove unsupported kube versions check and enablement of tests in RBD and CephFS #2617

merged 6 commits into from
Nov 18, 2021

Conversation

humblec
Copy link
Collaborator

@humblec humblec commented Nov 3, 2021 •
edited
Loading

Considering we are far out of these release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal hchiramm@redhat.com

@humblec humblec changed the title e2e: Remove unsupport kube versions check and enablement of tests in RBD e2e: Remove unsupported kube versions check and enablement of tests in RBD Nov 3, 2021
@mergify mergify bot added the component/testing Additional test cases or CI work label Nov 3, 2021
@humblec humblec added component/rbd Issues related to RBD cleanup labels Nov 3, 2021
@humblec humblec force-pushed the e2e-oldv branch 2 times, most recently from 0106234 to 7385925 Compare November 3, 2021 10:21
@humblec humblec changed the title e2e: Remove unsupported kube versions check and enablement of tests in RBD e2e: remove unsupported kube versions check and enablement of tests in RBD Nov 3, 2021
@humblec humblec requested a review from a team November 3, 2021 12:00
@humblec humblec changed the title e2e: remove unsupported kube versions check and enablement of tests in RBD e2e: remove unsupported kube versions check and enablement of tests in RBD and CephFS Nov 3, 2021
@humblec humblec added the component/cephfs Issues related to CephFS label Nov 3, 2021
@humblec
Copy link
Collaborator Author

humblec commented Nov 3, 2021

/retest ci/centos/mini-e2e/k8s-1.20

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Nov 8, 2021

Considering we are far out of these release and only care about
kubernetes releases from v1.19, there is no need to have this
version check in place for the tests.

we no longer test kube 1.19 adjust description and commit messages?

@humblec
Copy link
Collaborator Author

humblec commented Nov 8, 2021

Considering we are far out of these release and only care about
kubernetes releases from v1.19, there is no need to have this
version check in place for the tests.

we no longer test kube 1.19 adjust description and commit messages?

@Madhu-1 Sure, Overlooked at the version. will correct. 👍

@humblec
Copy link
Collaborator Author

humblec commented Nov 8, 2021

Considering we are far out of these release and only care about
kubernetes releases from v1.19, there is no need to have this
version check in place for the tests.

we no longer test kube 1.19 adjust description and commit messages?

@Madhu-1 Sure, Overlooked at the version. will correct. +1

@Madhu-1 done.. ptal.. thanks

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest all

@humblec humblec requested a review from nixpanic November 10, 2021 07:47
@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

Errors are bit crazy on this ci runs, looks spurious, but cannot confirm too.

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/k8s-e2e-external-storage/1.20

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.19

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/mini-e2e/k8s-1.21

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/mini-e2e/k8s-1.22

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.19

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

ci/centos/mini-e2e-helm/k8s-1.22

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/mini-e2e/k8s-1.21

@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2021

/retest ci/centos/mini-e2e-helm/k8s-1.22

@humblec
Copy link
Collaborator Author

humblec commented Nov 11, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 11, 2021

rebase

✅ Branch has been successfully rebased

@humblec humblec requested a review from Rakshith-R November 16, 2021 12:56
@humblec
Copy link
Collaborator Author

humblec commented Nov 16, 2021

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

@Rakshith-R Rakshith-R added the ci/retry/e2e Label to retry e2e retesting on approved PR's label Nov 17, 2021
@humblec
Copy link
Collaborator Author

humblec commented Nov 17, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2021

rebase

✅ Branch has been successfully rebased

@humblec
Copy link
Collaborator Author

humblec commented Nov 17, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2021

rebase

✅ Branch has been successfully rebased

@humblec
Copy link
Collaborator Author

humblec commented Nov 17, 2021

@nixpanic the job says : https://jenkins-ceph-csi.apps.ocp.ci.centos.org/blue/organizations/jenkins/mini-e2e-helm_k8s-1.22/detail/mini-e2e-helm_k8s-1.22/1107/pipeline

It started 3 hours back... may be those are stuck ?

@humblec
Copy link
Collaborator Author

humblec commented Nov 17, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2021

rebase

✅ Branch has been successfully rebased

@humblec
e2e: remove 1.17 based test enablement in rbd
5d0884c 
considering we are far out of this release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec
e2e: remove 1.16 based test enablement in rbd
536727d 
considering we are far out of this release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec
e2e: remove 1.15 based test enablement in rbd
00b4a17 
considering we are far out of this release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec
e2e: remove 1.17 based test enablement in cephfs
69b91ab 
Considering we are far out of these release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec
e2e: remove 1.16 based test enablement in cephfs
052159f 
Considering we are far out of these release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec
e2e: remove 1.15 based test enablement in cephfs
cd80b4c 
Considering we are far out of these release and only care about
kubernetes releases from v1.20, there is no need to have this
version check in place for the tests.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec
Copy link
Collaborator Author

humblec commented Nov 18, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 18, 2021

rebase

☑️ Nothing to do

  • -closed [:pushpin: rebase requirement]
  • #commits-behind>0 [:pushpin: rebase requirement]

@mergify mergify bot merged commit aef3cc0 into ceph:devel Nov 18, 2021
@humblec humblec mentioned this pull request Nov 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Madhu-1 Madhu-1 Madhu-1 approved these changes

@Rakshith-R Rakshith-R Rakshith-R approved these changes

@nixpanic nixpanic Awaiting requested review from nixpanic

@Yuggupta27 Yuggupta27 Awaiting requested review from Yuggupta27

@yati1998 yati1998 Awaiting requested review from yati1998

Assignees
No one assigned
Labels
ci/retry/e2e Label to retry e2e retesting on approved PR's cleanup component/cephfs Issues related to CephFS component/rbd Issues related to RBD component/testing Additional test cases or CI work
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@humblec @Madhu-1 @Rakshith-R