Add initial attempts to automatically clean up dangling hosts

[JacobCallahan]

Now, when workflows fail, Broker will attempt to find a dangling host and check it in if found.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.

Comments skipped due to low confidence (2)

broker/providers/ansible_tower.py:381

• Check for the existence of 'node.summary_fields["job"]' and 'node.summary_fields["job"]["failed"]' before accessing them to avoid potential KeyError.

if node.summary_fields["job"]["failed"]:

broker/providers/ansible_tower.py:388

• Ensure that 'job' is not None before proceeding to avoid potential AttributeError.

def _try_checkin_dangling_host(self, job):

Copilot reviewed 4 out of 4 changed files in this pull request and generated no suggestions.

Comments skipped due to low confidence (1)

broker/config_manager.py:102

• The condition 'and "example" not in module.name' could be more specific to avoid false positives.

if hasattr(module, "run_migrations") and "example" not in module.__name__:

[Copilot]

PR Overview

This PR introduces automatic clean‐up for dangling hosts by adding a configuration migration and extending the AnsibleTower provider to handle dangling hosts when workflows fail.

• Adds a new migration (v0_6_3.py) to set a default dangling_behavior of "checkin" for AnsibleTower.
• Implements new methods (_try_get_dangling_hosts and handle_dangling_hosts) in the AnsibleTower provider with support for prompting the user.
• Adjusts the config manager to bypass example migrations when running migrations.

Reviewed Changes

File	Description
broker/config_migrations/v0_6_3.py	New migration adding default dangling_behavior configuration
broker/config_migrations/example_migration.py	Provides template for creating new migrations
broker/providers/ansible_tower.py	Implements dangling host cleanup logic and user prompt
broker/config_manager.py	Filters out example migrations from the migrations list

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

PR Overview

This PR introduces an initial implementation for automatically cleaning up dangling hosts left behind by failed workflows in Broker’s AnsibleTower provider. The changes include:

• A new configuration migration (v0_6_3.py) that adds a "dangling_behavior" setting to AnsibleTower.
• Extensions to the AnsibleTower provider with new methods for detecting and handling dangling hosts, including an interactive prompt option.
• A minor update to the configuration manager to exclude example migration modules from execution.

Reviewed Changes

File	Description
broker/config_migrations/v0_6_3.py	Adds a migration function to introduce "dangling_behavior".
broker/providers/ansible_tower.py	Enhances the provider with methods (_try_get_dangling_hosts & handle_dangling_hosts) and adjusts configuration handling.
broker/config_migrations/example_migration.py	Provides an example migration file with usage instructions.
broker/config_manager.py	Updates migration discovery to skip example migration modules.

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.