-
Notifications
You must be signed in to change notification settings - Fork 61
Comparing changes
Open a pull request
base repository: RedHatQE/openshift-virtualization-tests
base: main@{1day}
head repository: RedHatQE/openshift-virtualization-tests
compare: main
- 6 commits
- 9 files changed
- 4 contributors
Commits on Mar 5, 2026
-
net, l2 bridge: Refactor setup
- Client is a mandatory parameter - Assign the DHCP interface on each VM according to the type client/server Signed-off-by: Asia Khromov <azhivovk@redhat.com>
Configuration menu - View commit details
-
Copy full SHA for 31ef485 - Browse repository at this point
Copy the full SHA 31ef485View commit details -
net, l2 bridge: Drop matrix in setup
During the L2 bridge test setup, a matrix was previously used so the fixtures would run twice: once with the Linux bridge and once with the OVS bridge. Since the use of a matrix is not explicit and can be confusing, the tests now use pytest.parametrize at the test class level instead. Signed-off-by: Asia Khromov <azhivovk@redhat.com>
Configuration menu - View commit details
-
Copy full SHA for a1db96e - Browse repository at this point
Copy the full SHA a1db96eView commit details
Commits on Mar 24, 2026
-
[Storage] replace artifactory with quay in smoke clone tests (#3678)
##### Short description: https://issues.redhat.com/browse/CNV-77532 ##### More details: ##### What this PR does / why we need it: ##### Which issue(s) this PR fixes: ##### Special notes for reviewer: The PR was assisted by Cursor AI ##### 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** * Replaced Cirros-based test fixtures and data sources with Fedora-based ones for storage clone scenarios. * Added Fedora data sources for filesystem and block volume modes; removed the corresponding Cirros fixtures. * Updated VM flavor and memory defaults to Fedora values. * Adjusted clone test references and size/overhead calculations to use Fedora images. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Configuration menu - View commit details
-
Copy full SHA for 2ed26a5 - Browse repository at this point
Copy the full SHA 2ed26a5View commit details -
instance_type_centos_os_list os_name changed from centos to centos.st…
…ream (#4196) ##### Short description: Changed instance_type_centos_os_list os_name changed from centos to centos.stream ##### More details: instance_type_centos_os_list os_name was incorrect as the data source and preference name are centos-stream/centos.stream leading to failure on VM creation in test_centos_os.py ##### What this PR does / why we need it: Fix failing tests at tests/infrastructure/instance_types/supported_os/test_centos_os.py <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Standardized CentOS naming in build/test matrices to "centos.stream" and simplified architecture suffix handling so S390X no longer receives special conditional treatment. * **Tests** * Updated unit tests to expect the "centos.stream" naming and the revised arch-suffix behavior across affected matrix scenarios. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Roni Kishner <rkishner@redhat.com>
Configuration menu - View commit details
-
Copy full SHA for fb1a391 - Browse repository at this point
Copy the full SHA fb1a391View commit details -
net, l2 bridge: Small fixes (#4097)
Small follow-up PR to apply requested changes: #3567 (comment) #3567 (comment) - Client is a mandatory parameter - Assign the DHCP interface on each VM according to the type client/server - Drop matrix in setup and use pytest.parametrize <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Expanded L2 bridge tests to run against both Linux and OVS bridge types and added a class-scoped fixture for selecting device type. * **Refactor** * Test fixtures and VM setup updated to propagate the chosen bridge device type and include explicit DHCP interface configuration for more consistent test provisioning. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Configuration menu - View commit details
-
Copy full SHA for 5c2c33a - Browse repository at this point
Copy the full SHA 5c2c33aView commit details -
net, network policy: Use fixtures' ports (#4051)
Small follow-up requested change https://github.com/RedHatQE/openshift-virtualization-tests/pull/3588/changes#r2753623060 Use the allowed ports provided by the fixtures in network policy object instead of the ports constant to improve test readability, and clearly distinguish between allowed and disallowed ports. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Updated network policy tests: renamed and reworked test fixtures and signatures to remove redundant decorators, tests now consume targeted fixtures and iterate over provided ports. This improves test clarity, reduces duplication, and makes the network policy test suite more reliable and easier to extend and maintain. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Asia Khromov <azhivovk@redhat.com>
Configuration menu - View commit details
-
Copy full SHA for 2609d0b - Browse repository at this point
Copy the full SHA 2609d0bView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff main@{1day}...main