[4.21] net, l2 bridge: Verify iface looping bug fix#4396
Closed
nirdothan wants to merge 88 commits intoRedHatQE:mainfrom
Closed
[4.21] net, l2 bridge: Verify iface looping bug fix#4396nirdothan wants to merge 88 commits intoRedHatQE:mainfrom
nirdothan wants to merge 88 commits intoRedHatQE:mainfrom
Conversation
##### Short description: Updated with the relevant bugs to fix: ``` /tests/storage/test_libguestfs.py: CNV-62312 target versions: ['4.21.0'], do not match expected version ['vfuture', '4.22.0', '4.22.z']. /tests/storage/conftest.py: CNV-18870 target versions: ['4.21.0'], do not match expected version ['vfuture', '4.22.0', '4.22.z']. /tests/storage/cdi_upload/test_upload_virtctl.py: CNV-74020 target versions: ['4.21.0'], do not match expected version ['vfuture', '4.22.0', '4.22.z']. ``` ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". -->
##### Short description: ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated test environment configurations for Polarion verification and Jira version tracking to align with current development branches and target versions. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Bugs was resolved ##### Short description: ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> Signed-off-by: Harel Meir <hmeir@redhat.com>
…k uploader (RedHatQE#3520) cherry-pick RedHatQE#3465 into cnv-4.21 requested-by geetikakay Signed-off-by: Geetika Kapoor <gkapoor@redhat.com> Co-authored-by: Geetika Kapoor <13978799+geetikakay@users.noreply.github.com>
…edHatQE#3506) ##### Short description: To allow open bugs check to be a GH check run, integrated with myk-org/github-webhook-server#961 - tox was updated with an explicit command Manual cherrypick of RedHatQE#3048 ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". -->
…lates test matrix (RedHatQE#3552) cherry-pick RedHatQE#3530 into cnv-4.21 requested-by SiboWang1997 Signed-off-by: Sibo Wang <siwang@redhat.com> Co-authored-by: Sibo Wang <42912340+SiboWang1997@users.noreply.github.com>
…RedHatQE#3541) cherry-pick RedHatQE#3535 into cnv-4.21 requested-by by target-branch label Signed-off-by: Harel Meir <hmeir@redhat.com> Co-authored-by: Harel Meir <hmeir@redhat.com>
…#3566) cherry-pick RedHatQE#3501 into cnv-4.21 requested-by azhivovk Signed-off-by: Asia Khromov <azhivovk@redhat.com> Co-authored-by: Asia Zhivov Khromov <azhivovk@redhat.com>
…HatQE#3564) Manual cherry-pick of RedHatQE#3387. There was no any manual intervention into the code, only cherry-pick itself. **Rationale:** VM migration from the external provider is a part of 4.21 GA, so, it should be in the current branch. **Jira-tickets:** https://issues.redhat.com/browse/CNV-63871, https://issues.redhat.com/browse/CNV-77573 Co-authored-by: Ruth Netser <rnetser@redhat.com>
…e Class Migration tests (RedHatQE#3614) cherry-pick RedHatQE#3571 into cnv-4.21 requested-by jpeimer Automation exception: https://issues.redhat.com/browse/CNV-77793 Co-authored-by: Jenia Peimer <86722603+jpeimer@users.noreply.github.com>
…dHatQE#3591) cherry-pick RedHatQE#3553 into cnv-4.21 requested-by duyanyan Co-authored-by: Yan Du <yadu@redhat.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
##### Short description: Bug fixed & merged. ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: https://issues.redhat.com/browse/CNV-75591 fixed ##### Special notes for reviewer: ##### jira-ticket: NONE Signed-off-by: Harel Meir <hmeir@redhat.com>
…r nodes (RedHatQE#3644) cherry-pick RedHatQE#3584 into cnv-4.21 requested-by servolkov Signed-off-by: Sergei Volkov <sevolkov@redhat.com> Co-authored-by: Sergei Volkov <153028767+servolkov@users.noreply.github.com>
…edHatQE#3683) cherry-pick RedHatQE#3619 into cnv-4.21 ##### jira-ticket: https://issues.redhat.com/browse/CNV-78426 <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> Co-authored-by: Ruth Netser <rnetser@redhat.com>
…E#3638) ##### Short description: Backport of RedHatQE#3504 PR ##### More details: Backport exception ticket - https://issues.redhat.com/browse/CNV-77329 ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". -->
…edHatQE#3714) cherry-pick RedHatQE#3467 into cnv-4.21 requested-by SamAlber Signed-off-by: Samuel Albershtein <salbersh@redhat.com> Co-authored-by: Samuel Albershtein <137210078+SamAlber@users.noreply.github.com>
…tions in upgrade tests (RedHatQE#3671) cherry-pick RedHatQE#3575 into cnv-4.21 requested-by dshchedr Co-authored-by: Den Shchedrivyi <dshchedr@redhat.com>
…ting for guest OS cores (RedHatQE#3673) cherry-pick RedHatQE#3615 into cnv-4.21 requested-by dshchedr Signed-off-by: dshchedr@redhat.com Co-authored-by: Den Shchedrivyi <dshchedr@redhat.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
##### What this PR does / why we need it: IP addresses specification on a VM Tests are aimed to cover the ability to define at VM definition its primary UDN IP address. STP Reference: https://github.com/RedHatQE/openshift-virtualization-tests-design-docs/blob/main/stps/sig-network/ip-request.md ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: This STD has been written based on guidelines and examples introduced in RedHatQE#2881 ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Added tests validating explicit VM IP assignment on user-defined networks: startup connectivity, external reachability, IP preservation across live migration, and retention through power cycles. * Added upgrade-suite test stubs (disabled) to verify IP persistence across cluster upgrades. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Edward Haas <edwardh@redhat.com>
This initial test is defining a VM, adds a specific IP address and checks it can be used to communicate with another VM. Signed-off-by: Edward Haas <edwardh@redhat.com>
Most test scenarios assume static IP configuration in the guest OS. This change adds static IP address configuration through cloud-init. Signed-off-by: Edward Haas <edwardh@redhat.com>
The helper is generic, fits better closer to the vm module and has now another usage there. Signed-off-by: Edward Haas <edwardh@redhat.com>
Test that a VM with an explicit IP address specified can preserve connectivity during live migration. <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> https://issues.redhat.com/browse/CNV-76572 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> * **Tests** * Added a reusable TCP client/server fixture to simplify VM-to-VM connectivity testing. * Updated the live-migration connectivity test to use the new fixture and explicitly verify TCP connectivity before and after migration. * Cleaned up test imports and flow to improve reliability and coverage of VM network migration scenarios. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Edward Haas <edwardh@redhat.com>
…io (RedHatQE#3668) cherry-pick RedHatQE#3303 into cnv-4.21 requested-by OhadRevah Co-authored-by: Ohad Revah <orevah@redhat.com> Co-authored-by: Ruth Netser <rnetser@redhat.com>
##### What this PR does / why we need it: Test that a VM with an explicit IP address specified can preserve its IP address over a power cycle (VM is stopped and started again). The naming of the test has been changed to reflect power cycle (and not lifecycle). ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: This PR depends on: - RedHatQE#3706 - RedHatQE#3708 and only the last commit should be reviewed. ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> https://issues.redhat.com/browse/CNV-76572 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Enhanced IP address preservation test with expanded verification to include post-restart validation steps and parameterized test arguments for more comprehensive validation across different VM and IP configurations. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Edward Haas <edwardh@redhat.com>
Manual cherry-pick RedHatQE#3664 ##### jira-ticket: https://issues.redhat.com/browse/CNV-78427 Signed-off-by: Sergei Volkov <sevolkov@redhat.com>
…reserved_during_server_live_migration (RedHatQE#3739) cherry-pick RedHatQE#3720 into cnv-4.21 requested-by servolkov Signed-off-by: Sergei Volkov <sevolkov@redhat.com> Co-authored-by: Sergei Volkov <153028767+servolkov@users.noreply.github.com>
…fig (RedHatQE#3712) cherry-pick RedHatQE#3399 into cnv-4.21 requested-by Ahmad-Hafe Co-authored-by: Ahmad Hafe <ahafe@redhat.com>
##### What this PR does / why we need it: Backport the IP specification feature coverage (STD + tests) ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: <!-- full-ticket-url needs to be provided. This would add a link to the pull request to the jira and close it when the pull request is merged If the task is not tracked by a Jira ticket, just write "NONE". --> https://issues.redhat.com/browse/CNV-76502
##### Short description: The bug is fixed when updating the windows deployment fixture. ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: ##### jira-ticket: https://issues.redhat.com/browse/CNV-79221
openshift-virtualization-qe-bot-2
requested review from
Ahmad-Hafe,
Anatw,
OhadRevah,
RoniKishner,
SiboWang1997,
azhivovk,
dshchedr,
jpeimer,
kgoldbla and
vsibirsk
Contributor
Author
|
mistake. closing. |
openshift-virtualization-qe-bot
added
tox:verify-tc-requirement-polarion:running
|
D/S test |
openshift-virtualization-qe-bot-2
requested review from
SamAlber,
albarker-rh,
dalia-frank,
duyanyan,
geetikakay,
hmeir,
jerry7z,
josemacassan,
mijankow,
qwang1,
rlobillo,
rnetser and
sarahbx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Short description:
A bug was causing secondary interface to disappear from VMI status.
Validate that kubevirt/kubevirt#17041 had fixed the issue by enhancing the secondary interface stability test.
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
jira-ticket: