ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

jenkins_plugin: avoid undefined variable when updates file is not downloaded (#6100)

Browse source 
Avoid undefined variable when updates file is not downloaded.
This commit is contained in:
Felix Fontein 2023-02-26 15:37:16 +01:00 committed by GitHub
parent 29f5033737
commit e8bdec2733
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/6100-jenkins_plugin.yml Normal file
View file

@ -0,0 +1,2 @@
bugfixes:
- "jenkins_plugin - fix error due to undefined variable when updates file is not downloaded (https://github.com/ansible-collections/community.general/pull/6100)."

8
plugins/modules/jenkins_plugin.py
View file

@ -641,6 +641,8 @@ class JenkinsPlugin(object):
self.module.fail_json(
msg="Cannot close the tmp updates file %s." % tmp_updates_file,
details=to_native(e))
else:
tmp_updates_file = updates_file
# Open the updates file
try:
@ -651,15 +653,15 @@ class JenkinsPlugin(object):
data = json.loads(f.readline())
except IOError as e:
self.module.fail_json(
msg="Cannot open temporal updates file.",
msg="Cannot open%s updates file." % (" temporary" if tmp_updates_file != updates_file else ""),
details=to_native(e))
except Exception as e:
self.module.fail_json(
msg="Cannot load JSON data from the tmp updates file.",
msg="Cannot load JSON data from the%s updates file." % (" temporary" if tmp_updates_file != updates_file else ""),
details=to_native(e))
# Move the updates file to the right place if we could read it
if download_updates:
if tmp_updates_file != updates_file:
self.module.atomic_move(tmp_updates_file, updates_file)
# Check if we have the plugin data available