Add lp-interop--OPP parent component mapping - #853
redhat-chai-bot wants to merge 1 commit into
Conversation
|
/test all AI-generated. Review for accuracy. |
|
/assign amp-rh AI-generated. Review for accuracy. |
|
/ok-to-test AI-generated. Review for accuracy. |
openshift-ci
Bot
added
the
ok-to-test
|
@sosiouxme Could you please review and AI-generated. Review for accuracy. |
Register the lp-interop--OPP component to map OPP (Operator Partner Program) interop test results from stolostron/policy-collection CI jobs to the correct Sippy/Component Readiness component. The component matches tests by: - Exact suite match on "lp-interop--OPP" - Suite regex match on "^lp-interop--OPP--interop-opp-" covering compound suite names like lp-interop--OPP--interop-opp-smoke, lp-interop--OPP--interop-opp-preflight, etc. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
b1d8362 to
32c5da1
Compare
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: amp-rh, redhat-chai-bot The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 @dgoodwin @deepsm007 — friendly ping for review! CI is all green (lint ✅, mapping ✅, unit ✅). This PR already has AI-generated. Review for accuracy. |
|
@sosiouxme — friendly ping! This PR has AI-generated. Review for accuracy. |
The ^acm-opp-app$ regex can never match because BigQuery ingestion filters suites to lp-interop--%. Update the regex to ^lp-interop--OPP--acm-opp-app$ to match the actual compound suite naming convention. Companion fix to PR #853.
|
@redhat-chai-bot you are pinging a lot of people on this pr, you also note that @sosiouxme added |
3 participants
Summary
Add the parent
lp-interop--OPPcomponent mapping for the OPP (Operator Partner Program) interop tests. This enables Sippy, TestGrid, and Component Readiness to correctly attribute OPP tests to the OPP component.Part of INTEROP-9271 (Phase 4 of INTEROP-8941).
Changes
pkg/components/lpinteropopp/component.go— DefinesLPinteropOPPComponentwith:lp-interop--OPPlp-interop--OPPand regex^interop-opp-(all OPP test steps)lpinteropoppackm,lpinteropoppodf,lpinteropoppquay)pkg/components/lpinteropopp/capabilities.go— Standard capabilities usingutil.DefaultCapabilitiespkg/registry/registry.go— Registers the new component alongside existing OPP sub-component registrationsContext
The OPP team runs layered product interop tests under
stolostron/policy-collection. Sub-component mappings already exist for individual products (ACM, ODF, Quay), but the parentlp-interop--OPPcomponent was missing. This mapping captures:^interop-opp-that aren't covered by product-specific sub-componentsTesting
go build .— passesgo test ./...— all packages passAI-generated. Review for accuracy.
@amp-rh requested via Chai Bot