Skip to content

@W-23751548: Preserve release archive artifacts - #174

Merged
mlischetti merged 3 commits into
masterfrom
fix-release-asset-allow-list
Sep 3, 2026
Merged

@W-23751548: Preserve release archive artifacts#174
mlischetti merged 3 commits into
masterfrom
fix-release-asset-allow-list

Conversation

@mlischetti

@mlischetti mlischetti commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • retain single-file CLI ZIP and Python wheel CI artifacts using archive: false
  • download CLI ZIPs and Python wheels without decompression
  • download artifact categories separately and only publish archives plus native-library files
  • fail the matrix build if a CLI ZIP or Python wheel is missing
  • remove the unused Python TCK JUnit report generation and upload

Root Cause

The publisher downloaded every artifact and allowed download-artifact@v8 to decompress CLI ZIP payloads. The broad regular-file scan then sent CLI internals, such as ndjson-module-2.12.2.jar, to the GitHub Release.

Verification

  • python3 -m pytest native-lib/python/tests/unit/test_ci_structure.py
  • Python YAML parsing for all changed Actions YAML files
  • git diff --check
  • Read-only code review: no findings

@mlischetti
mlischetti requested a review from a team as a code owner September 3, 2026 17:25
@mlischetti mlischetti changed the title Fix release asset selection @W-23751548: Fix release asset selection Sep 3, 2026
@mlischetti mlischetti changed the title @W-23751548: Fix release asset selection Preserve release archive artifacts Sep 3, 2026
@mlischetti mlischetti changed the title Preserve release archive artifacts @W-23751548: Preserve release archive artifacts Sep 3, 2026
@mlischetti
mlischetti merged commit 1b62b05 into master Sep 3, 2026
5 checks passed
@mlischetti
mlischetti deleted the fix-release-asset-allow-list branch September 3, 2026 19:58
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