Skip to content

buildkite: fix GitHub status after build-pr retry - #3350

Merged
reakaleek merged 3 commits into
masterfrom
cursor/a954a8cb
Aug 7, 2026
Merged

buildkite: fix GitHub status after build-pr retry#3350
reakaleek merged 3 commits into
masterfrom
cursor/a954a8cb

Conversation

@reakaleek

Copy link
Copy Markdown
Member

What

  • Move GitHub commit status updates from a separate teardown step into the build-pr job lifecycle
  • Post pending when build-pr starts and success or failure when it finishes, including on retries

Why

  • When a user retries a failed build-pr step, Buildkite only reruns that step
  • The teardown step already ran and does not rerun, so GitHub keeps the old failure status even when the build passes

Notes

  • The early build-pr-setup step still posts pending while the GCP agent starts
  • Status posting failures do not fail the build

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@reakaleek
reakaleek requested a review from a team as a code owner August 7, 2026 10:25
@reakaleek
reakaleek requested a review from cotti August 7, 2026 10:25
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@reakaleek
reakaleek requested a review from theletterf August 7, 2026 10:25
@reakaleek
reakaleek enabled auto-merge (squash) August 7, 2026 10:43
@reakaleek
reakaleek merged commit 18fd3ca into master Aug 7, 2026
2 of 3 checks passed
@reakaleek
reakaleek deleted the cursor/a954a8cb branch August 7, 2026 11:03
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