Skip to content

Comments

Safer bootstrapping#76

Merged
paweljw merged 5 commits intomainfrom
pawel/safer-bootstrap
Nov 24, 2025
Merged

Safer bootstrapping#76
paweljw merged 5 commits intomainfrom
pawel/safer-bootstrap

Conversation

@paweljw
Copy link
Member

@paweljw paweljw commented Nov 20, 2025

  • wait for up to 30s during bootstrap for Beyla socket to become available (assume slow start)
  • if it doesn't, start the local stack as-is, write a marker file denoting unsuccessful Beyla bootstrap
  • a change in remote files in telemetry (pending) will look for this marker and try to re-provision Beyla container if the socket becomes available
  • more guardrails around bootstrap:
    • if bootstrap marker is written,
    • but updater isn't running,
    • and bootstrap.sh is ran again,
    • assume bad bootstrap (maybe a bad file was pushed from telemetry); delete markers and try again

Additionally bumps Beyla to 2.7.7 - it's not significant for us, but since I'm cutting a release anyway, might as well.

@paweljw paweljw marked this pull request as ready for review November 20, 2025 11:02
@paweljw paweljw requested a review from curusarn November 20, 2025 11:02
Copy link
Contributor

@curusarn curusarn left a comment

Choose a reason for hiding this comment

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

Thanks!

Intuitively I'd hope Beyla could bootstrap/restart async. But this looks ok.

@paweljw
Copy link
Member Author

paweljw commented Nov 20, 2025

@curusarn I'm still thinking about that route, but it feels more brittle - more authority spread = more timing issues possible. We'll see :)

@paweljw paweljw merged commit 9f53cc6 into main Nov 24, 2025
3 checks passed
@paweljw paweljw deleted the pawel/safer-bootstrap branch November 24, 2025 09:14
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