WIP: net: Introduce dual-stream localnet tests by azhivovk · Pull Request #4784 · RedHatQE/openshift-virtualization-tests

azhivovk · 2026-05-08T09:05:45Z

Short description:

New dual-stream tests: localnet network scenarios

More details:

STD: #4569

What this PR does / why we need it:

Added automation for new dual-stream tests

Which issue(s) this PR fixes: -

Special notes for reviewer:

Follow-up implementation of new tests of localnet
Do not review the first infra commit - it'll be merged in first PR of this series #4772

jira-ticket: https://redhat.atlassian.net/browse/CNV-86194

coderabbitai · 2026-05-08T09:05:53Z

Important

Review skipped

Ignore keyword(s) in the title.

⛔ Ignored keywords (1)

WIP

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: b04edc45-8f90-4bf0-b913-87c471c68278

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

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

openshift-virtualization-qe-bot-3 · 2026-05-08T09:06:19Z

Report bugs in Issues

Welcome! 🎉

This pull request will be automatically processed with the following features:

🔄 Automatic Actions

Reviewer Assignment: Reviewers are automatically assigned based on the OWNERS file in the repository root
Size Labeling: PR size labels (XS, S, M, L, XL, XXL) are automatically applied based on changes
Issue Creation: A tracking issue is created for this PR and will be closed when the PR is merged or closed
Branch Labeling: Branch-specific labels are applied to track the target branch
Auto-verification: Auto-verified users have their PRs automatically marked as verified
Labels: Enabled categories: branch, can-be-merged, cherry-pick, has-conflicts, hold, needs-rebase, size, verified, wip

📋 Available Commands

PR Status Management

/wip - Mark PR as work in progress (adds WIP: prefix to title)
/wip cancel - Remove work in progress status
/hold - Block PR merging (approvers only)
/hold cancel - Unblock PR merging
/verified - Mark PR as verified
/verified cancel - Remove verification status
/reprocess - Trigger complete PR workflow reprocessing (useful if webhook failed or configuration changed)
/regenerate-welcome - Regenerate this welcome message

Review & Approval

/lgtm - Approve changes (looks good to me)
/approve - Approve PR (approvers only)
/assign-reviewers - Assign reviewers based on OWNERS file
/assign-reviewer @username - Assign specific reviewer
/check-can-merge - Check if PR meets merge requirements

Testing & Validation

/retest tox - Run Python test suite with tox
/retest build-container - Rebuild and test container image
/retest verify-bugs-are-open - verify-bugs-are-open
/retest all - Run all available tests

Container Operations

/build-and-push-container - Build and push container image (tagged with PR number)
- Supports additional build arguments: /build-and-push-container --build-arg KEY=value

Cherry-pick Operations

/cherry-pick <branch> - Schedule cherry-pick to target branch when PR is merged
- Multiple branches: /cherry-pick branch1 branch2 branch3

Label Management

/<label-name> - Add a label to the PR
/<label-name> cancel - Remove a label from the PR

✅ Merge Requirements

This PR will be automatically approved when the following conditions are met:

Approval: /approve from at least one approver
LGTM Count: Minimum 2 /lgtm from reviewers
Status Checks: All required status checks must pass
No Blockers: No wip, hold, has-conflicts labels and PR must be mergeable (no conflicts)
Verified: PR must be marked as verified

📊 Review Process

Approvers and Reviewers

Approvers:

EdDev
dshchedr
myakove
rnetser
vsibirsk

Reviewers:

Anatw
EdDev
RoniKishner
azhivovk
dshchedr
frenzyfriday
nirdothan
orelmisan
rnetser
servolkov
vsibirsk
yossisegev

Available Labels

hold
verified
wip
lgtm
approve

AI Features

Cherry-Pick Conflict Resolution: Enabled (claude/claude-opus-4-6[1m])

💡 Tips

WIP Status: Use /wip when your PR is not ready for review
Verification: The verified label is removed on new commits unless the push is detected as a clean rebase
Cherry-picking: Cherry-pick labels are processed when the PR is merged
Container Builds: Container images are automatically tagged with the PR number
Permission Levels: Some commands require approver permissions
Auto-verified Users: Certain users have automatic verification and merge privileges

For more information, please refer to the project documentation or contact the maintainers.

azhivovk · 2026-05-08T09:29:26Z

Change: Drop udn change

azhivovk · 2026-05-08T10:03:36Z

/wip

- Add mixed_os_nodes marker to pytest.ini to select tests only on compatible clusters - Add rhcos9_node/rhcos10_node fixtures to network conftest - Add update_vm_node_selector helper and HOSTNAME_LABEL to nodes lib Signed-off-by: Asia Khromov <azhivovk@redhat.com> Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

Signed-off-by: Asia Khromov <azhivovk@redhat.com> Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

OVN localnet bridges rely on kernel bridge and OVS components that differ between RHCOS versions. These tests verify that secondary localnet network connectivity is preserved when the server VM migrates between RHCOS 9 and RHCOS 10 nodes in both directions. Signed-off-by: Asia Khromov <azhivovk@redhat.com> Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

azhivovk · 2026-05-08T10:39:27Z

Change: edit commits

openshift-virtualization-qe-bot · 2026-05-08T10:39:34Z

Clean rebase detected — no code changes compared to previous head (357bc03).

github-actions Bot added the sig-network label May 8, 2026

openshift-virtualization-qe-bot-3 requested a review from RoniKishner May 8, 2026 09:06

openshift-virtualization-qe-bot-3 assigned azhivovk May 8, 2026

openshift-virtualization-qe-bot-3 requested a review from EdDev May 8, 2026 09:06

openshift-virtualization-qe-bot-3 mentioned this pull request May 8, 2026

net: Introduce dual-stream localnet tests - 4784 #4785

Open

openshift-virtualization-qe-bot-3 added the branch-main label May 8, 2026

openshift-virtualization-qe-bot-3 requested a review from vsibirsk May 8, 2026 09:06

openshift-virtualization-qe-bot-3 added needs-rebase size/L labels May 8, 2026

openshift-virtualization-qe-bot-3 requested review from Anatw, dshchedr, frenzyfriday, nirdothan, orelmisan, rnetser, servolkov and yossisegev May 8, 2026 09:06

openshift-virtualization-qe-bot added tox:verify-tc-requirement-polarion:running D/S Testing tox:verify-tc-requirement-polarion:passed D/S Testing and removed tox:verify-tc-requirement-polarion:running D/S Testing labels May 8, 2026

azhivovk force-pushed the new_rhcos9-10_tests_localnet branch from e970d75 to 357bc03 Compare May 8, 2026 09:28

openshift-virtualization-qe-bot-5 added the wip label May 8, 2026

openshift-virtualization-qe-bot-5 changed the title ~~net: Introduce dual-stream localnet tests~~ WIP: net: Introduce dual-stream localnet tests May 8, 2026

openshift-virtualization-qe-bot added tox:verify-tc-requirement-polarion:running D/S Testing and removed tox:verify-tc-requirement-polarion:passed D/S Testing tox:verify-tc-requirement-polarion:running D/S Testing labels May 8, 2026

openshift-virtualization-qe-bot added the tox:verify-tc-requirement-polarion:passed D/S Testing label May 8, 2026

azhivovk and others added 3 commits May 8, 2026 13:38

net, localnet: extend localnet_vm with optional node selector

e888103

Signed-off-by: Asia Khromov <azhivovk@redhat.com> Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

azhivovk force-pushed the new_rhcos9-10_tests_localnet branch from 357bc03 to 440185e Compare May 8, 2026 10:38

openshift-virtualization-qe-bot added tox:verify-tc-requirement-polarion:running D/S Testing tox:verify-tc-requirement-polarion:passed D/S Testing and removed tox:verify-tc-requirement-polarion:passed D/S Testing tox:verify-tc-requirement-polarion:running D/S Testing labels May 8, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

WIP: net: Introduce dual-stream localnet tests#4784

WIP: net: Introduce dual-stream localnet tests#4784
azhivovk wants to merge 3 commits intoRedHatQE:mainfrom
azhivovk:new_rhcos9-10_tests_localnet

azhivovk commented May 8, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented May 8, 2026 •

edited

Loading

Review skipped

Uh oh!

openshift-virtualization-qe-bot-3 commented May 8, 2026

Uh oh!

azhivovk commented May 8, 2026

Uh oh!

azhivovk commented May 8, 2026

Uh oh!

azhivovk commented May 8, 2026

Uh oh!

openshift-virtualization-qe-bot commented May 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

azhivovk commented May 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Short description:

More details:

What this PR does / why we need it:

Which issue(s) this PR fixes: -

Special notes for reviewer:

jira-ticket: https://redhat.atlassian.net/browse/CNV-86194

Uh oh!

coderabbitai Bot commented May 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

openshift-virtualization-qe-bot-3 commented May 8, 2026

Welcome! 🎉

🔄 Automatic Actions

📋 Available Commands

PR Status Management

Review & Approval

Testing & Validation

Container Operations

Cherry-pick Operations

Label Management

✅ Merge Requirements

📊 Review Process

💡 Tips

Uh oh!

azhivovk commented May 8, 2026

Uh oh!

azhivovk commented May 8, 2026

Uh oh!

azhivovk commented May 8, 2026

Uh oh!

openshift-virtualization-qe-bot commented May 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

azhivovk commented May 8, 2026 •

edited

Loading

coderabbitai Bot commented May 8, 2026 •

edited

Loading