Skip to content

Work around stale results in GH jobs API - #2720

Merged
pkwarren merged 1 commit into
mainfrom
pkw/BSR-6629-ci-changed-plugins-from-push
Sep 3, 2026
Merged

Work around stale results in GH jobs API#2720
pkwarren merged 1 commit into
mainfrom
pkw/BSR-6629-ci-changed-plugins-from-push

Conversation

@pkwarren

@pkwarren pkwarren commented Sep 3, 2026

Copy link
Copy Markdown
Member

Starting yesterday, the GH jobs API started returning stale commit information (seemingly random). This caused the code which detects the plugins to build (from the last successful CI run) to select a large number of plugins to build and publish which were unchanged.

This adds unnecessary load on the system and creates new plugin revisions in the BSR unnecessarily.

Update the CI workflow to use the new queue: max concurrency setting to ensure we build every commit pushed to main. With this setting in place, we only need to look at what changed since the prior commit (assuming all commits will build). If we ever need to force a rebuild of a plugin, that is possible with a manual run of the CI workflow and specifying the PLUGINS input.

Starting yesterday, the GH jobs API started returning stale commit
information (seemingly random). This caused the code which detects the
plugins to build (from the last successful CI run) to select a large
number of plugins to build and publish which were unchanged.

This adds unnecessary load on the system and creates new plugin
revisions in the BSR unnecessarily.

Update the CI workflow to use the new `queue: max` concurrency setting
to ensure we build every commit pushed to main. With this setting in
place, we only need to look at what changed since the prior commit
(assuming all commits will build). If we ever need to force a rebuild of
a plugin, that is possible with a manual run of the CI workflow and
specifying the PLUGINS input.
@pkwarren
pkwarren merged commit 5a93431 into main Sep 3, 2026
4 checks passed
@pkwarren
pkwarren deleted the pkw/BSR-6629-ci-changed-plugins-from-push branch September 3, 2026 21:36
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