Skip to content

fix(lego_openbao): only restart containers if they exist - #285

Merged
github-actions[bot] merged 1 commit into
mainfrom
sf/fix-lego-openbao-restart
Aug 12, 2026
Merged

github-actions[bot] merged 1 commit into
mainfrom
sf/fix-lego-openbao-restart

Conversation

@sebastian-de

Copy link
Copy Markdown
Contributor

While deploying a new host I discovered that this role fails if the containers defined in lego_openbao_restart_container_names don't exist, yet.

@sebastian-de
sebastian-de requested a review from a team as a code owner August 12, 2026 11:00
@cursor

cursor Bot commented Aug 12, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Small Ansible handler fix that only skips restarting missing containers; no auth, secrets, or core certificate logic changes.

Overview
Makes the restart_containers handler resilient on fresh hosts by checking container existence before restarting.

Previously it looped lego_openbao_restart_container_names directly and failed when named containers were missing. It now queries docker_container_info first and restarts only containers that already exist.

Reviewed by Cursor Bugbot for commit 6db534d. Bugbot is set up for automated code reviews on this repo. Configure here.

@sebastian-de

Copy link
Copy Markdown
Contributor Author

/fast-forward

@github-actions
github-actions Bot merged commit 6db534d into main Aug 12, 2026
2 checks passed
@github-actions
github-actions Bot deleted the sf/fix-lego-openbao-restart branch August 12, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants