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

Copy the permissions along with file for jboss module (#3426) (#3468)

Browse source 
* Copy the permissions along with file for jboss module

Issue: The deployment file is copied with file content only. The file
permission is set to 440 and belongs to root user. When the
JBossI(Wildfly) server is running under non root account, it can't read
the deployment file.

With is fix, the correct permission can be set from previous task for JBoss
server to pickup the deployment file.

* Update changelogs/fragments/3426-copy-permissions-along-with-file-for-jboss-module.yml

Co-authored-by: Ajpantuso <ajpantuso@gmail.com>

Co-authored-by: Ajpantuso <ajpantuso@gmail.com>
(cherry picked from commit 7cfdc2ce8c)

Co-authored-by: Pan Luo <xcompass@gmail.com>
This commit is contained in:
patchback[bot] 2021-09-29 05:44:53 +00:00 committed by GitHub
parent bcebeddb83
commit 2cec7a1779
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
changelogs/fragments/3426-copy-permissions-along-with-file-for-jboss-module.yml Normal file
View file

@ -0,0 +1,6 @@
bugfixes:
- jboss - fix the deployment file permission issue when Jboss server is running
under non-root user. The deployment file is copied with file content only. The
file permission is set to ``440`` and belongs to root user. When the
JBoss ``WildFly`` server is running under non-root user, it is unable to read
the deployment file (https://github.com/ansible-collections/community.general/pull/3426).

4
plugins/modules/web_infrastructure/jboss.py
View file

@ -142,7 +142,7 @@ def main():
# Clean up old failed deployment # Clean up old failed deployment
os.remove(os.path.join(deploy_path, "%s.failed" % deployment)) os.remove(os.path.join(deploy_path, "%s.failed" % deployment))
shutil.copyfile(src, os.path.join(deploy_path, deployment)) module.preserved_copy(src, os.path.join(deploy_path, deployment))
while not deployed: while not deployed:
deployed = is_deployed(deploy_path, deployment) deployed = is_deployed(deploy_path, deployment)
if is_failed(deploy_path, deployment): if is_failed(deploy_path, deployment):
@ -153,7 +153,7 @@ def main():
if state == 'present' and deployed: if state == 'present' and deployed:
if module.sha1(src) != module.sha1(os.path.join(deploy_path, deployment)): if module.sha1(src) != module.sha1(os.path.join(deploy_path, deployment)):
os.remove(os.path.join(deploy_path, "%s.deployed" % deployment)) os.remove(os.path.join(deploy_path, "%s.deployed" % deployment))
shutil.copyfile(src, os.path.join(deploy_path, deployment)) module.preserved_copy(src, os.path.join(deploy_path, deployment))
deployed = False deployed = False
while not deployed: while not deployed:
deployed = is_deployed(deploy_path, deployment) deployed = is_deployed(deploy_path, deployment)