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

Added new GW CLIs for 8.1 #4534

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Added new GW CLIs for 8.1 #4534

wants to merge 2 commits into from
+80 −19

Conversation

HaruChebrolu
Copy link
Contributor

@HaruChebrolu HaruChebrolu commented Mar 11, 2025 •
edited
Loading

Added new GW CLIs for 8.1 as per
https://jsw.ibm.com/browse/ISCE-1423
logs: http://magna002.ceph.redhat.com/ceph-qe-logs/hchebrol/cephci-run-DGTI05/

Note: The success logs were retrieved when we tried to do operations prior to listener delete
Once ceph/ceph-nvmeof#1162 gets fixed, this suite can be run without any errors

Description

click to expand checklist
  • Create a test case in Polarion reviewed and approved.
  • Create a design/automation approach doc. Optional for tests with similar tests already automated.
  • Review the automation design
  • Implement the test script and perform test runs
  • Submit PR for code review and approve
  • Update Polarion Test with Automation script details and update automation fields
  • If automation is part of Close loop, update BZ flag qe-test_coverage “+” and link Polarion test

@HaruChebrolu HaruChebrolu requested review from Manohar-Murthy and a team as code owners March 11, 2025 04:26
@mergify Mergify
Copy link
Contributor

mergify bot commented Mar 11, 2025

"This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes?"

@mergify @HaruChebrolu
Added new GW CLIs for 8.1
1149232 
Create 8.1 gateway operations test suite
@HaruChebrolu HaruChebrolu force-pushed the newGW_CLI branch 3 times, most recently from b6d21a9 to 01d3192 Compare March 11, 2025 06:36
@rahullepakshi rahullepakshi self-requested a review March 11, 2025 14:57
Manohar-Murthy
Manohar-Murthy approved these changes Mar 12, 2025
View reviewed changes
Copy link
Contributor

@Manohar-Murthy Manohar-Murthy left a comment

Choose a reason for hiding this comment

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

lgtm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Manohar-Murthy Manohar-Murthy Manohar-Murthy approved these changes

@rahullepakshi rahullepakshi rahullepakshi approved these changes

@sunilkumarn417 sunilkumarn417 Awaiting requested review from sunilkumarn417

@manasagowri manasagowri Awaiting requested review from manasagowri

At least 1 approving review is required to merge this pull request.

Assignees

@HaruChebrolu HaruChebrolu

Labels
nvmeof
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@HaruChebrolu @Manohar-Murthy @rahullepakshi