Skip to content

fix: stage LP4 architecture gates - #7

Merged
Sythos merged 2 commits into
mainfrom
fix/lp4-architecture-gates
Aug 25, 2026
Merged

fix: stage LP4 architecture gates#7
Sythos merged 2 commits into
mainfrom
fix/lp4-architecture-gates

Conversation

@Sythos

@Sythos Sythos commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • make the LP4 commit gate fast amd64-first by default
  • keep functional, Compose, and typed contract proofs on amd64
  • add an explicit final ARM64-only multiarch gate with QEMU and OCI attestation verification
  • record the architecture mode in the LP4 manifest and documentation

Verification

The branch is a temporary correction branch and may be deleted after the required checks pass and the PR is merged.

@Sythos
Sythos merged commit 49a0c89 into main Aug 25, 2026
2 checks passed
@Sythos
Sythos deleted the fix/lp4-architecture-gates branch August 25, 2026 08:22

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f4f720c290

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

uses: ./.github/workflows/quality-gates.yml
with:
context: pull-request
architecture_mode: ${{ inputs.architecture_mode || 'amd64' }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Require ARM64 validation before merge

Checked both caller workflows: automatic pull-request and push events leave inputs.architecture_mode unset, so this expression always selects amd64; multiarch is reachable only through a manual dispatch. Moreover, both modes emit the same reusable-job check name, so branch protection cannot specifically require the manual ARM64 result. A successful amd64 check can therefore permit merging without any ARM64 validation, contrary to the new arm64RequiredBeforeMergeOrRelease contract; expose ARM64 as a distinct automatically triggered required check or otherwise make the dependency enforceable.

Useful? React with 👍 / 👎.

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