Skip to content

[strategist] planning: RFC #4002 phased delivery roadmap (docs/rfc-4002-phased-roadmap.md) - #5635

Merged
clubanderson merged 2 commits into
v4from
strategy/rfc-4002-phased-roadmap
Sep 3, 2026
Merged

[strategist] planning: RFC #4002 phased delivery roadmap (docs/rfc-4002-phased-roadmap.md)#5635
clubanderson merged 2 commits into
v4from
strategy/rfc-4002-phased-roadmap

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Planning Artifact

Adds docs/rfc-4002-phased-roadmap.md (new file, only file touched): a phased delivery roadmap for RFC #4002 (re-entrant conversation-as-state turn model).

Cluster claimed: RFC #4002 planning only — no source code, no overlap with open hold PRs #5634 (escalation ledger), #5628 (pin-off docs), #5625 (mutation ledger persist), #5559 (UPGRADE.md), #4032 (proxy auth).

Refs #5555, Refs #4002


Filed by strategist agent (ACMM L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=strategist backend=copilot model=claude-sonnet-4-6

…ant turn model)

Adds docs/rfc-4002-phased-roadmap.md: compatibility contract, Phases 0-3
with measurable gates, and v5/v6 version targeting. Refs #5555, #4002.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: strategist <sec-check@hive.kubestellar.io>
Signed-off-by: sec-check <sec-check@hive.kubestellar.io>

@clubanderson clubanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

CONCERNS: this doc now competes with the phased roadmap posted on issue #5555 (operator comment, 2026-09-02), and the two disagree on substance, not just numbering:

  • Phase structure: this doc has Phases 0–3; the #5555 roadmap has Phases 0–5. This doc's Phase 1 (first live wiring on contributor agents) is the #5555 roadmap's Phase 3, and this doc omits that roadmap's Phase 1 (converging pkg/turn and pkg/convergence/mutation while still inert) and Phase 2 (a Promela/Spin model in src/formal/turn/ as a hard precondition before anything passes the edge channel) entirely — no formal-model step appears here at all.
  • Kill criterion: the #5555 Phase 0 has an explicit exit ramp (close #4002 if TurnLoss data shows Producing is a small fraction of Interruptions); this doc's Phase 0 gate is only 'report reviewed; go/no-go recorded'.
  • Version targeting conflicts: this doc sends its Phases 2–3 to v6 ('not a v5 GA blocker'); the #5555 roadmap is explicitly v5-line delivery (live wiring on v5 only, v4 untouched).
  • Scope deltas: this doc folds #4000 in ('should not ship separately') and already exercises handoff in its Phase 1 gate ('handoff exercised at least once across a process restart'); the #5555 roadmap defers handoff to Phase 5 'only if still wanted' and does not subordinate #4000.

Two competing roadmaps for the same RFC is worse than either alone. Suggest either rewriting this file to be the in-repo rendering of the #5555 roadmap once the operator accepts it, or closing this PR in its favor. Verdict: CONCERNS — do not merge as-is.

@clubanderson

Copy link
Copy Markdown
Member

Fixes #5555

@clubanderson clubanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed against the #5555 dispatch plan. I pushed a docs-only fix so the roadmap now names the already-shipped stages 1–2 and documented step-3 handoff eval, keeps the compatibility contract, makes Phase 2 cover all background agents, and adds measurable Phase-2 gates. go build ./... passed before the docs-only adjustment; PR comment links Fixes #5555.

@kubestellar-prow kubestellar-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Sep 3, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 19736b62c75919e401d620a3d249e3d36dde3ee8

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Andy Anderson <andy@clubanderson.com>
@clubanderson
clubanderson force-pushed the strategy/rfc-4002-phased-roadmap branch from 1842afe to 88c6c74 Compare September 3, 2026 01:43
@kubestellar-prow kubestellar-prow Bot removed the lgtm Indicates that a PR is ready to be merged. label Sep 3, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from clubanderson. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@clubanderson clubanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Re-reviewed after my amended docs-only fix. The roadmap now matches #5555's requested dispatch plan, including delivered investigation stages, all-background Phase 2, compatibility contract, and measurable gates. go build ./... passed before the docs-only changes.

@kubestellar-prow kubestellar-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Sep 3, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: fe072de26a73702b1511daa41831c78ead145591

@clubanderson
clubanderson merged commit b8c6120 into v4 Sep 3, 2026
31 of 32 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the strategy/rfc-4002-phased-roadmap branch September 3, 2026 01:51
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

Check out what's new:

Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

copilot dco-signoff: yes Indicates the PR's author has signed the DCO. hold lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant