Deprecate ssh_username in provider config

ggiamarchi · Sep 17, 2014 · f574e12 · f574e12
1 parent 0e4b086
commit f574e12
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/source/lib/vagrant-openstack-provider/config.rb b/source/lib/vagrant-openstack-provider/config.rb
@@ -183,6 +183,8 @@ def validate(machine)
         errors << I18n.t('vagrant_openstack.config.password_required') unless @password
         errors << I18n.t('vagrant_openstack.config.username_required') unless @username
 
+        validate_ssh_username
+
         if machine.config.ssh.private_key_path
           puts I18n.t('vagrant_openstack.config.keypair_name_required').yellow unless @keypair_name || @public_key_path
         else
@@ -201,6 +203,10 @@ def validate(machine)
         { 'Openstack Provider' => errors }
       end
 
+      def validate_ssh_username
+        puts I18n.t('vagrant_openstack.config.ssh_username_deprecated').yellow if @ssh_username
+      end
+
       private
 
       def valid_uri?(value)

diff --git a/source/locales/en.yml b/source/locales/en.yml
@@ -62,6 +62,10 @@ en:
       private_key_missing: |-
         config.ssh.private_key_path is required when either keypair_name or
         public_key_path is set in Vagrantfile
+      ssh_username_deprecated: |-
+        ssh_username provider config is deprecated for vagrant-openstack provider.
+        If you are using it, it will continue to work but we recommend to switch to the
+        standard vagrant configuration option `config.ssh.username` instead
 
     errors:
       default: |-