Skip to content

Update dependencied and required code to allow running it on new Vagrant#273

Open
ingwarsw wants to merge 3 commits into
mitchellh:masterfrom
ingwarsw:update-deps
Open

Update dependencied and required code to allow running it on new Vagrant#273
ingwarsw wants to merge 3 commits into
mitchellh:masterfrom
ingwarsw:update-deps

Conversation

@ingwarsw
Copy link
Copy Markdown
Contributor

@ingwarsw ingwarsw commented Sep 26, 2025

This pull request updates the project to support newer dependencies, modernizes the Ruby version, and introduces a minor feature improvement. The most significant changes are upgrading the fog-google dependency, updating the supported Ruby version, and adding a new default for resource_policies in configuration tests.

Dependency and compatibility updates:

  • Upgraded the fog-google gem dependency to version ~> 1.25.0 in vagrant-google.gemspec to ensure compatibility with newer versions.
  • Updated the Ruby version used in CI and development from 2.7.4 to 3.4, and removed the specific Ruby version label in .github/workflows/ci.yml.

Code and API adjustments for updated dependencies:

  • Updated references to Fog error classes from Fog::Compute::Google::* to Fog::Google::Compute::* in lib/vagrant-google/action/run_instance.rb and lib/vagrant-google/action/start_instance.rb to match the new fog-google gem API. [1] [2]

Feature and test improvements:

  • Added a test to verify that the default value for resource_policies is an empty array in test/unit/common/config_test.rb.

Versioning:

  • Bumped the plugin version from 2.7.0 to 2.8.0 in lib/vagrant-google/version.rb.

@ingwarsw
Copy link
Copy Markdown
Contributor Author

@Temikus Im not sure if your still maintainer here but would be cool to merge this and release new version of the plugin.

@ShubhamRasal
Copy link
Copy Markdown

@Temikus Im not sure if your still maintainer here but would be cool to merge this and release new version of the plugin.

yeah.. it would be really helpful

@doniz
Copy link
Copy Markdown

doniz commented Feb 18, 2026

Hi, any stops to merging this? I have tested this PR and works fine

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