Skip to content

chore: Update loadgen service's Docker image to Go 1.24.#432

Open
i-adarsh wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
i-adarsh:main
Open

chore: Update loadgen service's Docker image to Go 1.24.#432
i-adarsh wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
i-adarsh:main

Conversation

@i-adarsh
Copy link
Member

@i-adarsh i-adarsh commented Mar 3, 2026

Summary

The loadgen service in examples/instrumentation-quickstart/docker-compose.yaml fails to start because it uses the golang:1.21 image, but the command attempts to run github.com/rakyll/hey@latest. The latest version of hey (v0.1.5) requires Go 1.24 or higher.

Steps to Reproduce

  1. Clone the repository: git clone https://github.com/GoogleCloudPlatform/opentelemetry-operations-java.git
  2. Navigate to the example: cd opentelemetry-operations-java/examples/instrumentation-quickstart
  3. Run the example: docker compose up

Expected Behavior

The loadgen container should start successfully and generate load against the application.

Actual Behavior

The loadgen container exits immediately with the following error:

loadgen-1  | go: downloading github.com/rakyll/hey v0.1.5
loadgen-1  | go: github.com/rakyll/hey@latest: github.com/rakyll/hey@v0.1.5 requires go >= 1.24.0 (running go 1.21.13; GOTOOLCHAIN=local)

This causes the entire compose stack to shut down if --abort-on-container-exit is used (as recommended in the README).

Environment

  • Repository: GoogleCloudPlatform/opentelemetry-operations-java
  • Path: examples/instrumentation-quickstart
  • File: docker-compose.yaml
  • Docker Image: golang:1.21 (Current)

Proposed Fix

Update the loadgen service in docker-compose.yaml to use a newer Go image, such as golang:1.24 or golang:latest.

[
"go",
"run",
"github.com/rakyll/hey@latest",
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should pin a version here to avoid this in the future.

@psx95 wdyt?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah +1 to pinning the version.

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.

3 participants