Skip to content

Conversation

github-actions[bot]
Copy link

⚠️ Cherry-pick Conflicts - Manual Resolution Required

The automatic cherry-pick of PR #1820 from qwang1/openshift-virtualization-tests resulted in conflicts.

Original PR: [Chaos][OADP] Automate Polarion TC CNV-12020: Drain the node where the VM is located during OADP Backup
Author: @qwang1
Merge Commit: 1e18fc7

🔧 How to resolve:

  1. Check out this branch locally: git fetch origin && git checkout cherry-pick-pr-1820-17726271826-conflicts
  2. Resolve conflicts in the affected files
  3. Test your changes thoroughly
  4. Commit your resolution: git add . && git commit
  5. Push changes: git push origin cherry-pick-pr-1820-17726271826-conflicts
  6. Mark this PR as ready for review

📁 Files with conflicts:

tests/chaos/oadp/conftest.py tests/chaos/oadp/test_oadp.py 

Original PR Link: #1820
Original Commit: 1e18fc7

Original PR Description

Short description:

It needs #1678 as its dependence.

More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
jira-ticket:

Summary by CodeRabbit

  • Tests
    • Added a chaos test that drains a VM’s host node during an in-progress backup and verifies the backup completes within the expected timeout.
    • Added a supporting fixture to automate draining the host and verifying node schedulability during backup chaos scenarios.
    • Confirmed the existing reboot-during-backup chaos test behavior remains unchanged.

Copy link

coderabbitai bot commented Sep 15, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Choose a reason for hiding this comment

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

D/S test tox -e verify-bugs-are-open failed: cnv-tests-tox-executor/16702

Copy link

Choose a reason for hiding this comment

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

D/S test tox -e verify-bugs-are-open failed: cnv-tests-tox-executor/16704

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.

3 participants