Merge pull request #14474 from jameswnl/cred-attr-required

flag required attributes for Ansible credentials
ManageIQ · Mar 24, 2017 · 4053f2e · 4053f2e
2 parents bf6fac9 + 814f477
commit 4053f2e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/app/models/manageiq/providers/ansible_tower/shared/automation_manager/amazon_credential.rb b/app/models/manageiq/providers/ansible_tower/shared/automation_manager/amazon_credential.rb
@@ -4,12 +4,14 @@ module ManageIQ::Providers::AnsibleTower::Shared::AutomationManager::AmazonCrede
   COMMON_ATTRIBUTES = {
     :userid => {
       :label     => N_('Access Key'),
-      :help_text => N_('AWS Access Key for this credential')
+      :help_text => N_('AWS Access Key for this credential'),
+      :required  => true
     },
     :password => {
       :type      => :password,
       :label     => N_('Secret Key'),
-      :help_text => N_('AWS Secret Key for this credential')
+      :help_text => N_('AWS Secret Key for this credential'),
+      :required  => true
     }
   }.freeze
 

diff --git a/app/models/manageiq/providers/ansible_tower/shared/automation_manager/vmware_credential.rb b/app/models/manageiq/providers/ansible_tower/shared/automation_manager/vmware_credential.rb
@@ -4,12 +4,14 @@ module ManageIQ::Providers::AnsibleTower::Shared::AutomationManager::VmwareCrede
   COMMON_ATTRIBUTES = {
     :userid => {
       :label     => N_('Username'),
-      :help_text => N_('Username for this credential')
+      :help_text => N_('Username for this credential'),
+      :required  => true
     },
     :password => {
       :type      => :password,
       :label     => N_('Password'),
-      :help_text => N_('Password for this credential')
+      :help_text => N_('Password for this credential'),
+      :required  => true
     }
   }.freeze
 
@@ -18,7 +20,8 @@ module ManageIQ::Providers::AnsibleTower::Shared::AutomationManager::VmwareCrede
       :type       => :string,
       :label      => N_('vCenter Host'),
       :help_text  => N_('The hostname or IP address of the vCenter Host'),
-      :max_length => 1024
+      :max_length => 1024,
+      :required   => true
     }
   }.freeze