From d3df5de0ced929deb0e24099a70de8b47a0b2f04 Mon Sep 17 00:00:00 2001 From: Ryan Petrello Date: Wed, 13 Sep 2017 13:28:26 -0400 Subject: [PATCH] build extra_cred related urls for jobs and JTs a less volatile way see: https://github.com/ansible/ansible-tower/issues/7635 --- awx/api/serializers.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/awx/api/serializers.py b/awx/api/serializers.py index 5dfb375133..873f822171 100644 --- a/awx/api/serializers.py +++ b/awx/api/serializers.py @@ -2334,8 +2334,13 @@ class JobOptionsSerializer(LabelsListMixin, BaseSerializer): if obj.vault_credential: res['vault_credential'] = self.reverse('api:credential_detail', kwargs={'pk': obj.vault_credential.pk}) if self.version > 1: - view = 'api:%s_extra_credentials_list' % camelcase_to_underscore(obj.__class__.__name__) - res['extra_credentials'] = self.reverse(view, kwargs={'pk': obj.pk}) + if isinstance(obj, UnifiedJobTemplate): + res['extra_credentials'] = self.reverse( + 'api:job_template_extra_credentials_list', + kwargs={'pk': obj.pk} + ) + elif isinstance(obj, UnifiedJob): + res['extra_credentials'] = self.reverse('api:job_extra_credentials_list', kwargs={'pk': obj.pk}) else: cloud_cred = obj.cloud_credential if cloud_cred: