Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change log file path for pre/post deploy commands to /var/tmp #894

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

vitabaks
Copy link
Owner

We have changed the path for storing the log file for the pre and post deploy commands from /tmp/ to /var/tmp in order for this to work if the server restart is specified in the command.

Fixed:

TASK [Get post-deploy command log] ***************************************************************************************************************************************
failed: [10.0.0.8] (item=10.0.0.8) => {"ansible_loop_var": "item", "changed": false, "cmd": ["cat", "/tmp/post_deploy_command.log"], "delta": "0:00:00.002540", "end": "2025-02-14 11:50:10.446425", "item": "10.0.0.8", "msg": "non-zero return code", "rc": 1, "start": "2025-02-14 11:50:10.443885", "stderr": "cat: /tmp/post_deploy_command.log: No such file or directory", "stderr_lines": ["cat: /tmp/post_deploy_command.log: No such file or directory"], "stdout": "", "stdout_lines": []}

Variables pre_deploy_command_log and post_deploy_command_log have been added to redefine the log path.

@vitabaks vitabaks self-assigned this Feb 14, 2025
@vitabaks vitabaks added the sponsor's request A request from the project sponsor label Feb 14, 2025
@vitabaks vitabaks merged commit f1d7883 into master Feb 14, 2025
15 checks passed
@vitabaks vitabaks deleted the command-log-path branch February 14, 2025 16:22
@vitabaks vitabaks added the automation Automation functionality using Ansible label Feb 16, 2025
christofalpo pushed a commit to christofalpo/postgresql_cluster that referenced this pull request Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Automation functionality using Ansible sponsor's request A request from the project sponsor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant