Skip to content

ci: remove vagrant dependency workaround#6925

Merged
crazy-max merged 1 commit into
moby:masterfrom
crazy-max:fix-vagrant-install
Jul 3, 2026
Merged

ci: remove vagrant dependency workaround#6925
crazy-max merged 1 commit into
moby:masterfrom
crazy-max:fix-vagrant-install

Conversation

@crazy-max

@crazy-max crazy-max commented Jul 3, 2026

Copy link
Copy Markdown
Member

relates to: https://github.com/moby/buildkit/actions/runs/28630119054/job/84905246097?pr=6920#step:5:1356

Installing the 'vagrant-libvirt' plugin. This can take a few minutes...
/opt/vagrant/embedded/lib/ruby/3.3.0/rubygems/specification.rb:2245:in `raise_if_conflicts': Unable to activate fog-json-1.4.0, because json-2.7.2 conflicts with json (~> 2.19) (Gem::ConflictError)
	from /opt/vagrant/embedded/lib/ruby/3.3.0/rubygems/specification.rb:1383:in `activate'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/bundler.rb:771:in `block in activate_solution'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/bundler.rb:768:in `each'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/bundler.rb:768:in `activate_solution'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/bundler.rb:608:in `internal_install'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/bundler.rb:358:in `install'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/plugin/manager.rb:141:in `block in install_plugin'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/plugin/manager.rb:151:in `install_plugin'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/plugins/commands/plugin/action/install_gem.rb:33:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/action/warden.rb:38:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/action/builder.rb:183:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/action/runner.rb:104:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/util/busy.rb:22:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/action/runner.rb:104:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/plugins/commands/plugin/command/base.rb:17:in `action'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/plugins/commands/plugin/command/install.rb:73:in `block in execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/plugins/commands/plugin/command/install.rb:72:in `each'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/plugins/commands/plugin/command/install.rb:72:in `execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/plugins/commands/plugin/command/root.rb:69:in `execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/cli.rb:67:in `execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/lib/vagrant/environment.rb:317:in `cli'
	from /opt/vagrant/embedded/gems/gems/vagrant-2.4.9/bin/vagrant:238:in `<main>'

The previous workaround was added for an older Vagrant resolver issue, but the current failure happens while installing vagrant-libvirt with the latest HashiCorp package. Removing the workaround lets Vagrant use its normal dependency enforcement again, which should avoid allowing an incompatible plugin dependency solution.

@crazy-max crazy-max force-pushed the fix-vagrant-install branch from 2ae0875 to 1fa974e Compare July 3, 2026 08:48
@crazy-max crazy-max changed the title ci: use ubuntu vagrant and vagrant-libvirt packages ci: pin vagrant for freebsd job Jul 3, 2026
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max crazy-max force-pushed the fix-vagrant-install branch from 1fa974e to 719caf3 Compare July 3, 2026 08:56
@crazy-max crazy-max changed the title ci: pin vagrant for freebsd job ci: remove vagrant dependency workaround Jul 3, 2026
@crazy-max crazy-max requested review from thaJeztah and tonistiigi July 3, 2026 09:01
@crazy-max crazy-max marked this pull request as ready for review July 3, 2026 09:01

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@crazy-max crazy-max merged commit 6608791 into moby:master Jul 3, 2026
192 checks passed
@crazy-max crazy-max deleted the fix-vagrant-install branch July 3, 2026 13:47
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