cc @elastic/observablt-ci
Recent repository documentation audit found concrete routing documentation drift that can mislead users about when workflows run.
Changes Requiring Documentation Updates
1. Dependency-review routing omits the Control Plane dashboard gate
Commit(s): N/A (direct audit of current main state)
What changed (source of truth): Ingress requires dependency-review to pass the dashboard gate for workflow id obs:dependency-review.
- Source:
.github/workflows/oblt-aw-ingress.yml lines 59-66 (gate at line 65)
Documentation impact: docs/routing/dependency-review-routing.md currently lists event/action/author conditions only and does not document the required dashboard gate.
- Docs:
docs/routing/dependency-review-routing.md lines 11-20
2. Resource-not-accessible routing omits the shared dashboard gate for all three routes
Commit(s): N/A (direct audit of current main state)
What changed (source of truth): Ingress requires all three resource-not-accessible routes (detector, fixer, triage) to pass dashboard gate obs:resource-not-accessible-by-integration.
- Source:
.github/workflows/oblt-aw-ingress.yml lines 122-153 (gates at lines 126, 138, 152)
Documentation impact: docs/routing/resource-not-accessible-by-integration-routing.md describes trigger/label conditions but does not document the required dashboard gate.
- Docs:
docs/routing/resource-not-accessible-by-integration-routing.md lines 15-25
Suggested Actions
What is this? | From workflow: Observability Agentic Workflow Entrypoint
Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.
cc
@elastic/observablt-ciRecent repository documentation audit found concrete routing documentation drift that can mislead users about when workflows run.
Changes Requiring Documentation Updates
1. Dependency-review routing omits the Control Plane dashboard gate
Commit(s): N/A (direct audit of current
mainstate)What changed (source of truth): Ingress requires dependency-review to pass the dashboard gate for workflow id
obs:dependency-review..github/workflows/oblt-aw-ingress.ymllines 59-66 (gate at line 65)Documentation impact:
docs/routing/dependency-review-routing.mdcurrently lists event/action/author conditions only and does not document the required dashboard gate.docs/routing/dependency-review-routing.mdlines 11-202. Resource-not-accessible routing omits the shared dashboard gate for all three routes
Commit(s): N/A (direct audit of current
mainstate)What changed (source of truth): Ingress requires all three resource-not-accessible routes (detector, fixer, triage) to pass dashboard gate
obs:resource-not-accessible-by-integration..github/workflows/oblt-aw-ingress.ymllines 122-153 (gates at lines 126, 138, 152)Documentation impact:
docs/routing/resource-not-accessible-by-integration-routing.mddescribes trigger/label conditions but does not document the required dashboard gate.docs/routing/resource-not-accessible-by-integration-routing.mdlines 15-25Suggested Actions
docs/routing/dependency-review-routing.mdto add the required Control Plane dashboard gate condition (obs:dependency-review) alongside existing routing conditions.docs/routing/resource-not-accessible-by-integration-routing.mdto document that detector, triage, and fixer routes only run whenobs:resource-not-accessible-by-integrationis enabled.docs/workflows/oblt-aw-ingress.md(get-enabled-workflows/enabled-workflows) for gate semantics.What is this? | From workflow: Observability Agentic Workflow Entrypoint
Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.