Skip to content

feat: capture hostname field from JUnit XML test suites#43

Open
rhopp wants to merge 1 commit into
konflux-ci:mainfrom
rhopp:addHostname
Open

feat: capture hostname field from JUnit XML test suites#43
rhopp wants to merge 1 commit into
konflux-ci:mainfrom
rhopp:addHostname

Conversation

@rhopp

@rhopp rhopp commented Jan 16, 2026

Copy link
Copy Markdown

Summary

Adds tracking of hostname property of junit files.

This is required, because Playwright junit reporter populates this field with what's called Project in Playwright. This field contains important information for us (it basically contains currently tested configuration)

Example of such xunit file:

<testsuites id="" name="" tests="18" failures="1" skipped="4" errors="0" time="1105.067546">
<testsuite name="tssc/full_workflow.test.ts" timestamp="2026-01-16T11:02:39.362Z" hostname="e2e-go[github-tekton-quay-local]" tests="6" failures="1" skipped="4" time="151.114" errors="0">
<testcase name="TSSC Complete Workflow › Component Creation › should create a component successfully" classname="tssc/full_workflow.test.ts" time="45.914">
<system-out>
...

Other Information

Any other information that is important to this PR.

@snyk-io

snyk-io Bot commented Jan 16, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@rhopp

rhopp commented Jan 16, 2026

Copy link
Copy Markdown
Author

@flacatus I'm not sure how to properly test this locally, so this PR is sadly untested. Feel free to return this back to me if needed.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Radim Hopp <rhopp@redhat.com>
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.

1 participant