From 631f50d21bc3d38a63b80867b54b7f22965ac0d1 Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Fri, 5 Jul 2024 13:01:30 -0700 Subject: [PATCH] CI: set tftp folder permission In trying to chase down why the raw tftp boot of grub is not happy, I determined that the tftp folder being created had the wrong permissions out of the box. Ironic has an optional knob for this, so we're going to set it by default. Change-Id: If2a0e5e47163a3525ecd245e8b54cacea9a615de --- devstack/lib/ironic | 2 ++ playbooks/ci-workarounds/get_extra_logging.yaml | 3 +++ 2 files changed, 5 insertions(+) diff --git a/devstack/lib/ironic b/devstack/lib/ironic index 72d6c36dbe..53792e98a7 100644 --- a/devstack/lib/ironic +++ b/devstack/lib/ironic @@ -1600,6 +1600,8 @@ function configure_ironic { iniset $IRONIC_CONF_FILE agent deploy_logs_collect $IRONIC_DEPLOY_LOGS_COLLECT iniset $IRONIC_CONF_FILE agent deploy_logs_storage_backend $IRONIC_DEPLOY_LOGS_STORAGE_BACKEND iniset $IRONIC_CONF_FILE agent deploy_logs_local_path $IRONIC_DEPLOY_LOGS_LOCAL_PATH + # Allow ironic to set permissions for the tftp test usage. + iniset $IRONIC_CONF_FILE pxe dir_permission 493 # Set image_download_source for direct interface if [[ -n "$IRONIC_AGENT_IMAGE_DOWNLOAD_SOURCE" ]]; then iniset $IRONIC_CONF_FILE agent image_download_source $IRONIC_AGENT_IMAGE_DOWNLOAD_SOURCE diff --git a/playbooks/ci-workarounds/get_extra_logging.yaml b/playbooks/ci-workarounds/get_extra_logging.yaml index 117a633ef1..cfd83759ad 100644 --- a/playbooks/ci-workarounds/get_extra_logging.yaml +++ b/playbooks/ci-workarounds/get_extra_logging.yaml @@ -3,6 +3,9 @@ - name: Get tftpd info from journald shell: "journalctl -t in.tftpd > {{ zuul_output_dir }}/logs/tftpd-journal.txt" become: yes + - name: Get tftpd info from journald + shell: "ls -laR /opt/stack/data/ironic/tftpboot /opt/stack/data/ironic/httpboot > {{ zuul_output_dir }}/logs/ironic-files.txt" + become: yes - name: Get network connection information for ironic process shell: "netstat -apn > {{ zuul_output_dir }}/logs/post-job-network-connections.txt" become: yes