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

Add e2e volume group snapshot #4830

Merged
merged 4 commits into from
Sep 12, 2024
Merged

Add e2e volume group snapshot #4830

merged 4 commits into from
Sep 12, 2024

Conversation

Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Sep 10, 2024 •
edited
Loading

Adding test for volumegroupsnapshot for cephfs

update #4435

@Madhu-1 Madhu-1 marked this pull request as draft September 10, 2024 11:49
@mergify mergify bot added the component/testing Additional test cases or CI work label Sep 10, 2024
@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from 2c0697a to 5ee65b8 Compare September 10, 2024 11:52
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 added the ci/skip/multi-arch-build skip building on multiple architectures label Sep 10, 2024
@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from 5ee65b8 to ac16a77 Compare September 10, 2024 11:57
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

Ci run is happening here https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/blue/organizations/jenkins/mini-e2e_k8s-1.30/detail/mini-e2e_k8s-1.30/208/pipeline/

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.29/test_type-cephfs

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.29/cephfs

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/retest ci/centos/mini-e2e/k8s-1.29/test_type-cephfs

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.29

@nixpanic
Copy link
Member

/test ci/centos/mini-e2e-helm/k8s-1.31/test_type-cephfs

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.30

1 similar comment
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from d553f33 to d83377d Compare September 10, 2024 15:50
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 10, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from d83377d to 24855da Compare September 11, 2024 07:08
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from 24855da to e93e1f5 Compare September 11, 2024 07:38
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from e93e1f5 to 3ad83ef Compare September 11, 2024 08:49
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

1 similar comment
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from 3ad83ef to c95dda2 Compare September 11, 2024 10:11
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from c95dda2 to 84697e6 Compare September 11, 2024 10:30
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from 84697e6 to e5911f6 Compare September 11, 2024 10:58
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 11, 2024

/test ci/centos/mini-e2e/k8s-1.30

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from e5911f6 to 8517985 Compare September 11, 2024 11:17
@nixpanic
Copy link
Member

@Mergifyio rebase

@mergify Mergify
Copy link
Contributor

mergify bot commented Sep 12, 2024

rebase

✅ Branch has been successfully rebased

@nixpanic
Copy link
Member

Can be rebased and merged once #4833 is done.

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 12, 2024

@Mergifyio rebase

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 12, 2024

@Mergifyio queue

@Madhu-1
rebase: update external-snapshotter to v8
cc35e91 
updating external-snapshotter to v8
release.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1
e2e: set log level to 5 in csi pods
0949fa2 
all the sidecar log level is not set
to 5 by default, without debug logs
we cannot understand what is wrong
in the tests, this commits sets the
log level to 5 for csi pods.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1
ci: log pvc events
5eeb614 
incase if the pvc is not bound
yet, log the pvc events to
check what is happening.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1
e2e: add framework for volumegroupsnapshot
50efd20 
adding a framework to test the volumegroupsnapshot
for both cephfs and rbd and added a test case
for cephfs.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@mergify Mergify
Copy link
Contributor

mergify bot commented Sep 12, 2024

rebase

✅ Branch has been successfully rebased

@Madhu-1 Madhu-1 force-pushed the add-e2e-volume-group-snapshot branch from 1d9a414 to 50efd20 Compare September 12, 2024 10:33
@mergify Mergify
Copy link
Contributor

mergify bot commented Sep 12, 2024 •
edited
Loading

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Sep 12, 2024
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.30

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Sep 12, 2024
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 12, 2024

/test ci/centos/mini-e2e/k8s-1.31

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Sep 12, 2024

@Mergifyio requeue

@mergify Mergify
Copy link
Contributor

mergify bot commented Sep 12, 2024

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

@mergify mergify bot merged commit d2ddd52 into ceph:devel Sep 12, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nixpanic nixpanic nixpanic approved these changes

@iPraveenParihar iPraveenParihar iPraveenParihar approved these changes

Assignees
No one assigned
Labels
ci/skip/multi-arch-build skip building on multiple architectures component/testing Additional test cases or CI work
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Madhu-1 @nixpanic @ceph-csi-bot @iPraveenParihar