Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: RedHatQE/openshift-virtualization-tests
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main@{1day}
Choose a base ref
...
head repository: RedHatQE/openshift-virtualization-tests
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
  • 6 commits
  • 9 files changed
  • 4 contributors

Commits on Mar 5, 2026

  1. 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>
    azhivovk committed Mar 5, 2026
    Configuration menu
    Copy the full SHA
    31ef485 View commit details
    Browse the repository at this point in the history
  2. 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>
    azhivovk committed Mar 5, 2026
    Configuration menu
    Copy the full SHA
    a1db96e View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2026

  1. [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 -->
    kshvaika authored Mar 24, 2026
    Configuration menu
    Copy the full SHA
    2ed26a5 View commit details
    Browse the repository at this point in the history
  2. 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>
    RoniKishner authored Mar 24, 2026
    Configuration menu
    Copy the full SHA
    fb1a391 View commit details
    Browse the repository at this point in the history
  3. 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 -->
    rnetser authored Mar 24, 2026
    Configuration menu
    Copy the full SHA
    5c2c33a View commit details
    Browse the repository at this point in the history
  4. 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>
    azhivovk authored Mar 24, 2026
    Configuration menu
    Copy the full SHA
    2609d0b View commit details
    Browse the repository at this point in the history
Loading