a7ee56459b
With this change, the presence of Ansible configuration files (like playbook file, inventory path, galaxy role file, etc.) is no longer performed by the `config` classes, but by the `provisioner` classes (at the beginning of the provision command). This change fixes several issues: - Resolve #6984 as `provision` method are only executed when remote (ssh) communication with the guest machine is possible. - Resolve #6763 in a better way than 4e451c6 initially did. - Improve the general provisioner speed since the `config` checks are actually triggered by many vagrant actions (e.g. `destroy`,...), and can also be triggered multiple times during a vagrant run (e.g. on callback request made by the machine provider). Unlike the former `config`-based checks, the provision action won't collect all the invalid options, but only report the first invalid option found and abort the execution. Some unit tests were not implemented yet to save my scarce "open source contribution time" for other important issues, but they should be done at last via GH-6633.
112 lines
4.8 KiB
Ruby
112 lines
4.8 KiB
Ruby
module VagrantPlugins
|
|
module Ansible
|
|
module Config
|
|
class Base < Vagrant.plugin("2", :config)
|
|
|
|
GALAXY_COMMAND_DEFAULT = "ansible-galaxy install --role-file=%{role_file} --roles-path=%{roles_path} --force".freeze
|
|
|
|
attr_accessor :extra_vars
|
|
attr_accessor :galaxy_role_file
|
|
attr_accessor :galaxy_roles_path
|
|
attr_accessor :galaxy_command
|
|
attr_accessor :host_vars
|
|
attr_accessor :groups
|
|
attr_accessor :inventory_path
|
|
attr_accessor :limit
|
|
attr_accessor :playbook
|
|
attr_accessor :raw_arguments
|
|
attr_accessor :skip_tags
|
|
attr_accessor :start_at_task
|
|
attr_accessor :sudo
|
|
attr_accessor :sudo_user
|
|
attr_accessor :tags
|
|
attr_accessor :vault_password_file
|
|
attr_accessor :verbose
|
|
|
|
def initialize
|
|
@extra_vars = UNSET_VALUE
|
|
@galaxy_role_file = UNSET_VALUE
|
|
@galaxy_roles_path = UNSET_VALUE
|
|
@galaxy_command = UNSET_VALUE
|
|
@host_vars = UNSET_VALUE
|
|
@groups = UNSET_VALUE
|
|
@inventory_path = UNSET_VALUE
|
|
@limit = UNSET_VALUE
|
|
@playbook = UNSET_VALUE
|
|
@raw_arguments = UNSET_VALUE
|
|
@skip_tags = UNSET_VALUE
|
|
@start_at_task = UNSET_VALUE
|
|
@sudo = UNSET_VALUE
|
|
@sudo_user = UNSET_VALUE
|
|
@tags = UNSET_VALUE
|
|
@vault_password_file = UNSET_VALUE
|
|
@verbose = UNSET_VALUE
|
|
end
|
|
|
|
def finalize!
|
|
@extra_vars = nil if @extra_vars == UNSET_VALUE
|
|
@galaxy_role_file = nil if @galaxy_role_file == UNSET_VALUE
|
|
@galaxy_roles_path = nil if @galaxy_roles_path == UNSET_VALUE
|
|
@galaxy_command = GALAXY_COMMAND_DEFAULT if @galaxy_command == UNSET_VALUE
|
|
@host_vars = {} if @host_vars == UNSET_VALUE
|
|
@groups = {} if @groups == UNSET_VALUE
|
|
@inventory_path = nil if @inventory_path == UNSET_VALUE
|
|
@limit = nil if @limit == UNSET_VALUE
|
|
@playbook = nil if @playbook == UNSET_VALUE
|
|
@raw_arguments = nil if @raw_arguments == UNSET_VALUE
|
|
@skip_tags = nil if @skip_tags == UNSET_VALUE
|
|
@start_at_task = nil if @start_at_task == UNSET_VALUE
|
|
@sudo = false if @sudo != true
|
|
@sudo_user = nil if @sudo_user == UNSET_VALUE
|
|
@tags = nil if @tags == UNSET_VALUE
|
|
@vault_password_file = nil if @vault_password_file == UNSET_VALUE
|
|
@verbose = false if @verbose == UNSET_VALUE
|
|
end
|
|
|
|
# Just like the normal configuration "validate" method except that
|
|
# it returns an array of errors that should be merged into some
|
|
# other error accumulator.
|
|
def validate(machine)
|
|
@errors = _detected_errors
|
|
|
|
# Validate that a playbook path was provided
|
|
if !playbook
|
|
@errors << I18n.t("vagrant.provisioners.ansible.errors.no_playbook")
|
|
end
|
|
|
|
# Validate that extra_vars is either a Hash or a String (for a file path)
|
|
if extra_vars
|
|
extra_vars_is_valid = extra_vars.kind_of?(Hash) || extra_vars.kind_of?(String)
|
|
if extra_vars.kind_of?(String)
|
|
# Accept the usage of '@' prefix in Vagrantfile
|
|
# (e.g. '@vars.yml' and 'vars.yml' are both supported)
|
|
match_data = /^@?(.+)$/.match(extra_vars)
|
|
extra_vars_path = match_data[1].to_s
|
|
@extra_vars = '@' + extra_vars_path
|
|
end
|
|
|
|
if !extra_vars_is_valid
|
|
@errors << I18n.t(
|
|
"vagrant.provisioners.ansible.errors.extra_vars_invalid",
|
|
type: extra_vars.class.to_s,
|
|
value: extra_vars.to_s)
|
|
end
|
|
end
|
|
|
|
if raw_arguments
|
|
if raw_arguments.kind_of?(String)
|
|
@raw_arguments = [raw_arguments]
|
|
elsif !raw_arguments.kind_of?(Array)
|
|
@errors << I18n.t(
|
|
"vagrant.provisioners.ansible.errors.raw_arguments_invalid",
|
|
type: raw_arguments.class.to_s,
|
|
value: raw_arguments.to_s)
|
|
end
|
|
end
|
|
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|