Skip to content

IBX-11778: Updated deprecated GitHub Actions in auto_tag workflow#1891

Draft
alongosz wants to merge 4 commits into
4.6from
ibx-11778-gha-deprecated-actions
Draft

IBX-11778: Updated deprecated GitHub Actions in auto_tag workflow#1891
alongosz wants to merge 4 commits into
4.6from
ibx-11778-gha-deprecated-actions

Conversation

@alongosz

@alongosz alongosz commented Jul 21, 2026

Copy link
Copy Markdown
Member

Caution

  • Drop TMP commits before merging
🎫 Issue IBX-11778

Related PRs:

Description:

Upgraded or replaced the GitHub Actions in the auto_tag.yml workflow that still run on the deprecated Node.js 20 runtime (to be removed from GitHub runners on September 16th, 2026):

  • Replaced deprecated tibdex/github-app-token@v1 with actions/create-github-app-token@v3 (switching to client-id/private-key inputs; installation_id is no longer needed);
  • Replaced octokit/request-action@v2.x usages with equivalent gh api calls, preserving the data output consumed by later steps;
  • Upgraded actions/checkout to v7 and both jobs to the ubuntu-26.04 runner;
  • Removed dead steps guarded by other editions' repository names (the file was designed to be copy-pasted between edition repositories, but the copies have since diverged anyway — including a stale content condition that can never match since the rename to headless), an always-true if, and the unused cs2pr tool;
  • Fixed the Satis wait step: previously it could never fail (the for loop exit status masked the check result) and its final backoff sleep overran the 30-minute job timeout. It now re-checks on the 5-minute Satis cron cadence (30s/60s/120s ramp-up, then 5 × 300s), fails with an explicit error when the parent version never appears, and stays within the unchanged timeout. Also renamed the step to make clear it waits for the parent package ibexa/experience.
  • Replaced the legacy EZROBOT_PAT (ezrobot personal access token) in the Commit, tag and push step with the GitHub App token the job already generates: the step authenticates git through gh auth setup-git with the token in its env — the credential flow stays local to the step and no secret is written into the remote URL / .git/config. The release commit is now authored as github-actions[bot], and inputs.version is passed to the shell via the step env instead of raw workflow-expression interpolation.

All other workflows in this repository only call ibexa/gh-workflows/...@main reusable workflows (deliberately trunk-tracking, already covered by ibexa/gh-workflows#103) and required no changes.

Update: the workflow has since been refactored on this branch to use the shared composite actions from ibexa/gh-workflows#105 (check-composer-package + create-metapackage-tag) — the file now keeps only edition-specific data. The preparation check now queries the Composer repository (Packagist/Satis) that composer update actually consumes, instead of probing for a GitHub tag. A [TMP] commit points the action references at the gh-workflows feature branch for testing; it must be dropped before merging.

For QA:

No manual QA required. The workflow can be validated by dispatching Create Tag with real_op: false (dry run — no tag is pushed) once a suitable release.json exists in ibexa/release-maker.

Before merging: the Commit, tag and push path only runs with real_op: true, so PR CI does not exercise it — verify the automation GitHub App has contents write permission and any tag protection allowances previously granted to the ezrobot account.

Documentation:

No documentation required.

Aligned with the IBX-11778 treatment from ibexa/gh-workflows#103,
adjusted for metapackages:

- Replaced tibdex/github-app-token@v1 with
  actions/create-github-app-token@v3 (client-id/private-key inputs)
- Replaced octokit/request-action with equivalent gh api calls,
  preserving the data output consumed by later steps
- Bumped actions/checkout to v7 and both jobs to ubuntu-26.04
- Removed steps guarded by other repositories' names (dead code from
  the shared-file era), an always-true if, and the unused cs2pr tool
- Fixed the Satis wait loop: it could never fail (the for loop exit
  status masked the check) and its final sleep overran the job
  timeout; it now re-checks on the 5-minute Satis cron cadence and
  exits 1 when the parent version never appears

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
alongosz and others added 3 commits July 21, 2026 17:00
The Commit, tag and push step used the EZROBOT_PAT personal access
token and rewrote the origin remote URL to embed it, while the job
already generates a GitHub App installation token. Now:

- the push step receives the app token via its GITHUB_TOKEN env and
  authenticates git through gh auth setup-git (gh acting as the git
  credential helper), keeping the whole credential flow local to the
  step - checkout stays credential-free and no secret is written to
  .git/config
- the release commit is authored as github-actions[bot] instead of
  ezrobot, which no longer plays any role here
- inputs.version is passed to the shell via the step's VERSION env
  instead of raw workflow-expression interpolation

Note before merging: this path only runs with real_op: true, so PR CI
does not exercise it. The automation GitHub App must have contents
write permission and any tag protection allowances previously granted
to the ezrobot account.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The Create Tag workflow now keeps only its edition-specific data and
delegates the mechanics to two composite actions in ibexa/gh-workflows:

- check-composer-package (preparation) checks that the required package
  version is available on the Composer repository that composer update
  will actually consume — more accurate than the previous GitHub-tag
  probe, which could pass while the package was not yet installable;
- create-metapackage-tag (action) fetches the release definition, derives
  minimum stability, pins the parent/assets packages, patches require
  versions, updates composer.lock and, with real_op, commits, tags and
  pushes using the GitHub App token.

Behavior is unchanged apart from the more accurate preparation check and
the dropped sponge/moreutils dependency.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Allows testing the new composite actions before the ibexa/gh-workflows
pull request that introduces them is merged. Drop before merging.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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