Skip to content

Conversation

tdn21
Copy link
Contributor

@tdn21 tdn21 commented Aug 14, 2025

Description

What

  • Updated .github/workflows/benchmarks.yml to use the oracle-bare-metal-64cpu-512gb-x86-64 runner instead of the Equinix bare metal/self-hosted pool.

Why

The benchmarks workflow has been failing (timing out) since workflow run #435 for the past two weeks with the following error.

The job has exceeded the maximum execution time while awaiting a runner for 24h0m0s

Reason:
Equinix Metal is being sunset, and the current Equinix bare metal runners are no longer available. To restore functionality, we’re moving benchmarks execution to the Oracle bare metal runner pool.

References:

Open Question

For now, I’ve updated the runner. Should we switch this job to a container-based workflow, or continue running directly on the host? (otel-rust ref)

Fixes # No open issue yet (Please let me know if one is required)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

NA

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated - NA
  • Unit tests have been added - NA
  • Documentation has been updated - NA

@tdn21 tdn21 requested a review from a team as a code owner August 14, 2025 09:57
Copy link
Member

@emdneto emdneto left a comment

Choose a reason for hiding this comment

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

otel-java and otel-rust are also using this runner.

Copy link
Member

@aabmass aabmass left a comment

Choose a reason for hiding this comment

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

Thank you!

@aabmass aabmass added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 14, 2025
@aabmass aabmass enabled auto-merge (squash) August 14, 2025 15:05
@aabmass aabmass merged commit b1cf152 into open-telemetry:main Aug 14, 2025
466 of 468 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants