Skip to content

S3CSI-7: Add Helm-based CSI driver E2E Test Workflow for SystemdMounter with Scality Storage Integration #11

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.

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

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

anurag4DSB
Copy link

@anurag4DSB anurag4DSB commented Apr 10, 2025

This PR introduces a comprehensive E2E testing setup for the Mountpoint S3 CSI driver with Scality Storage integration. This mode launches mountpoint-s3 processes managed by systemd on the host node

Changes

  • Expanded a common setup action for E2E tests with reusable components
  • Updayed a controller E2E test workflow that tests to run without CSI driver
  • Implemented a Helm-based E2E test workflow that verifies driver functionality with Scality Storage for systems mounter
  • Added docker compose for s3 deployment. We used docker compose as we might add more scality services later which is not needed right now.
  • Updated actual tests to take the Ak, SK, and Engpoint of cloudserver(s3)

We use docker compose here as later on we will add other scality
services as well like Vault. This is not needed right now.
This commit adds Helm setup for the CSI driver and integrates with
Scality Storage. It includes:
By dafult this is deployed as most e2e tests will need this.
It can be turned off using `deploy_csi_driver: "false"`
- Configurable CSI driver deployment in common E2E test setup
- Scality Storage deployment with Docker Compose
- S3 endpoint URL configuration for tests
- Improved workflow for both controller and Helm E2E tests
This commit extends the CI pipeline with dedicated E2E test workflows:
- Adds controller E2E tests with CSI driver disabled
- Implements Helm-based E2E tests with full Scality storage integration
- Configures test environments with proper S3 endpoint and credentials
@anurag4DSB anurag4DSB changed the title S3CSI-7: Enable E2E systemd mounter tests S3CSI-7: Add Helm-based CSI driver E2E Test Workflow for SystemdMounter with Scality Storage Integration Apr 10, 2025
Copy link

codecov bot commented Apr 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.36%. Comparing base (62f7749) to head (b57293a).

Additional details and impacted files

Impacted file tree graph

see 1 file with indirect coverage changes

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   68.41%   68.36%   -0.05%     
==========================================
  Files          38       38              
  Lines        2137     2137              
==========================================
- Hits         1462     1461       -1     
- Misses        588      589       +1     
  Partials       87       87              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

In `testdriver.go`, modify the `CreateVolume` method to always use
standard buckets
Add skip statements to S3 Express test suites in `testsuites/cache.go`
Add skip statements to individual tests with S3 Express in their name
Update to match existing S3 client configuration,
but without the retry mechanism as it's not needed for S3-compatible
storage
Add a helper function to ensure all S3 clients use path style and
custom endpoint
Update any code that creates S3 clients directly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant