From c1455ee12501e3e5a5942450f3cfa0e9aaa56a19 Mon Sep 17 00:00:00 2001 From: Dien Nguyen Date: Thu, 13 Apr 2023 14:36:38 -0400 Subject: [PATCH] bugfix: add scm_branch to optional_args for workflow_launch (#13254) * add scm_branch to optional_args * add in limits * Update workflow_launch.py remove json from import to pass linting. --------- Co-authored-by: dien nguyen Co-authored-by: Jessica Steurer <70719005+jay-steurer@users.noreply.github.com> --- awx_collection/plugins/modules/workflow_launch.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/awx_collection/plugins/modules/workflow_launch.py b/awx_collection/plugins/modules/workflow_launch.py index 1613e4fa8b..afbf9ca8d6 100644 --- a/awx_collection/plugins/modules/workflow_launch.py +++ b/awx_collection/plugins/modules/workflow_launch.py @@ -91,7 +91,6 @@ EXAMPLES = ''' ''' from ..module_utils.controller_api import ControllerAPIModule -import json def main(): @@ -116,15 +115,18 @@ def main(): name = module.params.get('name') organization = module.params.get('organization') inventory = module.params.get('inventory') - optional_args['limit'] = module.params.get('limit') wait = module.params.get('wait') interval = module.params.get('interval') timeout = module.params.get('timeout') - # Special treatment of extra_vars parameter - extra_vars = module.params.get('extra_vars') - if extra_vars is not None: - optional_args['extra_vars'] = json.dumps(extra_vars) + for field_name in ( + 'limit', + 'extra_vars', + 'scm_branch', + ): + field_val = module.params.get(field_name) + if field_val is not None: + optional_args[field_name] = field_val # Create a datastructure to pass into our job launch post_data = {}