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.

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

RWOP - adding support for cephFS for all the test cases #9962

Merged
merged 2 commits into from
Jun 19, 2024
Merged

RWOP - adding support for cephFS for all the test cases #9962

merged 2 commits into from
Jun 19, 2024

Conversation

ypersky1980
Copy link
Contributor

This PR adds support ( and implementation) for CephFS for all the existing test cases of RWOP feature.

@ypersky1980
adding support for cephfs
0e554c5 
Signed-off-by: Yulia Persky <[email protected]>
@ypersky1980
renaming the test
0017d7e 
Signed-off-by: Yulia Persky <[email protected]>
@ypersky1980 ypersky1980 requested a review from a team as a code owner June 18, 2024 23:07
@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines label Jun 18, 2024
ocs-ci
ocs-ci reviewed Jun 19, 2024
View reviewed changes
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/functional/pv/pv_services/test_rwop_pvc.py
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job PASSED.

@ypersky1980 ypersky1980 added the Verified Mark when PR was verified and log provided label Jun 19, 2024
@openshift-ci openshift-ci bot added the lgtm label Jun 19, 2024
pintojoy
pintojoy approved these changes Jun 19, 2024
View reviewed changes
Copy link
Contributor

@pintojoy pintojoy left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jun 19, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: keemano, pintojoy, ramkiperiy, ypersky1980

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

@ramkiperiy ramkiperiy merged commit 488ba60 into master Jun 19, 2024
6 of 7 checks passed
@ypersky1980 ypersky1980 deleted the ypersky_rwop_enhancement branch June 19, 2024 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ocs-ci ocs-ci ocs-ci left review comments

@keemano keemano keemano approved these changes

@pintojoy pintojoy pintojoy approved these changes

@ramkiperiy ramkiperiy ramkiperiy approved these changes

Assignees

@ramkiperiy ramkiperiy

@pintojoy pintojoy

@keemano keemano

Labels
lgtm size/S PR that changes 10-29 lines Verified Mark when PR was verified and log provided
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ypersky1980 @ramkiperiy @ocs-ci @pintojoy @keemano