Skip to content

Conversation

MoteHue
Copy link
Contributor

@MoteHue MoteHue commented Mar 14, 2025

This is a risky playbook to run without confirmation, especially as it targets all hosts when not using a limit.

@MoteHue MoteHue requested a review from a team as a code owner March 14, 2025 13:19
@product-auto-label product-auto-label bot added size: s ansible Ansible playbooks workflows Workflow files have been modified labels Mar 14, 2025
@MoteHue MoteHue force-pushed the add-confirmation-to-reboot branch 2 times, most recently from dbb62f0 to 54b229f Compare March 20, 2025 16:57
@MoteHue MoteHue requested a review from jovial March 20, 2025 16:58
@MoteHue MoteHue force-pushed the add-confirmation-to-reboot branch from 54b229f to 2614ea0 Compare March 21, 2025 09:53
@MoteHue MoteHue requested a review from Alex-Welsh March 25, 2025 11:39
@MoteHue MoteHue requested a review from Alex-Welsh March 25, 2025 15:10
@MoteHue MoteHue force-pushed the add-confirmation-to-reboot branch from a33cae4 to d0fc5c7 Compare March 25, 2025 15:10
@Alex-Welsh
Copy link
Member

Some CI errors, looks like Ark is having a bad time

MoteHue and others added 5 commits April 1, 2025 15:07
This is a risky playbook to run without confirmation, especially as it
targets all hosts when not using a limit.
@MoteHue MoteHue force-pushed the add-confirmation-to-reboot branch from d0fc5c7 to 424cc35 Compare April 1, 2025 14:07
@Alex-Welsh
Copy link
Member

Part of me wants to separate this out into a task list so we can use it in other playbooks with include_tasks, but that seems like more of a nasty workaround than elegant solution

@Alex-Welsh Alex-Welsh merged commit e7b235d into stackhpc/2024.1 Apr 2, 2025
14 checks passed
@Alex-Welsh Alex-Welsh deleted the add-confirmation-to-reboot branch April 2, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ansible Ansible playbooks size: m workflows Workflow files have been modified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants