hashgraph / hashgraph/solo-weaver
Story 5.7 — Implement DaemonResult condition handshake before PendingNodeUpgrade
- Dominant language
- Go
- Stars
- 3
- Forks
- 0
- Avg merge
- 3d 5h
- Merged PRs (30d)
- 47
Description
**Epic:** #502 — Network Upgrade Workflow (Execute Phase)
Set the `DaemonResult` condition on the `NetworkUpgradeExecute` CR and transition to `PendingNodeUpgrade`
to communicate the daemon's outcome to the operator's reconciler.
**Finalized model (corrects the previous AC):** the daemon ALWAYS ends every execute-phase outcome —
success OR any failure/early-halt — by setting `DaemonResult` then transitioning to `PendingNodeUpgrade`.
The reconciler is the **sole writer** of terminal `Succeeded`/`Failed`. The daemon never writes a terminal
phase, and there is no `InProgress` phase.
**Acceptance criteria:**
- On success: set `DaemonResult` condition to `True` with reason + message, then transition to `PendingNodeUpgrade`.
- On any failure/halt: set `DaemonResult` to `False` with a reason + message identifying the failure (e.g. CR kind/name/filename, or `SelfUpgradeFailed`), then transition to `PendingNodeUpgrade` so the `ExecuteReconciler` can mark the CR `Failed`.
- The daemon does NOT patch `Succeeded`/`Failed` directly.
Contributor guide
Assessment
This issue has not been assessed yet.