Skip to content

Conversation

@ferranbt
Copy link
Collaborator

@ferranbt ferranbt commented Dec 16, 2025

Playground had two systems to validate the health of a service.

  • Functions in Go that would run on playground itself, used to determine whether the whole recipe was ready or not.
  • Healthcheck scripts in the docker containers managed by docker-compose to determine dependencies in the docker-compose file.

Both systems evolved independently from each other, first it was 1, and 2 came about when some services had hard dependencies and would not start without the others running. But, in insight they implement the same functions, i.e. to check whether an EL node is generating blocks in sequence and at correct block time intervals.
This PR eliminates 1 and integrates it inside 2.

@ferranbt ferranbt marked this pull request as ready for review December 17, 2025 11:33
@ferranbt ferranbt requested a review from canercidam as a code owner December 17, 2025 11:33

```bash
$ builder-playground inspect <service> <port>
$ builder-playground debug inspect <service> <port>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I put all the commands that look like debugging utilities under the debug subcommand.

Use: "debug",
}

var probeCmd = &cobra.Command{
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This new command is useful to check the result of a healthcheck probe

}

fmt.Printf("\nWaiting for network to be ready for transactions...\n")
networkReadyStart := time.Now()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This alongside watchdog.go and watchers.go has been removed since now we rely on Docker healthchecks directly. Up to this point this was being used to test only whether a chain was producing blocks, now that function is a healthcheck with healthmon.

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.

1 participant