Skip to content

chore: bump ruby/setup-ruby from 1.165.1 to 1.176.0#1635

Merged
robertlaurin merged 2 commits into
open-telemetry:mainfrom
robbkidd:fix-macos-in-ci
May 8, 2024
Merged

chore: bump ruby/setup-ruby from 1.165.1 to 1.176.0#1635
robertlaurin merged 2 commits into
open-telemetry:mainfrom
robbkidd:fix-macos-in-ci

Conversation

@robbkidd

@robbkidd robbkidd commented May 8, 2024

Copy link
Copy Markdown
Member
  • ruby-setup changes in GitHub Actions

To fix an error[1] on macos runners resolved in v1.171.0[2]:

The current runner (macos-14-arm64) was detected as self-hosted
because the platform does not match a GitHub-hosted runner image
(or that image is deprecated and no longer supported).
  • opentelemetry-common

"require 'uri'" was added to utilities.rb in opentelemetry-common to resolve a "NameError: uninitialized constant OpenTelemetry::Common::Utilities::URI" error that appeared in the Ruby 3.3 test. This error started appearing when setup-ruby bumped from v1.174.0 to v1.175.0[3] which moved the latest Ruby 3.3 from 3.3.0 to 3.3.1. The URI gem appears to have been vendored into Ruby 3.3.1[4].

[1] ruby/setup-ruby#568
[2] ruby/setup-ruby#567
[3] ruby/setup-ruby@v1.174.0...v1.175.0
[4] ruby/ruby@d64d0b5

To fix an error[1] on macos runners resolved in v1.171.0[2]:

    The current runner (macos-14-arm64) was detected as self-hosted
    because the platform does not match a GitHub-hosted runner image
    (or that imageis deprecated and no longer supported).

[1] ruby/setup-ruby#568
[2] ruby/setup-ruby#567
@robbkidd

robbkidd commented May 8, 2024

Copy link
Copy Markdown
Member Author

@robbkidd robbkidd added the ci label May 8, 2024

@robertlaurin robertlaurin 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.

Thanks Robb!

@robertlaurin robertlaurin merged commit 7162e34 into open-telemetry:main May 8, 2024
@robbkidd robbkidd deleted the fix-macos-in-ci branch May 8, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants