vagrant-vbguest 0.24.0 → 0.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 24a98ba4b8c7a2b83a47eabdcf9f61336b88b4873bf3600164d31515ec1b5b2c
-  data.tar.gz: e638b52785a1f5d7718840255a0c2df6f1b63d6e3af95a8edf1e1f6c1f1dce28
+  metadata.gz: 5b06593adf78b594ec9a8cd373156ce66b65c2c9636d5ce18b491031b1f42cea
+  data.tar.gz: 801da70b4ae593718792ea976f43f7cc3a5feaa16d342680a33d1cf9f3fe406d
 SHA512:
-  metadata.gz: b7576c6a63d66c27ecdfcb1ea7a13d1ceb3decb8ac82a91063f8d22e93222f1a11d80876f26efe6c7807d2fc5024872810e0aaf033859714c3a264d465d78ce2
-  data.tar.gz: b840b1d1bd9f069f3160089a47159722ff493b577760aa802e4618cb4859b3167ea72a8d91393bf55812cfdf8befdcab8f0b2741fe3f7d1a29df65ff65f62ec2
+  metadata.gz: ccebf9ed8b4ded67948a85d08fdf69021c46b9aff7dbd12d025a1516f4858f06b2102537efd484e715c5893e38d5fb4242cb123496abdd6ea7d2bb1b3ee8e81f
+  data.tar.gz: c872bcdb579253435983228ceb7cefe68aa7af4988a99d451b5ead39939e03061c2b6358b0b0919d23973ab4ef412e3aca868f8fd5e38ba320d0ca4c16e6f2e5

data/.github/dependabot.yml

@@ -0,0 +1,13 @@
+# To get started with Dependabot version updates, you'll need to specify which
+# package ecosystems to update and where the package manifests are located.
+# Please see the documentation for all configuration options:
+# https://help.github.com/github/administering-a-repository/configuration-options-for-dependency-updates
+
+version: 2
+updates:
+  - package-ecosystem: "bundler" # See documentation for possible values
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "daily"
+    ignore:
+      - dependency-name: "vagrant"

data/CHANGELOG.md

@@ -1,3 +1,32 @@
+## 0.29.0 (2021-01-03)
+
+- Add installer_option :enablerepo for CentOS. [GH-402], Fixes [GH-398]
+- Add installer_option :running_kernel_modules for all Linux installers. [GH-403], Fixes [GH-396]
+
+## 0.28.0 (2020-11-20)
+
+- Work around a bug in vagrant 2.2.11 to 2.2.13, which would not load the vbguest middleware. [GH-394], Fixes [GH-391]
+
+## 0.27.0 (2020-11-15)
+
+- Un-mounting the guest additions iso will now take place in the cleanup step [GH-393]
+- Running detection now checks for vboxguest and vboxsf kernel module. [GH-392], Fixes [GH-387]
+
+## 0.26.0 (2020-10-30)
+
+- Add a new configuration `installer_hooks` allows to configure arbitrary scripts to be run on the guest before/after installer steps are executed.
+- Fix RHEL8 perl vs. perl-interpreter [GH-368]
+- On RedHat, use `dnf` instead of `yum` if available. [GH-368]
+- Fix incorrect re-use of installer_* options.
+
+## 0.25.0 (2020-09-26)
+
+- Renames mainline branch from "master" to "main"
+- Fix not detecting running GuestAdditions on some systems [GH-347], [GH-376]. Thanks @Morac2 for [GH-377]
+- Fixes package installation issues on CentOS 8.
+  CentOS installers no longe use the `--enablerepo` parameter when running `yum` to install packages.
+  Thanks @ghoneycutt for [GH-384]
+
 ## 0.24.0 (2020-04-29)
 
 No code changes to 0.24.0.beta1
@@ -80,7 +109,7 @@ See discussion in [GH-300]. Thanks @cbj4074 for asking the right questions.
 
 ## 0.15.2 (2018-05-22)
 
-- On RedHad based guest, the installer will now try to install the 'kernel-devel' package additionally to 'kernel-devel-`uname -r`' Thanks @ghoneycutt [GH-299]
+- On RedHat based guest, the installer will now try to install the 'kernel-devel' package additionally to 'kernel-devel-`uname -r`' Thanks @ghoneycutt [GH-299]
 
 ## 0.15.1 (2018-01-08)
 

data/CODE_OF_CONDUCT.md

@@ -0,0 +1,76 @@
+# Contributor Covenant Code of Conduct
+
+## Our Pledge
+
+In the interest of fostering an open and welcoming environment, we as
+contributors and maintainers pledge to making participation in our project and
+our community a harassment-free experience for everyone, regardless of age, body
+size, disability, ethnicity, sex characteristics, gender identity and expression,
+level of experience, education, socio-economic status, nationality, personal
+appearance, race, religion, or sexual identity and orientation.
+
+## Our Standards
+
+Examples of behavior that contributes to creating a positive environment
+include:
+
+* Using welcoming and inclusive language
+* Being respectful of differing viewpoints and experiences
+* Gracefully accepting constructive criticism
+* Focusing on what is best for the community
+* Showing empathy towards other community members
+
+Examples of unacceptable behavior by participants include:
+
+* The use of sexualized language or imagery and unwelcome sexual attention or
+ advances
+* Trolling, insulting/derogatory comments, and personal or political attacks
+* Public or private harassment
+* Publishing others' private information, such as a physical or electronic
+ address, without explicit permission
+* Other conduct which could reasonably be considered inappropriate in a
+ professional setting
+
+## Our Responsibilities
+
+Project maintainers are responsible for clarifying the standards of acceptable
+behavior and are expected to take appropriate and fair corrective action in
+response to any instances of unacceptable behavior.
+
+Project maintainers have the right and responsibility to remove, edit, or
+reject comments, commits, code, wiki edits, issues, and other contributions
+that are not aligned to this Code of Conduct, or to ban temporarily or
+permanently any contributor for other behaviors that they deem inappropriate,
+threatening, offensive, or harmful.
+
+## Scope
+
+This Code of Conduct applies both within project spaces and in public spaces
+when an individual is representing the project or its community. Examples of
+representing a project or community include using an official project e-mail
+address, posting via an official social media account, or acting as an appointed
+representative at an online or offline event. Representation of a project may be
+further defined and clarified by project maintainers.
+
+## Enforcement
+
+Instances of abusive, harassing, or otherwise unacceptable behavior may be
+reported by contacting the project team at info@dotless.de. All
+complaints will be reviewed and investigated and will result in a response that
+is deemed necessary and appropriate to the circumstances. The project team is
+obligated to maintain confidentiality with regard to the reporter of an incident.
+Further details of specific enforcement policies may be posted separately.
+
+Project maintainers who do not follow or enforce the Code of Conduct in good
+faith may face temporary or permanent repercussions as determined by other
+members of the project's leadership.
+
+## Attribution
+
+This Code of Conduct is adapted from the [Contributor Covenant][homepage], version 1.4,
+available at https://www.contributor-covenant.org/version/1/4/code-of-conduct.html
+
+[homepage]: https://www.contributor-covenant.org
+
+For answers to common questions about this code of conduct, see
+https://www.contributor-covenant.org/faq

data/Gemfile

@@ -4,16 +4,16 @@ group :development do
   # We depend on Vagrant for development, but we don't add it as a
   # gem dependency because we expect to be installed within the
   # Vagrant environment itself using `vagrant plugin`.
-  if ENV["VAGRANT_VERSION"]
-    gem "vagrant", :git => "https://github.com/hashicorp/vagrant.git", tag: "v#{ENV["VAGRANT_VERSION"]}"
-  elsif ENV["VAGRANT_LOCAL"]
-    gem "vagrant", path: ENV["VAGRANT_LOCAL"]
+  if ENV['VAGRANT_VERSION']
+    gem 'vagrant', :git => 'https://github.com/hashicorp/vagrant.git', tag: "v#{ENV['VAGRANT_VERSION']}"
+  elsif ENV['VAGRANT_LOCAL']
+    gem 'vagrant', path: ENV['VAGRANT_LOCAL']
   else
-    gem "vagrant", :git => "https://github.com/hashicorp/vagrant.git"
+    gem 'vagrant', :git => 'https://github.com/hashicorp/vagrant.git', branch: "main"
   end
-  gem "rake", ">= 12.3.3"
+  gem 'rake', '>= 12.3.3'
 end
 
 group :plugins do
-  gem "vagrant-vbguest", path: "."
+  gem 'vagrant-vbguest', path: '.'
 end

data/Readme.md

@@ -4,7 +4,7 @@
 
 [![Join the chat at https://gitter.im/dotless-de/vagrant-vbguest](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/dotless-de/vagrant-vbguest?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 [![Code Climate](https://codeclimate.com/github/dotless-de/vagrant-vbguest.svg)](https://codeclimate.com/github/dotless-de/vagrant-vbguest)
-[![Inline docs](http://inch-ci.org/github/dotless-de/vagrant-vbguest.svg?branch=master)](http://inch-ci.org/github/dotless-de/vagrant-vbguest)
+[![Inline docs](http://inch-ci.org/github/dotless-de/vagrant-vbguest.svg?branch=main)](http://inch-ci.org/github/dotless-de/vagrant-vbguest)
 [![Gem Version](https://badge.fury.io/rb/vagrant-vbguest.svg)](https://badge.fury.io/rb/vagrant-vbguest)
 
 
@@ -59,6 +59,7 @@ vbguest will try to autodetect the best option for your system. WTF? see below.
 * `installer` (`VagrantVbguest::Installers::Base`, optional) : Reference to a (custom) installer class
 * `installer_arguments` (Array, default: `['--nox11']`) : List of additional arguments to pass to the installer. eg: `%w{--nox11 --force}` would execute `VBoxLinuxAdditions.run install --nox11 --force`
 * `installer_options` (Hash, default: `{}`) : Configure how a Installer internally works. Should be set on a `vm` level.
+* `installer_hooks` (Hash, default: `{}`) : Configure scripts to be run before/after installer steps.
 * `yes` (Boolean or String, default: `true`): Wheter to pipe `yes` to the installer. If `true`, executes `yes | VBoxLinuxAdditions.run install`. With `false`, the command is executed without `yes`. You can also put in a string here for `yes` (e.g. `no` to refuse all messages)
 
 
@@ -97,12 +98,21 @@ Vagrant.configure("2") do |config|
   end
 end
 ```
+##### Linux
 
+This option is available on all Linux type installers.
+
+* `:running_kernel_modules` (default: `["vboxguest", "vboxsf"]`) The list used to check for the "running" state. Each of these modules need to appear at the beginning of a line in `/proc/modules`.
 
 ##### CentOS
 
 * `:allow_kernel_upgrade` (default: `false`): If `true`, instead of trying to find matching the matching kernel-devel package to the installed kernel version, the kernel will be updated and the (now matching) up-to-date kernel-devel will be installed. __NOTE__: This will trigger a reboot of the box.
 * `:reboot_timeout` (default: `300`): Maximum number of seconds to wait for the box to reboot after a kernel upgrade.
+* `:enablerepo`: (default: `false`) Configure `yum --enablerepo` settings. Possible values are:
+  - Single String: `"C*-base"` converts to `--enablerepo=C*-base`
+  - Array of Strings: `['C*-base', 'C*-updates']` converts to `--enablerepo=C*-base --enablerepo=C*-updates`
+  - `true`: Enables `C${release_version}-base` and `C${release_version}-updates` with release_version read from `/etc/centos-release` if the repo exists, otherwise `*`.
+  - `false`: Does not set any `--enablerepo`
 
 #### Global Configuration
 
@@ -128,6 +138,26 @@ VagrantVbguest::Config.auto_update = false
 Settings in a project's `Vagrantfile` will overwrite those setting. When executed as a command, command line arguments will overwrite all of the above.
 
 
+#### Installer Hooks (`installer_hooks`)
+
+Additionally to the build-in `installer_options`, you can configure to execute scripts around the install steps `install`, `rebuild` and `start`.
+Accepts either a single command or and array of commands. Box-specific settings overwrite earlier settings, just like with `installer_options`.
+
+Use this make changes to the guest, for example to install specific dependencies or tweak the network setup.
+
+* `before_install`/`after_install`: Runs before/after the install step. That is before uploading the iso file into the guest and after unmounting the iso file.
+* `before_rebuild`/`after_rebuild`: Runs before/after the installer runs a command to let the GuestAdditions rebuild itself.
+* `before_start`/`after_start`: Runs before/after the installer runs a command to start the GuestAdditions service.
+
+```ruby
+Vagrant.configure("2") do |config|
+  config.vm.define "my_cent_os_box" do |c|
+    c.vbguest.installer_hooks[:before_install] = ["yum install -y epel-release", "sleep 1"]
+  end
+end
+```
+
+
 ### Running as a middleware
 
 Running as a middleware is the default way of using *vagrant-vbguest*. 
@@ -302,14 +332,14 @@ end
 
 If you find yourself copying the same installer in each of your vagrant project, it might be a good idea to make it a plugin itself. Like vagrant-vbguest itself, installers can be [distributed as ruby gems](http://guides.rubygems.org/publishing/)
 
-This project contains a [sample installer gem](https://github.com/dotless-de/vagrant-vbguest/tree/master/testdrive/vagrant-vbguest-unikorn) which might serve as an boilerplate. 
+This project contains a [sample installer gem](https://github.com/dotless-de/vagrant-vbguest/tree/main/testdrive/vagrant-vbguest-unikorn) which might serve as an boilerplate. 
 
 
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/dotless-de/vagrant-vbguest.
 
-For the forseeable future, no more pull requests will be accepted which add new guest support/installers. Please build your gem as a plugin to vbguest (see "Very Advanced Usage" section of this file). In fact, please open a pull request to remove support for a guest system if feel confident that your solution is better.
+For the foreseeable future, no more pull requests will be accepted which add new guest support/installers. Please build your gem as a plugin to vbguest (see "Very Advanced Usage" section of this file). In fact, please open a pull request to remove support for a guest system if feel confident that your solution is better.
 
 
 ## Known Issues

data/SECURITY.md

@@ -0,0 +1,10 @@
+# Security Policy
+
+## Supported Versions
+
+Only the latest version of vagrant-vbguest is supported and will receive security updates.
+
+## Reporting a Vulnerability
+
+Download GPG key from: https://keybase.io/fnordfish/pgp_keys.asc (fingerprint `B3CA4DF4D703CBDFFF211CB8A8B249A38963D0B6`)
+Write an gpg encrypted email to `vbguest-security@dotless.de`

data/VERSION

@@ -1 +1 @@
-0.24.0
+0.29.0

data/lib/vagrant-vbguest.rb

@@ -31,8 +31,6 @@ require 'vagrant-vbguest/installers/suse'
 require 'vagrant-vbguest/installers/archlinux'
 require 'vagrant-vbguest/installers/windows'
 
-require 'vagrant-vbguest/middleware'
-
 module VagrantVbguest
 
   class Plugin < Vagrant.plugin("2")
@@ -52,13 +50,26 @@ module VagrantVbguest
       Command
     end
 
-    # hook after anything that boots:
-    # that's all middlewares which will run the buildin "VM::Boot" action
-    action_hook('vbguest') do |hook|
-      if defined?(VagrantPlugins::ProviderVirtualBox::Action::CheckGuestAdditions)
-        hook.before(VagrantPlugins::ProviderVirtualBox::Action::CheckGuestAdditions, VagrantVbguest::Middleware)
-      else
-        hook.after(VagrantPlugins::ProviderVirtualBox::Action::Boot, VagrantVbguest::Middleware)
+    if ("2.2.11".."2.2.13").include?(Gem::Version.create(Vagrant::VERSION).release.to_s)
+      $stderr.puts(
+        "Vagrant v#{Vagrant::VERSION} has a bug which prevents vagrant-vbguest to register probably.\n" \
+        "vagrant-vbguest will try to do the next best thing, but might not be working as expected.\n" \
+        "If possible, please upgrade Vagrant to >= 2.2.14"
+      )
+      action_hook('vbguest', :machine_action_up) do |hook|
+        require 'vagrant-vbguest/middleware'
+        hook.append(VagrantVbguest::Middleware)
+      end
+    else
+      # hook after anything that boots:
+      # that's all middlewares which will run the buildin "VM::Boot" action
+      action_hook('vbguest') do |hook|
+        require 'vagrant-vbguest/middleware'
+        if defined?(VagrantPlugins::ProviderVirtualBox::Action::CheckGuestAdditions)
+          hook.before(VagrantPlugins::ProviderVirtualBox::Action::CheckGuestAdditions, VagrantVbguest::Middleware)
+        else
+          hook.after(VagrantPlugins::ProviderVirtualBox::Action::Boot, VagrantVbguest::Middleware)
+        end
       end
     end
   end

data/lib/vagrant-vbguest/config.rb

@@ -5,7 +5,7 @@ module VagrantVbguest
     module Attributes
       attr_accessor :auto_update, :auto_reboot, :no_install, :no_remote,
                     :installer, :installer_arguments, :installer_options,
-                    :allow_downgrade,
+                    :installer_hooks, :allow_downgrade,
                     :iso_path, :iso_upload_path, :iso_mount_point, :yes
     end
 
@@ -18,6 +18,7 @@ module VagrantVbguest
       def no_remote;   @no_remote.nil?   ? false : @no_remote   end
       def installer_arguments; @installer_arguments.nil? ? '--nox11' : @installer_arguments end
       def installer_options; @installer_options ||= {} end
+      def installer_hooks; @installer_hooks ||= {} end
       def yes; @yes.nil? ? true : @yes end
 
       def iso_path
@@ -34,8 +35,9 @@ module VagrantVbguest
     def auto_reboot; @auto_reboot.nil? ? self.class.auto_reboot : @auto_reboot end
     def no_install;  @no_install.nil?  ? self.class.no_install  : @no_install  end
     def no_remote;   @no_remote.nil?   ? self.class.no_remote   : @no_remote   end
-    def installer_arguments; @installer_arguments.nil? ? self.class.installer_arguments : @installer_arguments end
-    def installer_options; @installer_options ||= self.class.installer_options end
+    def installer_arguments; @installer_arguments.nil? ? self.class.installer_arguments.dup : @installer_arguments end
+    def installer_options; @installer_options ||= self.class.installer_options.dup end
+    def installer_hooks; @installer_hooks ||= self.class.installer_hooks.dup end
     def yes; @yes.nil? ? self.class.yes : @yes end
 
     def iso_path
@@ -51,6 +53,7 @@ module VagrantVbguest
         :installer => installer,
         :installer_arguments => installer_arguments,
         :installer_options => installer_options,
+        :installer_hooks => installer_hooks,
         :iso_path => iso_path,
         :iso_upload_path => iso_upload_path,
         :iso_mount_point => iso_mount_point,

data/lib/vagrant-vbguest/installer.rb

@@ -60,8 +60,10 @@ module VagrantVbguest
       installer = guest_installer
       raise NoInstallerFoundError, :method => 'install' if !installer
 
-      installer.install do |type, data|
-        @env.ui.info(data, :prefix => false, :new_line => false)
+      with_hooks(:install) do
+        installer.install do |type, data|
+          @env.ui.info(data, :prefix => false, :new_line => false)
+        end
       end
     ensure
       cleanup
@@ -71,8 +73,10 @@ module VagrantVbguest
       installer = guest_installer
       raise NoInstallerFoundError, :method => 'rebuild' if !installer
 
-      installer.rebuild do |type, data|
-        @env.ui.info(data, :prefix => false, :new_line => false)
+      with_hooks(:rebuild) do
+        installer.rebuild do |type, data|
+          @env.ui.info(data, :prefix => false, :new_line => false)
+        end
       end
     end
 
@@ -80,8 +84,10 @@ module VagrantVbguest
       installer = guest_installer
       raise NoInstallerFoundError, :method => 'manual start' if !installer
 
-      installer.start do |type, data|
-        @env.ui.info(data, :prefix => false, :new_line => false)
+      with_hooks(:start) do
+        installer.start do |type, data|
+          @env.ui.info(data, :prefix => false, :new_line => false)
+        end
       end
     end
 
@@ -145,8 +151,32 @@ module VagrantVbguest
     end
 
     def cleanup
-      @guest_installer.cleanup if @guest_installer
+      return unless @guest_installer
+
+      @guest_installer.cleanup do |type, data|
+        @env.ui.info(data, :prefix => false, :new_line => false)
+      end
     end
 
+    def with_hooks(step_name)
+      run_hook(:"before_#{step_name}")
+      ret = yield
+      run_hook(:"after_#{step_name}")
+
+      ret
+    end
+
+    def run_hook(hook_name)
+      hooks = @options.dig(:installer_hooks, hook_name)
+
+      return if !hooks || hooks.empty?
+
+      @vm.ui.info I18n.t("vagrant_vbguest.installer_hooks.#{hook_name}")
+      Array(hooks).each do |l|
+        @vm.communicate.sudo(l) do |type, data|
+          @env.ui.info(data, :prefix => false, :new_line => false)
+        end
+      end
+    end
   end
 end

data/lib/vagrant-vbguest/installers/base.rb

@@ -236,12 +236,13 @@ module VagrantVbguest
 
       # A helper method to delete the uploaded GuestAdditions iso file
       # from the guest box
-      def cleanup
+      def cleanup(opts, &block)
         unless options[:no_cleanup]
           @host.cleanup
-          communicate.execute("test -f #{tmp_path} && rm #{tmp_path}", :error_check => false) do |type, data|
-            env.ui.error(data.chomp, :prefix => false)
-          end
+
+          opts = (opts || {}).merge(:error_check => false)
+          block ||= proc { |type, data| env.ui.error(data.chomp, :prefix => false) }
+          communicate.execute("test -f #{tmp_path} && rm #{tmp_path}", opts, &block)
         end
       end
     end

data/lib/vagrant-vbguest/installers/centos.rb

@@ -29,6 +29,26 @@ module VagrantVbguest
         installer_options[:allow_kernel_upgrade]
       end
 
+      # parse the installer_option :enablerepo
+      def enablerepo
+        unless instance_variable_defined?(:@enablerepo)
+          enablerepo = installer_options.fetch(:enablerepo, false)
+          @enablerepo =
+            case enablerepo
+            when String
+              "--enablerepo=#{enablerepo}"
+            when Array
+              enablerepo.map { |e| "--enablerepo=#{e}" }.join(" ")
+            when true
+              rel = has_rel_repo? ? release_version : '*'
+              "--enablerepo=C#{rel}-base --enablerepo=C#{rel}-updates"
+            else
+              nil
+            end
+        end
+        @enablerepo
+      end
+
       def install_kernel_deps(opts=nil, &block)
         unless has_kernel_devel_info?
           update_release_repos(opts, &block)
@@ -68,8 +88,7 @@ module VagrantVbguest
       end
 
       def install_kernel_devel(opts=nil, &block)
-        rel = has_rel_repo? ? release_version : '*'
-        cmd = "yum install -y kernel-devel-`uname -r` --enablerepo=C#{rel}-base --enablerepo=C#{rel}-updates"
+        cmd = "yum install -y kernel-devel-`uname -r` #{enablerepo}"
         communicate.sudo(cmd, opts, &block)
       end
 

data/lib/vagrant-vbguest/installers/linux.rb

@@ -67,7 +67,6 @@ module VagrantVbguest
         mount_iso(opts, &block)
         execute_installer(opts, &block)
         start(opts, &block)
-        unmount_iso(opts, &block) unless options[:no_cleanup]
       end
 
       # @param opts [Hash] Optional options Hash wich meight get passed to {Vagrant::Communication::SSH#execute} and firends
@@ -75,10 +74,20 @@ module VagrantVbguest
       # @yieldparam [String] type Type of the output, `:stdout`, `:stderr`, etc.
       # @yieldparam [String] data Data for the given output.
       def running?(opts=nil, &block)
-        opts = {
-          :sudo => true
-        }.merge(opts || {})
-        communicate.test('grep -qE "^vboxguest\s.+\s\([^\s]*O[^\s]*\)$" /proc/modules && ls /lib/modules/`uname -r`/misc/vboxsf.ko*', opts, &block)
+        kmods = nil
+        communicate.sudo('cat /proc/modules', opts) do |type, data|
+          block.call(type, data) if block
+          kmods = data if type == :stdout && data
+        end
+
+        unless kmods
+          env.ui.warn "Could not read /proc/modules"
+          return true
+        end
+
+        Array(installer_options[:running_kernel_modules]).all? do |mod|
+          /^#{mod}\b/i.match(kmods)
+        end
       end
 
       # This overrides {VagrantVbguest::Installers::Base#guest_version}
@@ -233,6 +242,12 @@ module VagrantVbguest
         @installer_arguments ||= Array(options[:installer_arguments]).join " "
       end
 
+      def installer_options
+        @installer_options ||= {
+          running_kernel_modules: ["vboxguest", "vboxsf"]
+        }.merge!(super)
+      end
+
       # Determine if yes needs to be called or not
       def yes
         value = @options[:yes]
@@ -269,6 +284,13 @@ module VagrantVbguest
         env.ui.info(I18n.t("vagrant_vbguest.unmounting_iso", :mount_point => mount_point))
         communicate.sudo("umount #{mount_point}", opts, &block)
       end
+
+      def cleanup(opts=nil, &block)
+        unless options[:no_cleanup]
+          unmount_iso(opts, &block)
+          super
+        end
+      end
     end
   end
 end

data/lib/vagrant-vbguest/installers/redhat.rb

@@ -16,7 +16,11 @@ module VagrantVbguest
 
     protected
       def install_dependencies_cmd
-        "yum install -y #{dependencies}"
+        "#{package_manager_cmd} install -y #{dependencies}"
+      end
+
+      def package_manager_cmd
+        "`bash -c 'type -p dnf || type -p yum'`"
       end
 
       def dependencies
@@ -26,11 +30,22 @@ module VagrantVbguest
           'gcc',
           'binutils',
           'make',
-          'perl',
+          perl_dependency,
           'bzip2',
           'elfutils-libelf-devel'
         ].join(' ')
       end
+
+      def perl_dependency
+        unless instance_variable_defined?(:@perl_dependency)
+          @perl_dependency = if communicate.test("#{package_manager_cmd} list perl-interpreter")
+            "perl-interpreter"
+          else
+            "perl"
+          end
+        end
+        @perl_dependency
+      end
     end
   end
 end

data/lib/vagrant-vbguest/installers/windows.rb

@@ -51,7 +51,6 @@ module VagrantVbguest
         upload(iso_file)
         mount_iso(opts, &block)
         execute_installer(opts, &block)
-        unmount_iso(opts, &block) unless options[:no_cleanup]
       end
 
       def reboot_after_install?(opts = nil, &block)
@@ -113,6 +112,13 @@ module VagrantVbguest
         communicate.execute("Remove-Item -Path #{tmp_path}", opts, &block)
       end
 
+      def cleanup(opts = nil, &block)
+        unless options[:no_cleanup]
+          unmount_iso(opts, &block)
+          super
+        end
+      end
+
       def yield_installation_error_warning(path_to_installer)
         @env.ui.warn I18n.t(
           "vagrant_vbguest.windows.install_error",

data/locales/en.yml

@@ -34,6 +34,14 @@ en:
       ok: "[%{vm_name}] GuestAdditions %{guest_version} running --- OK."
       skipped_downgrade: "GuestAdditions are newer than your host but, downgrades are disabled. Skipping."
 
+    installer_hooks:
+      before_install: "Executing pre-install hooks"
+      after_install: "Executing post-install hooks"
+      before_rebuild: "Executing pre-rebuild hooks"
+      after_rebuild: "Executing post-rebuild hooks"
+      before_start: "Executing pre-start hooks"
+      after_start: "Executing post-start hooks"
+
     errors:
       autodetect_iso_path: |-
         Could not locate a local Virtualbox Guest Additions iso file.

data/vagrant-vbguest.gemspec

@@ -23,4 +23,10 @@ Gem::Specification.new do |s|
   s.executables   = s.files.grep(%r{^exe/}) { |f| File.basename(f) }
   s.require_paths = ["lib"]
 
+  s.metadata = {
+    "bug_tracker_uri" => "https://github.com/dotless-de/vagrant-vbguest/issues",
+    "changelog_uri" => "https://github.com/dotless-de/vagrant-vbguest/blob/main/CHANGELOG.md",
+    "documentation_uri" => "http://rubydoc.info/gems/vagrant-vbguest",
+    "source_code_uri" => "https://github.com/dotless-de/vagrant-vbguest"
+  }
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-vbguest
 version: !ruby/object:Gem::Version
-  version: 0.24.0
+  version: 0.29.0
 platform: ruby
 authors:
 - Robert Schulze
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-04-29 00:00:00.000000000 Z
+date: 2021-01-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: micromachine
@@ -66,13 +66,16 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
 - ".github/stale.yml"
 - ".gitignore"
 - CHANGELOG.md
+- CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE
 - Rakefile
 - Readme.md
+- SECURITY.md
 - VERSION
 - lib/vagrant-vbguest.rb
 - lib/vagrant-vbguest/command.rb
@@ -107,8 +110,12 @@ files:
 homepage: https://github.com/dotless-de/vagrant-vbguest
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  bug_tracker_uri: https://github.com/dotless-de/vagrant-vbguest/issues
+  changelog_uri: https://github.com/dotless-de/vagrant-vbguest/blob/main/CHANGELOG.md
+  documentation_uri: http://rubydoc.info/gems/vagrant-vbguest
+  source_code_uri: https://github.com/dotless-de/vagrant-vbguest
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -124,7 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.6
 requirements: []
 rubygems_version: 3.0.3
-signing_key: 
+signing_key:
 specification_version: 4
 summary: A Vagrant plugin to install the VirtualBoxAdditions into the guest VM
 test_files: []