Skip to content

fix(inkless:ci): copy integrationTest JUnit XML into build/junit-xml - #793

Merged
jeqo merged 1 commit into
mainfrom
jeqo/fix-integrationtest-junit-xml
Sep 9, 2026
Merged

jeqo merged 1 commit into
mainfrom
jeqo/fix-integrationtest-junit-xml

Conversation

@jeqo

@jeqo jeqo commented Sep 9, 2026 •

Copy link
Copy Markdown
Contributor

copyTestXml only ran after the unit test task, so Inkless CI's integrationTest failures never reached junit.py (Found 0 JUnit results). Copy XML from every Test task that did work, and finalize integrationTest the same way.

Tested on failing integration test and now it reports failing test:

Run # Export the catalog here so parameterized display names are checked on
Found 7 JUnit results
Read env GITHUB_WORKSPACE: /home/runner/work/inkless/inkless
Parsing 7 JUnit Report Files
Generating Test Catalog Files
Finished processing 7 reports
Read env GRADLE_TEST_EXIT_CODE: 1
Read env JUNIT_REPORT_URL: https://github.com/aiven/inkless/actions/runs/34324570056/artifacts/10093806833
Read env THREAD_DUMP_URL: 
Could not find env RUN_FLAKY
Found 1 test failures:
FAILED ❌ S3ErrorMetricsTest > "testS3ServerExceptions(int, String, WireMockRuntimeInfo).statusCode=429, metricName=throttling-errors"
Gradle task had a failure exit code. Failing this script.
35 tests cases run in 24s.

34 PASSED ✅, 1 FAILED ❌, 0 FLAKY ⚠️ , 0 SKIPPED 🙈, 0 QUARANTINED 😷, and 0 errors.
Error: Process completed with exit code 1.

compared to https://github.com/aiven/inkless/actions/runs/34228704621/job/102074449310?pr=791

@jeqo
jeqo force-pushed the jeqo/fix-integrationtest-junit-xml branch from 51a5e1a to c93468e Compare September 9, 2026 08:45
@jeqo
jeqo requested a lite review from Copilot September 9, 2026 08:46

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jeqo
jeqo changed the base branch from feat/inkless-s3-api-timeout-defaults to main September 9, 2026 08:47
@jeqo
jeqo requested a lite review from Copilot September 9, 2026 08:47

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.

@jeqo
jeqo marked this pull request as ready for review September 9, 2026 08:48
@jeqo
jeqo requested a review from tvainika September 9, 2026 08:49
copyTestXml only ran after the unit test task, so Inkless CI's
integrationTest failures never reached junit.py (Found 0 JUnit results).
Copy XML from every Test task that did work, and finalize integrationTest
the same way.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jeqo
jeqo force-pushed the jeqo/fix-integrationtest-junit-xml branch from c93468e to f23eeb5 Compare September 9, 2026 09:29
@jeqo
jeqo merged commit 8d97bb4 into main Sep 9, 2026
4 checks passed
@jeqo
jeqo deleted the jeqo/fix-integrationtest-junit-xml branch September 9, 2026 10:26
jeqo added a commit that referenced this pull request Sep 9, 2026
…793)

copyTestXml only ran after the unit test task, so Inkless CI's
integrationTest failures never reached junit.py (Found 0 JUnit results).
Copy XML from every Test task that did work, and finalize integrationTest
the same way.

Co-authored-by: Cursor <cursoragent@cursor.com>
jeqo added a commit that referenced this pull request Sep 9, 2026
…793)

copyTestXml only ran after the unit test task, so Inkless CI's
integrationTest failures never reached junit.py (Found 0 JUnit results).
Copy XML from every Test task that did work, and finalize integrationTest
the same way.

Co-authored-by: Cursor <cursoragent@cursor.com>
jeqo added a commit that referenced this pull request Sep 9, 2026
…793)

copyTestXml only ran after the unit test task, so Inkless CI's
integrationTest failures never reached junit.py (Found 0 JUnit results).
Copy XML from every Test task that did work, and finalize integrationTest
the same way.

Co-authored-by: Cursor <cursoragent@cursor.com>
jeqo added a commit that referenced this pull request Sep 10, 2026
* docs(inkless:sync): archive the 0.48 cherry-pick sessions for 4.1, 4.2, 4.3

Session logs for bringing the three active release branches up to main
(5fa6669, #784) ahead of the 0.48 release. inkless-4.3 took its 8
commits clean. inkless-4.2 and inkless-4.1 took 19 each; the logs record
the conflict resolutions and the sync(compile)/sync(test) adaptations
needed for the 4.2.1 and 4.1.2 APIs (Record package, LogReadResult
constructor, Seq-based DelayedFetch statuses).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(inkless:sync): complete the 0.48 cherry-pick session logs

The three branches took six more main commits (#794, #787, #792, #793,
tables and the verification that covered them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): list inkless-4.3 as an active release branch

inkless-4.3 is on Kafka 4.3.1 (#779) and gets its first Kafka-base tag
with 0.48, so it moves from Planned to Active in RELEASES.md. The release
workflow header names it among the branches the cherry-pick sync has to
cover before a run.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): add 0.48 changelog entry

Generated from inkless-release-0.47..inkless-release-0.48 and curated:
every metric delta traced to its commit, the four config default changes
the configs.rst diff shows added next to the four new keys, and the V28
lock on logs described for operators.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
giuseppelillo pushed a commit that referenced this pull request Sep 18, 2026
* docs(inkless:sync): archive the 0.48 cherry-pick sessions for 4.1, 4.2, 4.3

Session logs for bringing the three active release branches up to main
(5fa6669, #784) ahead of the 0.48 release. inkless-4.3 took its 8
commits clean. inkless-4.2 and inkless-4.1 took 19 each; the logs record
the conflict resolutions and the sync(compile)/sync(test) adaptations
needed for the 4.2.1 and 4.1.2 APIs (Record package, LogReadResult
constructor, Seq-based DelayedFetch statuses).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(inkless:sync): complete the 0.48 cherry-pick session logs

The three branches took six more main commits (#794, #787, #792, #793,
tables and the verification that covered them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): list inkless-4.3 as an active release branch

inkless-4.3 is on Kafka 4.3.1 (#779) and gets its first Kafka-base tag
with 0.48, so it moves from Planned to Active in RELEASES.md. The release
workflow header names it among the branches the cherry-pick sync has to
cover before a run.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): add 0.48 changelog entry

Generated from inkless-release-0.47..inkless-release-0.48 and curated:
every metric delta traced to its commit, the four config default changes
the configs.rst diff shows added next to the four new keys, and the V28
lock on logs described for operators.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
giuseppelillo pushed a commit that referenced this pull request Sep 18, 2026
* docs(inkless:sync): archive the 0.48 cherry-pick sessions for 4.1, 4.2, 4.3

Session logs for bringing the three active release branches up to main
(5fa6669, #784) ahead of the 0.48 release. inkless-4.3 took its 8
commits clean. inkless-4.2 and inkless-4.1 took 19 each; the logs record
the conflict resolutions and the sync(compile)/sync(test) adaptations
needed for the 4.2.1 and 4.1.2 APIs (Record package, LogReadResult
constructor, Seq-based DelayedFetch statuses).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(inkless:sync): complete the 0.48 cherry-pick session logs

The three branches took six more main commits (#794, #787, #792, #793,
tables and the verification that covered them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): list inkless-4.3 as an active release branch

inkless-4.3 is on Kafka 4.3.1 (#779) and gets its first Kafka-base tag
with 0.48, so it moves from Planned to Active in RELEASES.md. The release
workflow header names it among the branches the cherry-pick sync has to
cover before a run.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): add 0.48 changelog entry

Generated from inkless-release-0.47..inkless-release-0.48 and curated:
every metric delta traced to its commit, the four config default changes
the configs.rst diff shows added next to the four new keys, and the V28
lock on logs described for operators.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
giuseppelillo pushed a commit that referenced this pull request Sep 24, 2026
* docs(inkless:sync): archive the 0.48 cherry-pick sessions for 4.1, 4.2, 4.3

Session logs for bringing the three active release branches up to main
(5fa6669, #784) ahead of the 0.48 release. inkless-4.3 took its 8
commits clean. inkless-4.2 and inkless-4.1 took 19 each; the logs record
the conflict resolutions and the sync(compile)/sync(test) adaptations
needed for the 4.2.1 and 4.1.2 APIs (Record package, LogReadResult
constructor, Seq-based DelayedFetch statuses).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(inkless:sync): complete the 0.48 cherry-pick session logs

The three branches took six more main commits (#794, #787, #792, #793,
tables and the verification that covered them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): list inkless-4.3 as an active release branch

inkless-4.3 is on Kafka 4.3.1 (#779) and gets its first Kafka-base tag
with 0.48, so it moves from Planned to Active in RELEASES.md. The release
workflow header names it among the branches the cherry-pick sync has to
cover before a run.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* docs(inkless:release): add 0.48 changelog entry

Generated from inkless-release-0.47..inkless-release-0.48 and curated:
every metric delta traced to its commit, the four config default changes
the configs.rst diff shows added next to the four new keys, and the V28
lock on logs described for operators.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.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.

3 participants