Skip to content

Remap egressip test names for Polarion upload - #28

Merged
openshift-merge-bot[bot] merged 1 commit into
mainfrom
fix-egressip-polarion-classname
Jul 30, 2026
Merged

Remap egressip test names for Polarion upload#28
openshift-merge-bot[bot] merged 1 commit into
mainfrom
fix-egressip-polarion-classname

Conversation

@tusharjadhav3302

@tusharjadhav3302 tusharjadhav3302 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix Polarion upload failure for egressip test results by remapping
openshift-tests-private test names to the sig-installer.Suite_openshift_openstack.*
pattern that JUMP expects.

Problem

EgressIP tests run via openshift-tests-private (extended-platform-tests) produce
JUnit XML with classname="sig-networking" and names like SDN_OVN_EgressIP_*.
JUMP builds match keys as classname.namesig-networking.SDN_OVN_EgressIP_* which
doesn't match any Polarion test case, causing:

JUMP: ERROR   Cannot proceed without xml with tempest results

PR #27 fixed the OTP. prefix issue for the main openstack tests (from openshift-tests)
but didn't address egressip because it uses a fundamentally different naming convention
from openshift-tests-private.

Fix

Add a remap_private_test_name() function that detects known private-test suites
(via PRIVATE_TEST_SUITES dict) and rewrites their names to the expected Polarion pattern:

sig-networking.SDN_OVN_EgressIP_*sig-installer.Suite_openshift_openstack.egressip.SDN_OVN_EgressIP_*

The mapping is extensible for future private-test suites.

Before/After

Before: classname="sig-networking" name="SDN_OVN_EgressIP_Author_huirwang-..." → JUMP key: sig-networking.SDN_OVN_EgressIP_... (no Polarion match → crash)

After: classname="sig-installer" name="Suite_openshift_openstack.egressip.SDN_OVN_EgressIP_Author_huirwang-..." → JUMP key: sig-installer.Suite_openshift_openstack.egressip.SDN_OVN_EgressIP_... (matches Polarion convention)

Validation

Traced transform on sample test name from Zuul artifacts (build 369a5d61, job shiftstack-rhel9-rhoso18.0-ocp423):

  • Input: [sig-networking] SDN OVN EgressIP Author:huirwang-ConnectedOnly-Medium-47272-...
  • Output: sig-installer.Suite_openshift_openstack.egressip.SDN_OVN_EgressIP_Author_huirwang-ConnectedOnly-Medium-47272-...

Note

For results to appear as linked test records in Polarion, egressip test cases must
be registered in the RHOSO project under the sig-installer.Suite_openshift_openstack.egressip.*
pattern — or cifmw_polarion_create_testcases must be enabled for the first run to
auto-create them.

Comment thread collection/tools/roles/tools_openshift_tests/scripts/modifyE2ETags.py Outdated
Tests from openshift-tests-private use [sig-networking] classname
and SDN_OVN_EgressIP_* naming. After format_test_case_name() these
become sig-networking.SDN_OVN_EgressIP_* which JUMP cannot match
against Polarion entries, causing "Cannot proceed without xml with
tempest results" and dropping all egressip results from reporting.

Add a remap step for known private-test suites (egressip_tests) that
rewrites names to sig-installer.Suite_openshift_openstack.egressip.*
matching the convention used by the main openstack test suites.

Co-authored-by: Cursor <[email protected]>
@tusharjadhav3302
tusharjadhav3302 force-pushed the fix-egressip-polarion-classname branch from e58e2c6 to ddb39b2 Compare July 30, 2026 09:00
@tusharjadhav3302 tusharjadhav3302 added the ready-for-review PR is ready for code review label Jul 30, 2026

@ekuris-redhat ekuris-redhat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci

openshift-ci Bot commented Jul 30, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ekuris-redhat

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit e5dcdea into main Jul 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm ready-for-review PR is ready for code review

Development

Successfully merging this pull request may close these issues.

2 participants