Skip to content

(maint) Constrain orchestrator_client to < 0.7.1#528

Merged
jhbuchanan45 merged 4 commits into
mainfrom
maint/fix_orchestrator_client
Nov 26, 2024
Merged

(maint) Constrain orchestrator_client to < 0.7.1#528
jhbuchanan45 merged 4 commits into
mainfrom
maint/fix_orchestrator_client

Conversation

@jhbuchanan45

@jhbuchanan45 jhbuchanan45 commented Nov 25, 2024

Copy link
Copy Markdown
Contributor

Summary

Based on puppetlabs/puppetlabs-puppet_agent#743 to fix an issue with the version 0.7.1 of orchestrator_client

"The 0.7.1 release of orchestrator_client broadened its dependency on Faraday (see puppetlabs/orchestrator_client-ruby@6f8661e). This change has caused Bundler to resolve Faraday in unexpected ways when there are other Gems with dependencies on Faraday.

This commit constrains orchestrator_client to < 0.7.1 until this issue is resolved."

Additional Context

Add any additional context about the problem here.

Related Issues (if any)

ruby/rubygems#8286

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.

Changes include test coverage?

  • Yes
  • Not needed

Have you updated the documentation?

  • Yes, I've updated the appropriate docs
  • Not needed

…t 0.7.1 by pinning to an old version until fixed
@jhbuchanan45
jhbuchanan45 requested review from a team as code owners November 25, 2024 10:25
@jhbuchanan45
jhbuchanan45 merged commit 5d12b3a into main Nov 26, 2024
@jhbuchanan45
jhbuchanan45 deleted the maint/fix_orchestrator_client branch November 26, 2024 11:50
davidmalloncares pushed a commit that referenced this pull request Nov 19, 2025
* (maint) Fix weird bundler resolution of Faraday in orchestrator client 0.7.1 by pinning to an old version until fixed

* Apply Yamlfix format changes.

* (maint) Bump ruby_task_helper to version 1.0.0 to fix CI error and support puppet 8

---------

Co-authored-by: jhbuchanan45 <jhbuchanan45@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants