diff --git a/plugins/kernel_v2/config/trigger.rb b/plugins/kernel_v2/config/trigger.rb index b486aaf55..6e77f225f 100644 --- a/plugins/kernel_v2/config/trigger.rb +++ b/plugins/kernel_v2/config/trigger.rb @@ -90,6 +90,7 @@ module VagrantPlugins end # Validate Trigger settings + # TODO: Validate not called if there are providers defined in vagrantfile def validate(machine) errors = _detected_errors @_before_triggers.each do |bt| diff --git a/plugins/kernel_v2/config/vm_trigger.rb b/plugins/kernel_v2/config/vm_trigger.rb index 7211bca88..72bccdd3a 100644 --- a/plugins/kernel_v2/config/vm_trigger.rb +++ b/plugins/kernel_v2/config/vm_trigger.rb @@ -114,12 +114,12 @@ module VagrantPlugins errors = _detected_errors commands = [] - # TODO: Should this be cached...? Vagrant.plugin("2").manager.commands.each do |key,data| commands.push(key) end if !commands.include?(@command) + # does this make sense to print on this machine? machine.ui.warn(I18n.t("vagrant.config.triggers.bad_command_warning", cmd: @command)) end