Skip to content

Commit ce8a96f

Browse files
rebtooropenshift-merge-bot[bot]
authored andcommitted
[cifmw_cephadm] Fix log path when post.yml delegates to localhost
cifmw_cephadm_log_path uses ansible_user_dir which resolves to /root on compute nodes running with become: true. The post.yml and logs.yml tasks delegate_to: localhost to write log files, but on the controller the zuul user cannot create directories under /root. Use cifmw_basedir (always /home/zuul/ci-framework-data in CI) with a fallback to the original expression for non-CI contexts. Related-Issue: ANVIL-109 Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>
1 parent bf73b99 commit ce8a96f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

roles/cifmw_cephadm/defaults/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ cifmw_cephadm_urischeme: "http"
110110
cifmw_cephadm_config_key_set_ssl_option: no_sslv2:sslv3:no_tlsv1:no_tlsv1_1
111111
# Deployment logs variables
112112
cifmw_cephadm_log_dump: true
113-
cifmw_cephadm_log_path: "{{ ansible_user_dir ~ '/ci-framework-data/logs/ceph' }}"
113+
cifmw_cephadm_log_path: "{{ cifmw_basedir | default(ansible_user_dir ~ '/ci-framework-data') }}/logs/ceph"
114114
cifmw_cephadm_log_commands:
115115
# Get deployed Ceph daemons
116116
- type: "daemons"

0 commit comments

Comments
 (0)