mirror of
https://github.com/ansible/awx.git
synced 2026-05-06 08:57:35 -02:30
Fix for incorrect error messages on incomplete credentials.
This commit is contained in:
@@ -1345,15 +1345,17 @@ class UnifiedJob(
|
|||||||
|
|
||||||
# verify that any associated credentials aren't missing required field data
|
# verify that any associated credentials aren't missing required field data
|
||||||
missing_credential_inputs = []
|
missing_credential_inputs = []
|
||||||
|
credential_name = ""
|
||||||
for credential in self.credentials.all():
|
for credential in self.credentials.all():
|
||||||
defined_fields = credential.credential_type.defined_fields
|
defined_fields = credential.credential_type.defined_fields
|
||||||
for required in credential.credential_type.inputs.get('required', []):
|
for required in credential.credential_type.inputs.get('required', []):
|
||||||
if required in defined_fields and not credential.has_input(required):
|
if required in defined_fields and not credential.has_input(required):
|
||||||
missing_credential_inputs.append(required)
|
missing_credential_inputs.append(required)
|
||||||
|
credential_name = credential.name
|
||||||
|
|
||||||
if missing_credential_inputs:
|
if missing_credential_inputs:
|
||||||
self.job_explanation = '{} cannot start because Credential {} does not provide one or more required fields ({}).'.format(
|
self.job_explanation = '{} cannot start because Credential {} does not provide one or more required fields ({}).'.format(
|
||||||
self._meta.verbose_name.title(), credential.name, ', '.join(sorted(missing_credential_inputs))
|
self._meta.verbose_name.title(), credential_name, ', '.join(sorted(missing_credential_inputs))
|
||||||
)
|
)
|
||||||
self.save(update_fields=['job_explanation'])
|
self.save(update_fields=['job_explanation'])
|
||||||
return (False, None)
|
return (False, None)
|
||||||
|
|||||||
Reference in New Issue
Block a user