Skip to content

fix: exclude extensions from make kuadrant - #1036

Merged
silvi-t merged 1 commit into
Kuadrant:mainfrom
silvi-t:turnoff-extensions
Aug 7, 2026
Merged

fix: exclude extensions from make kuadrant#1036
silvi-t merged 1 commit into
Kuadrant:mainfrom
silvi-t:turnoff-extensions

Conversation

@silvi-t

@silvi-t silvi-t commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Exclude extension tests from the make kuadrant target by adding not extensions marker filter

  • Extension tests remain runnable via the dedicated make extensions target

  • Temporary measure until the extension framework is stabilized

  • Added extensions marker to trlp_metrics tests since they depend on TelemetryPolicy from the extension framework

Summary by CodeRabbit

  • Tests
    • Updated the standard test run to exclude extension-specific tests, alongside standalone, disruptive, and UI tests.
    • Existing parallel test execution and test distribution remain unchanged.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The kuadrant Makefile target now excludes tests marked extensions, alongside the existing excluded markers. Test parallelism and load-file distribution remain unchanged.

Changes

Kuadrant test selection

Layer / File(s) Summary
Update Kuadrant marker filter
Makefile
The pytest marker expression excludes extensions tests in addition to standalone_only, disruptive, and ui tests.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: averevki, crstrn13

Poem

A rabbit checks the test-run trail,
“Extensions stay beyond the gate.”
The markers guide each hopping step,
While parallel runs keep their pace.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title uses Conventional Commits format and clearly identifies the main change to the make kuadrant target.
Description check ✅ Passed The description explains the main change, the dedicated target, and the temporary rationale, but it omits the template headings and verification details.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: Silvia Tarabova <starabov@redhat.com>
@silvi-t
silvi-t force-pushed the turnoff-extensions branch from e7074aa to ff8c05e Compare August 7, 2026 07:39

@trepel trepel 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

@silvi-t
silvi-t merged commit 01d9b45 into Kuadrant:main Aug 7, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants