Skip to content

Allowlist carabiner ampel-bootstrap@v1.2.1 (fix transitive-failures canary)#1037

Merged
raboof merged 1 commit into
mainfrom
fix/transitive-carabiner-ampel-bootstrap
Jul 14, 2026
Merged

Allowlist carabiner ampel-bootstrap@v1.2.1 (fix transitive-failures canary)#1037
raboof merged 1 commit into
mainfrom
fix/transitive-carabiner-ampel-bootstrap

Conversation

@potiuk

@potiuk potiuk commented Jul 13, 2026

Copy link
Copy Markdown
Member

Problem

The scheduled Check for transitive failures canary has been failing on every run (all day 2026-07-12). The composite probes carabiner-dev/actions/install/download-and-verify@f882fa9e (v1.2.1), whose action.yml pulls carabiner-dev/actions/install/ampel-bootstrap@94f29392 (v1.2.1) -- a transitive dependency that was not on the allowlist -- so GitHub's org policy blocks the whole composite from loading:

##[error]The action carabiner-dev/actions/install/ampel-bootstrap@94f29392187fe5082d1195a7d4cae3a7ddf09d9c
is not allowed in apache/infrastructure-actions ...

Root cause

The v1.2.1 bump (94f29392) was allowlisted for ampel/verify, install/ampel and install/bnd, but the transitive install/ampel-bootstrap@94f29392 ref was missed. It isn't a directly-probed step (it's reached via download-and-verify@f882fa9e), so the update job never learned about it and it had to be added by hand -- like the other transitive carabiner refs in actions.yml.

Traced chain confirming it's the only gap:

  • download-and-verify@f882fa9e (v1.2.1, probed) -> ampel-bootstrap@94f29392 (v1.2.1) <- was missing
  • the v1.1.7 branch (ampel/verify@94f29392 -> install/{ampel,bnd}@2a11d59a -> download-and-verify@6022a065 -> ampel-bootstrap@...) resolves entirely through already-allowlisted SHAs.

Fix

Add ampel-bootstrap@94f29392 (v1.2.1) with a 12-week expiry (the gateway's standard for transitive refs), and regenerate approved_patterns.yml. It's the same commit already trusted via three sibling subpaths, so no new trust surface. The composite is unchanged -- the entry carries an expiry, so it's not a dependabot-update candidate.

Once this syncs to the org allow list, the canary should go green on its next run.

🤖 Generated with Claude Code

…anary)

The scheduled 'Check for transitive failures' workflow has been failing every
run because the composite probes carabiner-dev/actions/install/download-and-verify
@f882fa9e (v1.2.1), whose action.yml pulls
carabiner-dev/actions@94f29392 (v1.2.1) -- a transitive
dependency that was not on the allowlist. GitHub's org-wide policy then blocks
the whole composite from loading:

  The action carabiner-dev/actions@94f29392... is not
  allowed in apache/infrastructure-actions

The v1.2.1 bump (94f29392) had been added for ampel/verify, install/ampel and
install/bnd, but this transitive ampel-bootstrap ref was missed (it is not a
directly-probed step, so the update job never learned about it).

Add ampel-bootstrap@94f29392 (v1.2.1) with a 12-week expiry, matching how the
other manually-added transitive carabiner refs are managed. It is the same
commit already trusted via three sibling subpaths, so no new trust surface.
approved_patterns.yml regenerated to match (composite unchanged -- the entry
carries an expiry so it is not a dependabot-update candidate).

Generated-by: Claude Opus 4.8 (1M context) via Claude Code
Comment thread actions.yml
# completes the v1.2.1 (94f29392) approval so the transitive-failures canary resolves.
94f29392187fe5082d1195a7d4cae3a7ddf09d9c:
tag: v1.2.1
expires_at: 2026-10-05

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why does this entry need an expiration?

@raboof raboof merged commit e03060d into main Jul 14, 2026
11 checks passed
@raboof raboof deleted the fix/transitive-carabiner-ampel-bootstrap branch July 14, 2026 09:49
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