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

WIP: Create new test for create multiple device classes #11096

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yitzhak12
Copy link
Contributor

@yitzhak12 yitzhak12 requested review from a team as code owners January 2, 2025 17:12
@pull-request-size pull-request-size bot added the size/L PR that changes 100-499 lines label Jan 2, 2025
Copy link

openshift-ci bot commented Jan 2, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: yitzhak12

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

@yitzhak12 yitzhak12 force-pushed the test-create-multiple-device-classes branch from e150cdf to a42d51d Compare January 5, 2025 16:34
- Create new test for create multiple device classes
- Create new Yaml files for multiple device classes
- Create new helper file for multiple device classes
- Create new functions for creating the LVS, add new deviceset in the storagecluster, create CephBlockpool, create deviceclass storageclass for multiple device classes

Signed-off-by: Itzhak Kave <[email protected]>
@yitzhak12 yitzhak12 force-pushed the test-create-multiple-device-classes branch from a42d51d to 2726dc2 Compare January 6, 2025 15:45
@pull-request-size pull-request-size bot added size/XL and removed size/L PR that changes 100-499 lines labels Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants