Skip to content

Fix relaunching a sliced job at the Workflow level - #431

Merged
cigamit merged 2 commits into
mainfrom
sliced_relaunch
Jun 15, 2026
Merged

Fix relaunching a sliced job at the Workflow level#431
cigamit merged 2 commits into
mainfrom
sliced_relaunch

Conversation

@cigamit

@cigamit cigamit commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

This was working before, but recent changes seemed to have broken it

This was working before, but recent changes seemed to have broken it
@cigamit
cigamit requested review from TheWitness and Copilot June 15, 2026 18:02
@cigamit cigamit self-assigned this Jun 15, 2026
@cigamit cigamit added the bug Something isn't working label Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR restores the ability to relaunch sliced WorkflowJobs created from federated inventories, which were being incorrectly blocked by a stale slice-count check that relies on inventory.hosts.count() (always 0 for federated inventories).

Changes:

  • Add a functional regression test ensuring federated-inventory sliced workflows can be relaunched successfully.
  • Adjust the WorkflowJob relaunch validation to skip the stale-slice-count check when the inventory kind is federated.

Reviewed changes

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

File Description
awx/main/tests/functional/api/test_workflow_job.py Adds a regression test covering relaunch behavior for federated-inventory sliced workflow jobs.
awx/api/views/init.py Updates relaunch validation to exempt federated inventories from the host-count-based stale slice check.

Comment thread awx/main/tests/functional/api/test_workflow_job.py Outdated
@cigamit
cigamit merged commit 4aa8a72 into main Jun 15, 2026
@cigamit
cigamit deleted the sliced_relaunch branch July 20, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

3 participants