Skip to content

chore(ENG-12828): remove PEX .platforms config #320

Merged
BartoszBlizniak merged 6 commits into
masterfrom
eng-12828-remove-dead-pex-platforms-build-config
Jul 17, 2026
Merged

chore(ENG-12828): remove PEX .platforms config #320
BartoszBlizniak merged 6 commits into
masterfrom
eng-12828-remove-dead-pex-platforms-build-config

Conversation

@BartoszBlizniak

Copy link
Copy Markdown
Member

Description

Removes the dead .github/.platforms/ PEX platform configuration. The multi-platform PEX .platforms/*.json files and their generator are superseded by the PyInstaller binary build.

No runtime or build behaviour change.

giphy

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Documentation update
  • Refactoring
  • Other (dead-code / build-config cleanup)

Additional Notes

N/A

@BartoszBlizniak BartoszBlizniak changed the title chore(ENG-12828: remove PEX .platforms config chore(ENG-12828): remove PEX .platforms config Jun 19, 2026
@BartoszBlizniak
BartoszBlizniak force-pushed the eng-12827-publish-standalone-binaries-via-the-release-docker-pipelines branch from 4f2fdab to 81c8125 Compare July 16, 2026 12:15
BartoszBlizniak and others added 3 commits July 16, 2026 17:17
…NG-12827)

Wire the standalone PyInstaller binaries (ENG-12826) into the release and Docker
pipelines, dogfooding the freshly-built cloudsmith binary to authenticate
(GitHub OIDC) and publish:

- Publish per-platform archives + SHA256 to Cloudsmith and build/push the
  container image, using the built binary + OIDC and the Docker credential
  helper instead of cloudsmith-cli-action / static-key docker login.
- GPG-sign the Linux archives (detached .sig sibling files).
- Idempotent, immutable-repo-safe publishing (skip already-published artifacts/tags).
- Tag raw uploads (os/arch/libc/target/type) for queryable CI selection.
- Dockerfile: multi-stage Alpine, musl binary, non-root, OCI labels; Docker Hub floating tags.
- CI hardening + perf: single-source target list, composite bootstrap action,
  GH_REPO for gh, trimmed clean-room docker runs, pinned runners.
- Redact PII from online smoketest logs (SMOKETEST_DEBUG for detail).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- setup-cloudsmith-binary: verify the .sha256 checksum before extracting
  the archive onto PATH, so a corrupted/unexpected artifact is never
  executed (the .sha256 is already downloaded alongside the archive).
- release.yml: document at the binaries call site that publishing is
  OIDC-only and the API key secret is required solely for the read-only
  online smoketest (fail-fast when online_smoketest is requested); the
  OIDC online path is covered by the binaries `oidc` job.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@BartoszBlizniak
BartoszBlizniak force-pushed the eng-12827-publish-standalone-binaries-via-the-release-docker-pipelines branch from 81c8125 to 0ee57a6 Compare July 16, 2026 16:18
@BartoszBlizniak
BartoszBlizniak force-pushed the eng-12828-remove-dead-pex-platforms-build-config branch from 074155a to bac2c7d Compare July 16, 2026 16:20
BartoszBlizniak and others added 2 commits July 16, 2026 17:48
Export cloudsmith-cli-release-key.asc during sign-linux, stage it as a
release asset, assert its presence before publishing, and point the
README verification docs at it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@BartoszBlizniak
BartoszBlizniak force-pushed the eng-12828-remove-dead-pex-platforms-build-config branch from bac2c7d to e492c66 Compare July 16, 2026 16:49
@BartoszBlizniak
BartoszBlizniak marked this pull request as ready for review July 17, 2026 09:31
@BartoszBlizniak
BartoszBlizniak requested a review from a team as a code owner July 17, 2026 09:31
Base automatically changed from eng-12827-publish-standalone-binaries-via-the-release-docker-pipelines to master July 17, 2026 10:31
@BartoszBlizniak
BartoszBlizniak merged commit 13a3f63 into master Jul 17, 2026
9 checks passed
@BartoszBlizniak
BartoszBlizniak deleted the eng-12828-remove-dead-pex-platforms-build-config branch July 17, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants