Skip to content

fix(ui): count failed hosts and workloads apart in rollout dialogs - #837

Merged
notheotherben merged 1 commit into
mainfrom
claude/project-thread-ah1rd0
Nov 6, 2026
Merged

notheotherben merged 1 commit into
mainfrom
claude/project-thread-ah1rd0

Conversation

@notheotherben

Copy link
Copy Markdown
Member

Requested by Ben · project thread

Rollout confirmation dialogs tallied every unit as a host, so a failed placement read as "N failed hosts (nomad, …)".

  • logic::units::tally counts hosts, clusters and workloads apart, naming placements workload on cluster: 2 failed hosts (pi-1, pi-2) and 1 failed workload (grafana on nomad).
  • Used by Retry, Skip this phase, Expedite and Roll back dialogs; the paused banner says "Retry what failed".
  • Unit test covers mixed, workload-only, host-only and empty tallies.

Generated by Claude Code

Retry, Skip this phase, Expedite and Roll back confirmations tallied every
unit as a host, so failed placements read as "N failed hosts" listing
cluster names. units::tally counts hosts, clusters and workloads apart and
names placements "workload on cluster".

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WkowUuST81Z3SBP92m6Tyv
@notheotherben notheotherben self-assigned this Nov 6, 2026
@notheotherben
notheotherben merged commit 9f9e485 into main Nov 6, 2026
6 checks passed
@notheotherben
notheotherben deleted the claude/project-thread-ah1rd0 branch November 6, 2026 10:03
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