hashgraph / hashgraph/solo-weaver

Story 5.7 — Implement DaemonResult condition handshake before PendingNodeUpgrade

Open
#543 0 comments 0 reactions 1 assignee Claimed by @leninmehedy View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.