Run outcomes: refusal and no-op must not report bare Succeeded

Open
#129 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

@ibolton336 is already working on this.

Since Oct 5, 2026.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
48/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Quiet
Tech stack
go, kubernetes
Domain
api, backend

Research direction

Start with ADR 0011 and issue #119 to understand the existing termination and exit-code contract, then inspect the E2E coverage for run status and the outcome reporting mentioned in PR #114. Done means a refused gate is not reported as Succeeded, and a no-change run is distinguishable from a changed run in the UI-facing status.

Written by the indexing model from the issue text.

Description

bug good first issue

Today a stage-gate refusal still reports Succeeded — #96 fixed cancels only. Define the outcome contract — refused / failed / succeeded / succeeded-with-no-changes — and surface it on run status for the UI. Frame it as an extension of ADR 0011's exit-code contract: exit 2 → LimitReached (#119) is the precedent; a refusal wants the same shape, not a bespoke channel. Related hook: the "results pushed" notice overstates outcomes the same way — flagged in PR #114.

Done when:

  • E2E shows a refused gate ≠ Succeeded.
  • A no-change run is distinguishable from a changed one.

Depends on: #119 (rides the same termination mechanism).

Dominant language
Go
Stars
5
Forks
13
Avg merge
3d 11h
Merged PRs (30d)
41

Getting set up

  • Ships a Dockerfile or Docker Compose file
  • Has a pull request template
  • No contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from konveyor/agentic-controller

All issues in konveyor/agentic-controller

Similar issues

More Go issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.