Add basic E2E testing for the csi driver #257
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Madhu-1
force-pushed
the
add-e2e-tests
branch
20 times, most recently
from
a10481b to
fbe5307
Compare
Madhu-1
force-pushed
the
add-e2e-tests
branch
3 times, most recently
from
1bba10f to
9d148c5
Compare
leelavg
reviewed
Jun 24, 2025
Contributor
|
Better to add a concurrency control to the test action as it's running for 40 min and do we need any knob to specifically skip the e2e? I'll approve after that. |
Collaborator
Author
These are more of an enhancement which we can work on like skipping for some scripts and also some other changes like document, is this what you mean? We dont have automation to merge it PR, its still manual and maintainers can still do it. |
provide an option in the value.yaml to conditionally deploy the required drivers as per user requirement. Signed-off-by: Madhu Rajanna <[email protected]>
These are the required yamls for testing for cephcsi drivers using the kubernetes e2e tests. This help us to test the basic functionality for the csi driver. Signed-off-by: Madhu Rajanna <[email protected]>
This is a github action to test the basic cephfs/nfs/rbd functionility using the standard kubernetes e2e tests. The idea is borrowed from the ceph-csi repo https://github.com/ceph/ceph-csi/tree/devel/scripts/k8s-storage Signed-off-by: Madhu Rajanna <[email protected]>
we need to have blockpool, filesystem and the nfs to be created to test the pvc workflow. Signed-off-by: Madhu Rajanna <[email protected]>
updating template version from alpha to v1 as its the latest. Signed-off-by: Madhu Rajanna <[email protected]>
Currently we are building image with latest tag and kubernetes will try to pull the image from the quay as its a latest image, using different image tag for testing local built image. Signed-off-by: Madhu Rajanna <[email protected]>
add osd name to the function and also for logging. Signed-off-by: Madhu Rajanna <[email protected]>
its called as enableSeLinuxHostMount in the spec field not EnableSeLinuxHostMount. Signed-off-by: Madhu Rajanna <[email protected]>
we need to install snapshot controller and its CRD to test the snapshot functionality Signed-off-by: Madhu Rajanna <[email protected]>
install snapshot controller and its CRD in the action. Signed-off-by: Madhu Rajanna <[email protected]>
update the app label of the operator to fetch the right logs. Signed-off-by: Madhu Rajanna <[email protected]>
updating the ceph-csi image to v3.14.1 where we have fix for NFS issue. Signed-off-by: Madhu Rajanna <[email protected]>
leelavg
approved these changes
Jun 24, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains below changes to have basic e2e testing to test the csi driver, The idea is borrowed from cephcsi where we have similar tests