Skip to content

CSI: FiX E2E failures for new ceph version 19.2.2 #5460

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
Merged
merged 9 commits into from
Aug 4, 2025
Merged

CSI: FiX E2E failures for new ceph version 19.2.2 #5460

merged 9 commits into from
Aug 4, 2025

Conversation

@Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Jul 29, 2025 •
edited
Loading

This PR contains multiple changes

  • nonempty fuse mount is not supported anymore for that reason removing it from the default. If this is still required, This can be set in the storageclass.
  • Improve the e2e tests
  • Fixing logging format for cephfs clone failure
  • Add debug logging before creating a clone

@mergify mergify bot added the component/cephfs Issues related to CephFS label Jul 29, 2025
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Jul 29, 2025

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

@Madhu-1 Madhu-1 added the DNM DO NOT MERGE label Jul 29, 2025
@Madhu-1
cephfs: remove nonempty fuse mount option
5803378 
nonempty fuse mount is not supported anymore
for that reason removing it from the default.
If this is still required, This can be set in
the storageclass.

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

Madhu-1 commented Jul 29, 2025

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

@iPraveenParihar
Copy link
Contributor

@Madhu-1, cephfs: remove nonempty fuse mount option are you opening up a PR for this?

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Jul 29, 2025

@iPraveenParihar yes if CI passes we can do it.

@Madhu-1 Madhu-1 removed the DNM DO NOT MERGE label Jul 29, 2025
@Madhu-1 Madhu-1 changed the title cephfs: testing CI cephfs: Remove unsupport nonempty flag from the mount option Jul 29, 2025
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Jul 29, 2025

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

@Madhu-1 Madhu-1 requested review from a team July 29, 2025 15:40
iPraveenParihar
iPraveenParihar previously approved these changes Jul 29, 2025
View reviewed changes
Copy link
Contributor

@iPraveenParihar iPraveenParihar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! @Madhu-1

Rakshith-R
Rakshith-R previously approved these changes Jul 29, 2025
View reviewed changes
@Madhu-1 Madhu-1 added the ok-to-test Label to trigger E2E tests label Jul 29, 2025
@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

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

@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-helm/k8s-1.33

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jul 29, 2025
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Jul 29, 2025

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Jul 29, 2025 •
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 about the failure.

iPraveenParihar
iPraveenParihar approved these changes Aug 4, 2025
View reviewed changes
Copy link
Contributor

@iPraveenParihar iPraveenParihar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

19.2.2 in the PR title.

@Madhu-1 Madhu-1 changed the title CSI: FiX E2E failures for new ceph version 19.2.3 CSI: FiX E2E failures for new ceph version 19.2.2 Aug 4, 2025
@Rakshith-R
Copy link
Contributor

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Aug 4, 2025 •
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 about the failure.

@Madhu-1 Madhu-1 added the ok-to-test Label to trigger E2E tests label Aug 4, 2025
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@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-helm/k8s-1.32

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Aug 4, 2025
@mergify
Copy link
Contributor

mergify bot commented Aug 4, 2025

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 4, 2025

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

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 4, 2025 

I0804 16:59:42.764663 85215 pod.go:314] stdErr occurred: 2025-08-04T16:59:42.757+0000 7fba777fe640 -1 monclient: get_auth_request but no auth handler is set up

  I0804 16:59:42.764680 85215 utils.go:2009] failed to validate rbd images in pool replicapool trash: failed to list images in trash 2025-08-04T16:59:42.757+0000 7fba777fe640 -1 monclient: get_auth_request but no auth handler is set up

The above is reason for failure. it a ceph cluster problem

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 4, 2025

@Mergifyio requeue

@mergify
Copy link
Contributor

mergify bot commented Aug 4, 2025

requeue

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

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 4, 2025

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Aug 4, 2025 •
edited
Loading

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 6357fcb

@mergify mergify bot merged commit 6357fcb into ceph:devel Aug 4, 2025
37 checks passed
@mergify mergify bot mentioned this pull request Aug 4, 2025
Closed
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 5, 2025

@Mergifyio backport release-v3.14

@mergify
Copy link
Contributor

mergify bot commented Aug 5, 2025

backport release-v3.14

✅ Backports have been created

@Madhu-1 Madhu-1 mentioned this pull request Aug 5, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iPraveenParihar iPraveenParihar iPraveenParihar approved these changes

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

@nixpanic nixpanic Awaiting requested review from nixpanic

Assignees

No one assigned

Labels

backport-to-release-v3.14 Label to backport from devel to release-v3.14 branch component/cephfs Issues related to CephFS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Madhu-1 @iPraveenParihar @ceph-csi-bot @nixpanic @Rakshith-R