External-Mirrors/awx
2
0
Fork 0
mirror of https://github.com/ansible/awx.git synced 2026-05-08 01:47:35 -02:30
Code Issues Packages Projects Releases Wiki Activity

fix a bug that breaks workflows w/ a survey password + inventory sync

Browse Source 
prior versions of awx did not raise an exception for this scenario
- they simply ignored kwargs that they couldn't accept.  this change is
a sort of middle ground - it ignores them, but gives a clue in the logs
as to why

see: https://github.com/ansible/awx/issues/955
related: https://github.com/ansible/awx/pull/803
This commit is contained in:
Ryan Petrello
2018-01-22 09:41:30 -05:00
parent 09d63b4883
commit ac3f7d0fac
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
awx/main/models/unified_jobs.py
View File

@@ -355,7 +355,8 @@ class UnifiedJobTemplate(PolymorphicModel, CommonModelNameNotUnique, Notificatio
fields = self._get_unified_job_field_names() fields = self._get_unified_job_field_names()
unallowed_fields = set(kwargs.keys()) - set(fields) unallowed_fields = set(kwargs.keys()) - set(fields)
if unallowed_fields: if unallowed_fields:
raise Exception('Fields {} are not allowed as overrides.'.format(unallowed_fields)) logger.warn('Fields {} are not allowed as overrides.'.format(unallowed_fields))
map(kwargs.pop, unallowed_fields)
unified_job = copy_model_by_class(self, unified_job_class, fields, kwargs) unified_job = copy_model_by_class(self, unified_job_class, fields, kwargs)