Skip to content

test: Add less restrictive gem spec for mongo gem#1537

Merged
arielvalentin merged 5 commits intoopen-telemetry:mainfrom
donoghuc:jruby-10
May 27, 2025
Merged

test: Add less restrictive gem spec for mongo gem#1537
arielvalentin merged 5 commits intoopen-telemetry:mainfrom
donoghuc:jruby-10

Conversation

@donoghuc
Copy link
Copy Markdown
Contributor

It looks like latest features (including jruby 10 enablement) will require the latest minor release of the mongo gem.

Specifically:

The latest mongo gem relaxes the range for the ruby-bson gem. This would allow the uptake of mongodb/bson-ruby#353 if it is accepted.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented May 15, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: arielvalentin / name: Ariel Valentin (8e74715, 68b7144)
  • ✅ login: donoghuc / name: Cas Donoghue (fc8d3a1, 56e0cf8)
  • ✅ login: kaylareopelle / name: Kayla Reopelle (00ed560)

@donoghuc donoghuc mentioned this pull request May 15, 2025
Comment thread instrumentation/mongo/Appraisals Outdated
donoghuc added 2 commits May 15, 2025 17:40
It looks like latest features (including jruby 10 enablement) will require the
latest minor release of the mongo gem.

Specifically:

The latest mongo gem relaxes the range for the ruby-bson gem. This would allow
the uptake of mongodb/bson-ruby#353 if it is accepted.
This commit updates the test expectation for an updated message string. The same
behavior is observbed, the message has just been updated.

This commit also updates construction of a test mongodb client with a newly
required value for auth_source when building a simple client.
@kaylareopelle kaylareopelle changed the title Add less restrictive gem spec for mongo gem test: Add less restrictive gem spec for mongo gem May 16, 2025
@arielvalentin arielvalentin enabled auto-merge (squash) May 24, 2025 19:44
@arielvalentin arielvalentin merged commit 47de526 into open-telemetry:main May 27, 2025
62 checks passed
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