Skip to content

[Chore] Wait for registry to fix broken CI - #9978

Merged
weronikaolejniczak merged 2 commits into
elastic:mainfrom
weronikaolejniczak:fix/wait-for-snapshot-registry-before-kibana
Sep 1, 2026
Merged

[Chore] Wait for registry to fix broken CI#9978
weronikaolejniczak merged 2 commits into
elastic:mainfrom
weronikaolejniczak:fix/wait-for-snapshot-registry-before-kibana

Conversation

@weronikaolejniczak

Copy link
Copy Markdown
Contributor

Summary

What: Retry yarn kbn bootstrap up to 5 times (60s apart) when preparing the Kibana integration PR.
Why: Snapshot publish succeeds, then Kibana bootstrap dies on registry lag (Couldn't find any versions for @elastic/eui-theme-borealis@…). No Kibana PR opens.
How: Same step, just retry. Happy path still runs once.

Failing builds

QA instructions for reviewer

Cannot be tested before merging to main.

@weronikaolejniczak
weronikaolejniczak requested a review from a team as a code owner September 1, 2026 07:57
Copilot AI lite review requested due to automatic review settings September 1, 2026 07:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change is localized to CI workflow scripting and implements a straightforward retry mechanism aligned with the documented registry-lag failure mode.

Pull request overview

Updates the Kibana integration “Prepare changes” GitHub Actions workflow to better tolerate transient npm registry lag after publishing EUI snapshot packages, reducing flaky failures when bootstrapping Kibana.

Changes:

  • Wraps yarn kbn bootstrap in a 5-attempt retry loop with a 60s delay between attempts.
  • Enables stricter bash error handling (set -euo pipefail) for the bootstrap/normalize step.
File summaries
File Description
.github/workflows/update_kibana_dependencies__prepare_changes.yml Adds retry logic to the Kibana bootstrap step to avoid failures when newly published snapshot versions aren’t yet visible in the registry.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/update_kibana_dependencies__prepare_changes.yml
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@mgadewoll mgadewoll left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Changes LGTM

@weronikaolejniczak
weronikaolejniczak merged commit 891499b into elastic:main Sep 1, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants