Skip to content

ci: cancel stale Go Tests runs on force-push#5738

Merged
eriknordmark merged 1 commit intolf-edge:masterfrom
rucoder:rucoder/go-tests-concurrency
Apr 3, 2026
Merged

ci: cancel stale Go Tests runs on force-push#5738
eriknordmark merged 1 commit intolf-edge:masterfrom
rucoder:rucoder/go-tests-concurrency

Conversation

@rucoder
Copy link
Copy Markdown
Contributor

@rucoder rucoder commented Apr 2, 2026

Description

Add concurrency group to the Go Tests workflow so that a new push to the same branch (or PR update) automatically cancels any in-progress run for the previous commit. Without this, force-pushing a PR branch
accumulates multiple parallel Go Tests runs that hold runners and produce stale results.

How to test and validate this PR

  • Force-push a PR branch twice in quick succession
  • Verify that the first Go Tests run is cancelled when the second starts

Changelog notes

No user-facing changes.

PR Backports

  • 16.0-stable: yes
  • 14.5-stable: yes
  • 13.4-stable: yes

Checklist

  • I've provided a proper description

  • I've written the test verification instructions

  • I've set the proper labels to this PR

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

@eriknordmark
Copy link
Copy Markdown
Contributor

@rucoder can you rebase on master so we can get the yetus to run.

Add concurrency group to the Go Tests workflow so that a new push to
the same branch (or PR update) automatically cancels any in-progress
run for the previous commit. Without this, force-pushing a PR branch
accumulates multiple parallel Go Tests runs that hold runners and
produce stale results.

Signed-off-by: Mikhail Malyshev <mike.malyshev@gmail.com>
@rucoder rucoder force-pushed the rucoder/go-tests-concurrency branch from cc8843e to f1398ba Compare April 3, 2026 09:12
@rucoder
Copy link
Copy Markdown
Contributor Author

rucoder commented Apr 3, 2026

@eriknordmark rebased

@rucoder rucoder added the stable Should be backported to stable release(s) label Apr 3, 2026
Copy link
Copy Markdown
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

Run tests

@eriknordmark eriknordmark merged commit ae2d51e into lf-edge:master Apr 3, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stable Should be backported to stable release(s)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants