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.

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

ISA Plugin coverage for EC pool #4411

Merged
merged 1 commit into from
Feb 3, 2025
Merged

Conversation

Divya-78
Copy link
Contributor

@Divya-78 Divya-78 commented Jan 31, 2025

Add tests for ISA Erasure Code (EC) plugin for pool creation, modification, deletion, and overwrites

This commit adds two tests related to the ISA erasure code plugin for Ceph:

EC Pool with ISA Plugin:
Purpose: This test verifies the creation, modification, and deletion of an EC pool using the ISA plugin. It tests the process of setting up an EC pool with specified parameters (k=2, m=2), setting pool configurations, and deleting the pool after running the IO operations.
Test Details:
Creates an EC pool with the ISA plugin.
Modifies pool configurations including pg_num, pgp_num, and compression settings.
Deletes the pool after the operations are performed.
Polarion ID: CEPH-83606527.

ISA Plugin for EC Pool with Overwrites:
Purpose: This test focuses on verifying the behaviour of the ISA EC pool with overwrites enabled. It creates an EC pool with overwrite configurations, runs IO operations, and ensures that the RBD pool is created and behaves as expected.
Test Details:
Enables overwrites (erasure_code_use_overwrites: "true") for an EC pool using the ISA plugin.
Creates a corresponding RBD pool and performs IO operations.
Deletes both the EC pool and the associated metadata pool after the test.
Polarion ID: CEPH-83606782.

@Divya-78
Copy link
Contributor Author

@neha-gangadhar
Copy link
Contributor

@Divya-78 Plugin name should be "ISA-L"

@Divya-78 Divya-78 changed the title ISA plugin coverage for EC pool ISA-L Plugin coverage for EC pool Feb 3, 2025
@Divya-78 Divya-78 force-pushed the divya-isa branch 2 times, most recently from fd4a14b to 738526b Compare February 3, 2025 08:16
@Divya-78
Copy link
Contributor Author

Divya-78 commented Feb 3, 2025

@Divya-78 Plugin name should be "ISA-L"

The documentation refers to it as the ISA Erasure Code Plugin.
https://docs.ceph.com/en/reef/rados/operations/erasure-code-isa/

@Divya-78 Divya-78 changed the title ISA-L Plugin coverage for EC pool ISA Plugin coverage for EC pool Feb 3, 2025
@openshift-ci openshift-ci bot added the lgtm Add this label when the PR is good to be merged label Feb 3, 2025
Copy link
Contributor

openshift-ci bot commented Feb 3, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Divya-78, pdhiran

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mergify mergify bot merged commit 004d03a into red-hat-storage:main Feb 3, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Add this label when the PR is good to be merged RADOS Rados Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants