OLS-3558 Decouple execution success from symptom checks

[onmete]

Summary

• Draft prompt change for OLS-3558 (execution agent false failure from inline/symptom checks).
• Clarifies that top-level success means required apply work completed, not that the symptom has cleared.
• Treats verification.conditionOutcome as observational only; Unchanged/Degraded must not force success: false.
• Keeps authoritative symptom assessment for the dedicated Verification step when configured.

Context

Refinement of OLS-3558: an execution run correctly patched a NetworkPolicy, then set success: false because frontend logs still showed timeouts (~3s later / stale log lines). The operator hard-fails on success: false, so formal Verification never ran.

This PR is prompt-only and intended for design review during refinement. Spec updates (success vs inline verification contract) and any analysis-prompt changes (symptom post-checks in the execution script) are follow-ups.

Test plan

• Review prompt wording for trust mode (no spec.verification step)
• Confirm team agrees success must not be driven by symptom lag
• Follow-up: encode the same rules in .ai/spec (sandbox-execution.md / crd-api.md)
• Follow-up: eval or e2e covering apply-success + conditionOutcome: Unchanged → execution advances to Verification

Made with Cursor

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 171086f0-09a1-45ce-a422-3d2f9f2e1524

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}

[openshift-ci]

[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 assign onmete for approval. 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:

• OWNERS

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