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

[RBD-Mirror] rbd mirror common test suites #4531

Merged
merged 1 commit into from
Mar 12, 2025
Merged

[RBD-Mirror] rbd mirror common test suites #4531

merged 1 commit into from
Mar 12, 2025
+307 −6

Conversation

sunilangadi2
Copy link
Contributor

@sunilangadi2 sunilangadi2 commented Mar 10, 2025 •
edited
Loading

Description

Added common sanity and regression tests for rbd mirror.

RBD Baremetal runs getting failed with package issue ongpgcheck failure,
hence added --nogpgcheck on package installaion.

Please include Automation development guidelines. Source of Test case - New Feature/Regression Test/Close loop of customer BZs

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

@sunilangadi2 sunilangadi2 requested review from Manohar-Murthy and a team as code owners March 10, 2025 14:27
@sunilangadi2 sunilangadi2 added the RBD Rados Bock Device label Mar 10, 2025
rbd mirror common suites
9a3b408 
Signed-off-by: Sunil Angadi <[email protected]>
@chaitanyadommeti
Copy link

LGTM

tintumathew10
tintumathew10 reviewed Mar 12, 2025
View reviewed changes
Copy link
Contributor

@tintumathew10 tintumathew10 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Team review is done. Approved.

@mergify mergify bot merged commit 1fc0f7f into red-hat-storage:main Mar 12, 2025
5 checks passed
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

@tintumathew10 tintumathew10 tintumathew10 approved these changes

@vamahaja vamahaja Awaiting requested review from vamahaja

@chaitanyadommeti chaitanyadommeti Awaiting requested review from chaitanyadommeti

@aarsharm aarsharm Awaiting requested review from aarsharm

@manasagowri manasagowri Awaiting requested review from manasagowri

@sunilkumarn417 sunilkumarn417 Awaiting requested review from sunilkumarn417

Assignees
No one assigned
Labels
RBD Rados Bock Device
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sunilangadi2 @chaitanyadommeti @Manohar-Murthy @tintumathew10