opendev/system-config
Code Issues Proposed changes

Fix nodepool image export cron

Browse Source 
This is a followup to the prior fix that addressed the path issue. Now
we have the problem of docker-compose attempting to allocate a tty (the
default) which isn't possible by default under cron. We don't need a tty
so we pass -T to disable tty allocation in the first place.

I should've caught this the last time around but my testing didn't catch
it because I was running from a shell.

Change-Id: I57797c8d140335d9edcdcd324239fdefb09882d4
This commit is contained in:
Clark Boylan 2025-03-19 07:53:45 -07:00
parent 161f709c2d
commit f1bc64a58e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
playbooks/roles/nodepool-builder/tasks/main.yaml
View File

@ -49,4 +49,4 @@
hour: '{{ 23 | random(seed=inventory_hostname) }}'
# Note we backup to /var/log/nodepool because that is what we have bind
# mounted in the container and we are operating relative to that context
job: /usr/local/bin/docker-compose -f /etc/nodepool-builder-compose/docker-compose.yaml exec nodepool-builder nodepool export-image-data /var/log/nodepool/nodepool-image-backup.json
job: /usr/local/bin/docker-compose -f /etc/nodepool-builder-compose/docker-compose.yaml exec -T nodepool-builder nodepool export-image-data /var/log/nodepool/nodepool-image-backup.json