Skip to content

Add branch:tag override syntax in branches.yaml#381

Merged
barnabasbusa merged 1 commit into
masterfrom
bbusa/branches-tag-override
May 18, 2026
Merged

Add branch:tag override syntax in branches.yaml#381
barnabasbusa merged 1 commit into
masterfrom
bbusa/branches-tag-override

Conversation

@barnabasbusa
Copy link
Copy Markdown
Collaborator

Summary

  • branches.yaml now accepts <branch>:<tag> entries — build from <branch> but publish under <tag>. Replaces the previously unused @ separator (which read as a git commit specifier and was a footgun), freeing @ for future commit-pinning.
  • Applied to teku focil: prototype/focil:focil so it tracks the actual upstream branch instead of the nonexistent focil (which is why scheduled focil builds were failing).
  • README + branches.yaml header updated to document the syntax.

Test plan

  • python3 generate_config.py runs cleanly — generates target.tag: focil with source.ref: prototype/focil for teku focil.
  • Trigger scheduled or manual teku focil build after merge and confirm it pulls prototype/focil and publishes ethpandaops/teku:focil.

Lets a build pull from one branch but publish under a different docker tag,
e.g. `prototype/focil:focil` builds the prototype/focil branch and tags it
`focil`. Replaces the previously unused `@` separator with `:`, which reads
more naturally and frees `@` for future commit-pinning use.

Applied to teku focil so it tracks consensys/teku@prototype/focil instead of
the nonexistent `focil` branch (which is why scheduled builds for focil were
failing).
@qu0b-reviewer
Copy link
Copy Markdown

qu0b-reviewer Bot commented May 18, 2026

🤖 qu0b-reviewer

The diff is clean and well-executed. Swapping @ for : in the override syntax is a straightforward, isolated change.

No blockers. Looks good. 🦋


Reviewed @ 509d1951

@barnabasbusa barnabasbusa merged commit 6d050c3 into master May 18, 2026
3 checks passed
@barnabasbusa barnabasbusa deleted the bbusa/branches-tag-override branch May 18, 2026 11:13
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