Skip to content

Conversation

github-actions[bot]
Copy link

⚠️ Cherry-pick Conflicts - Manual Resolution Required

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

Original PR: [VIRT] add descheduler test to validate psi values on nodes
Author: @dshchedr
Merge Commit: 4a53659

🔧 How to resolve:

  1. Check out this branch locally: git fetch origin && git checkout cherry-pick-pr-2086-17822124836-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-2086-17822124836-conflicts
  6. Mark this PR as ready for review

📁 Files with conflicts:

tests/virt/node/descheduler/test_descheduler_psi_metrics.py 

Original PR Link: #2086
Original Commit: 4a53659

Original PR Description

add descheduler test to validate psi values within the threshold on nodes

Short description:
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 automated checks to ensure node pressure/PSI metrics stay within acceptable thresholds during descheduler rebalancing.
    • Introduced a shared validation utility to verify PSI thresholds across instances, improving reliability and clarity of failures.
    • Enhances test coverage for load-aware behavior with descriptive assertions to speed up triage.
    • No changes to runtime behavior or user-facing functionality.

Copy link

coderabbitai bot commented Sep 18, 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/16880

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