turbaszek commented on a change in pull request #9947:
URL: https://github.com/apache/airflow/pull/9947#discussion_r463925330
##########
File path: airflow/providers/jenkins/operators/jenkins_job_trigger.py
##########
@@ -123,20 +129,20 @@ def build_job(self, jenkins_server):
"""
# Warning if the parameter is too long, the URL can be longer than
# the maximum allowed size
- if self.parameters and isinstance(self.parameters, str):
+ if parameters and isinstance(parameters, str):
import ast
- self.parameters = ast.literal_eval(self.parameters)
+ parameters = ast.literal_eval(parameters)
- if not self.parameters:
- # We need a None to call the non parametrized jenkins api end point
- self.parameters = None
+ if not parameters:
+ # We need to a None to call the non parametrized jenkins api end
point
Review comment:
```suggestion
# We need None to call the non parametrized jenkins api end point
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
