Service EN4A
Pull Request Review
Runs transactional. Starts when a threshold is crossed. It runs 6,838 times a week across 6 stations. 2 of those stations stop for a person and 4 are done by an agent, 3 of them on the shared spine.
Runs a week
6,838
92 in flight right now.
Stations
6
2 person · 1 agent · 3 spine
Agent share of steps
67%
Autonomy on this service is set at 82% against a ceiling of 88%.
Decisions a person makes
2
Held by Director, Engineering Quality.
Touchless
89%
Share of runs that complete without anybody opening them.
Timetable against actual
11.9h
Actual 14.8h, 2.9h over the timetable.
Operations
The lane diagram
Read it left to right. more
Operations
Every station in order
The same stations as a list, with the dwell the estate records against each one and the system a station of that shape would touch.
| # | Station | Lane | Who | Layer | Dwell | Target | In flight | System |
|---|---|---|---|---|---|---|---|---|
| 1 | Static analysis | GBS | Code Quality Orchestrator | gbs | 6m | 14m | 20 | Group ERP — Materials and Inventory |
| 2 | Generate review comments | GBS | Static Analysis Agent | gbs | 1h 24m | 2h 11m | 18 | Group ERP — Materials and Inventory |
| 3 | Score risk | Agent | Complexity Monitor | workflow | 2h 37m | 5h 46m | 46 | Configuration Management Database |
| 4 | Auto-approve trivial | GBS | Complexity Monitor | gbs | 1h 16m | 2h 43m | 3 | Group ERP — Materials and Inventory |
| 5 | Human review Held for a named human when the unit exceeds the pull request decision envelope, when confidence falls below the floor, or when a policy clause is engaged. | Human | Director, Engineering Quality | policy | 1h 49m | 3h 1m | 1 | Group ERP — Materials and Inventory |
| 6 | Merge Held for a named human when the unit exceeds the pull request decision envelope, when confidence falls below the floor, or when a policy clause is engaged. | Human | Director, Engineering Quality | policy | 2h 26m | 4h 34m | 4 | Group ERP — Materials and Inventory |
Operations
Run it
Change the volume, move the autonomy line, release a gate, and watch what happens to the hours in each lane and to the time it takes one run to get through. more
The simulation is arithmetic on the estate, not a queueing model. more
How it runs
Run it again, going wrong
The same unit and the same stations, broken the way the estate already records breaking. more
6 stations, walked twice
Once clean, once with the categories the estate counts
This is the same unit on the same stations, breaking. more
No break below was chosen. more
One play head, two tracks, the same scale. The clean run stops when it runs out of stations; the other one keeps going.
As written
6 stations, nothing waiting, nothing sent back
0 min of 9.6 hrs
now Static analysis · Code Quality Orchestrator · 6 min
As the ledger has it
the same stations, plus 5 categories the estate already counts
0 min of 40 hrs
now Static analysis · Code Quality Orchestrator · 6 min
What breaks, and where it is counted
No break below was chosen. more
What the two runs say together
5 counted categories land on this procedure. more
What this run cannot claim
These breaks are shown together on one unit so the shape of a bad day fits on one screen. more
The separation register records a duty conflict against a sub-function and a named post, not against one procedure. more
Two clocks run. Worked time is the sum of the time the work is actually being done … more
Live observability
Systems this service would touch
Attributed by rule from the function's system list. more
2 systems attributed
0 of 2 returning records
Group ERP — Materials and Inventory
SAP S/4HANA MM · supply · bidirectional · high
5 of 6 stations attributed here · would write back
Configuration Management Database
ServiceNow CMDB · itsm · inbound · high
1 of 6 stations attributed here · read only
System attribution is a rule, not a reading. more
Who runs it
Orchestration and accountability
- Orchestrated by
- Code Quality Orchestrator
- That orchestrator’s remit
- Owns pull request work end to end inside Engineering. Runs 3 lines across 14 stations, drains the repeatable share into no shared tower, and holds 5 human gates.
- Accountable person
- Director, Engineering Quality
- Sub-function
- Code Review & Quality
- Shared spine tower
- Not on the shared spine
- Hand-over points
- None — this service stays inside its function
Other services in this sub-function
2 alongside this one
What this page is and is not
This is a catalogue of the modeled estate. more