Adservio Code Review Analysis
A read-only analysis of recent merge request review behavior in GitLab, focused on visible review discussion, reviewer distribution, queue health, MR size, and traceability gaps.
Executive Summary
Bottom line: review throughput is not the main risk here; traceability and queue hygiene are. The team is still merging code, but most merges leave no durable visible review trail and the open queue is overwhelmingly stale.
What Matters
- Traceability is the clearest control gap: 72 of 87 merged MRs (82.8%) showed no visible non-author review comment in GitLab.
- `CR - Approved` is currently too weak as evidence on its own: 33.3% of merged MRs carried that label without any visible review discussion.
- Queue health is poor: 77.6% of active open MRs are already older than 48 working hours. The oldest example is adservio/web!2091 at 241.5d open.
- Visible review capacity is thin and concentrated: only 18 visible review touches were detected in the whole window, and the top 3 reviewers handled 83.3% of them.
- adservio/frontend is the highest-volume review surface with 23 merged MRs and only 30.4% visible review coverage. adservio/web is the next meaningful flow at 13 merges and 7.7% visible review coverage.
- MR size is not the main story: median changed files were 3, and only 16.1% of merged MRs were above 20 files. Even the slowest merged example, adservio/library/backend!27, took 82.0d with 7 changed files, so the dominant problem still looks like review discipline and queue ownership rather than consistently oversized MRs.
Leadership Analysis
- The operating rhythm is serviceable at the median (11.4h created-to-merged), which means the biggest leverage is improving process fidelity rather than pushing for even more raw throughput.
- With 77.6% of active MRs already stale, the open list is functioning more like inventory than prioritization support. adservio/web!2091 being open for 241.5d is a strong sign that ownership and review handoffs are not being actively reset.
- Reviewer concentration is now a resilience issue, not just a fairness issue. When the top 3 reviewers carry 83.3% of visible touches, review quality, continuity, and queue health all become sensitive to a very small set of people.
- The spread between a 3-file median MR and a 82.0d longest merged cycle (adservio/library/backend!27, 7 changed files) suggests a few ownership-heavy exceptions are dragging on outside normal flow. Those should be managed as explicit escalations, not left to age inside the common queue.
Recommended Actions
- Make one durable review artifact mandatory before merge: a substantive non-author GitLab review comment, a recorded approval, or both. A label alone should no longer count as sufficient evidence. The approval API coverage is also low, so the fix should include tooling or policy that leaves an auditable trace by default.
- Run a stale-MR reset on anything older than 48 working hours: close abandoned work, re-confirm owner and reviewer on the rest, and turn the open list back into a real queue instead of a parking lot. Start with adservio/web!2091 and the rest of the oldest backlog because their age now obscures whether they are still real priorities.
- Put adservio/helm2 under explicit review guardrails for the next cycle. It combines enough volume with weak visible review coverage, which makes it the best place to tighten the process and see impact quickly.
- Broaden the reviewer bench beyond Dragoș Ivan and the current small reviewer set. The present concentration is a continuity risk and will keep queue health fragile when one person is away.
Improvement Ideas
- Set explicit operating standards for the next quarter: require one auditable peer-review artifact before merge, target first review inside one working day for active MRs, and keep the stale active backlog below 15%. What gets measured this clearly is far more likely to improve.
- Run a four-week process experiment on adservio/helm2: dedicated reviewer-of-the-week coverage, explicit reviewer assignment at MR creation, and a lightweight weekly audit of silent merges. It is the highest-yield place to test whether better ownership changes the data.
- Create a reviewer rotation with named backup coverage for the busiest repos. Good review systems depend on redundancy, not heroics, and rotation is the fastest way to lower reviewer bus factor without slowing delivery.
Supporting Evidence
- 72 of 87 merged MRs (82.8%) had no visible non-author review comment in GitLab.
- 41 of 87 merged MRs (47.1%) carried the `CR - Approved` label.
- 29 merged MRs (33.3%) were labeled `CR - Approved` without any visible non-author review comment.
- 16.1% of merged MRs changed more than 20 files, while 4.6% changed more than 50 files.
- Among MRs with visible review discussion, the median last-author-commit to first-review time was 23.1h.
- The median last-author-commit to `CR - Approved` time was 45.2h.
- The top 3 reviewers accounted for 83.3% of visible review touches.
- 38 of 49 active open MRs (77.6%) were older than 48 working hours.
Merged MRs Merged MRs inside the analysis window. | 87 |
Open MRs All currently open group MRs. | 58 |
Active open MRs Open MRs excluding drafts. | 49 |
Draft open MRs Open MRs marked draft/WIP. | 9 |
Visible review rate Merged MRs with at least one visible non-author GitLab comment. | 17.2% |
Merged without visible review comment Merged MRs with zero visible non-author comments. | 82.8% |
CR - Approved label coverage Merged MRs carrying the CR-approved label. | 47.1% |
Approval API coverage Merged MRs with at least one approver in the GitLab approvals API. | 12.6% |
CR - Approved without visible review comment CR-approved MRs with no visible non-author comment. | 33.3% |
Median MR created -> first review Working time from MR creation to first visible non-author comment. | 2.2d |
Median last author commit -> first review Working time from last author commit before review to first visible non-author comment. | 23.1h |
Median first review -> CR - Approved Working time from first visible comment to CR-approved label after review start, else merge. | 1.9h |
Median MR created -> CR - Approved Working time from MR creation to CR-approved label after review start, else merge. | 3.1d |
Median last author commit -> CR - Approved Working time from last author commit before review to CR-approved label after review start, else merge. | 45.2h |
Median MR created -> merged Working time from MR creation to merge. | 11.4h |
Median changed files Median GitLab `changes_count` value. | 3 |
MRs > 5 files Merged MRs where `changes_count` is above 5. | 40.2% |
MRs > 10 files Merged MRs where `changes_count` is above 10. | 26.4% |
MRs > 20 files Merged MRs where `changes_count` is above 20. | 16.1% |
MRs > 50 files Merged MRs where `changes_count` is above 50. | 4.6% |
Top 3 reviewer touch share Share of visible review touches handled by the top 3 reviewers. | 83.3% |
Active open backlog rate Non-draft open MRs older than 48 working hours. | 77.6% |
Timing Breakdown
These headline timing medians bring back the anchor-to-anchor numbers directly, using working time with weekends excluded.
| Median MR created -> first review | 2.2d |
| Median last author commit -> first review | 23.1h |
| Median MR created -> CR - Approved | 3.1d |
| Median first review -> CR - Approved | 1.9h |
| Median last author commit -> CR - Approved | 45.2h |
| Median MR created -> merged | 11.4h |
Reviewer Distribution
| Dragoș Ivan | 10 | 39 | 55.6% |
| Edgar Alexa | 3 | 15 | 16.7% |
| George Murgoci | 2 | 2 | 11.1% |
| Daniel Onisoru | 2 | 6 | 11.1% |
| Ioana Ivan | 1 | 14 | 5.6% |
Repo Breakdown
| adservio/frontend | 23 | 30.4% | 78.3% | 4 |
| adservio/web | 13 | 7.7% | 53.8% | 1 |
| adservio/devops/iac | 12 | 8.3% | 8.3% | 4.5 |
| adservio/helm2 | 12 | 0.0% | 0.0% | 1.0 |
| adservio/docgen2/backend | 6 | 16.7% | 66.7% | 5.0 |
| adservio/uni/backend | 6 | 0.0% | 33.3% | 21.0 |
| adservio/admission/backend | 5 | 40.0% | 80.0% | 8 |
| adservio/billing/backend | 3 | 33.3% | 33.3% | 2 |
| adservio/services/backend-for-frontend | 3 | 33.3% | 66.7% | 2 |
| adservio/accommodations/backend | 1 | 0.0% | 0.0% | 1 |
| adservio/library/backend | 1 | 0.0% | 0.0% | 7 |
| adservio/social | 1 | 100.0% | 100.0% | 25 |
Visible Review Discussion MRs
Compact by default: key CR metrics stay visible, and each row expands inline for the full audit trail. The table starts sorted by Last Commit -> First Review.
| Details | ||||||
|---|---|---|---|---|---|---|
| adservio/docgen2/backend!137 | [ADS-7905] Pentru clasele primare, catalogul PDF trebuie sa fie conform EDU | 2/5 Light | 0.9h | 0.4h | 2.8h | |
Repo adservio/docgen2/backend Author Petronel Pavel Reviewers Dragoș Ivan Review Comments 1 Quality Notes 1 visible comment; 1 substantive comment; single reviewer Substantive Comments 1 Reviewer Count 1 Post-Review Pushes 0 MR Created 2026-06-16 11:57 Last Author Commit 2026-06-16 12:29 First Review 2026-06-16 12:52 CR Approved 2026-06-16 14:44 Merged 2026-06-16 14:44 First Review -> CR Approved 1.9h Last Commit -> CR Approved 2.2h Created -> Merged 2.8h | ||||||
| adservio/devops/iac!37 | ADS-7820: private storage buckets + web-s3-user IAM in dev/staging/prod | 3/5 Solid | 2.3h | 2.3h | 4.3d | |
Repo adservio/devops/iac Author Rojan Shrestha Reviewers Daniel Onisoru Review Comments 1 Quality Notes 1 visible comment; 1 substantive comment; single reviewer; 5 post-review pushes Substantive Comments 1 Reviewer Count 1 Post-Review Pushes 5 MR Created 2026-06-09 11:30 Last Author Commit 2026-06-09 11:29 First Review 2026-06-09 13:49 CR Approved n/a Merged 2026-06-15 18:13 First Review -> CR Approved 4.2d Last Commit -> CR Approved 4.3d Created -> Merged 4.3d | ||||||
| adservio/frontend!1924 | [ADS-7838] Modificari serii -adaugare dropdown Tip Invatamant | 2/5 Light | 2.8h | 2.8h | 0.8h | |
Repo adservio/frontend Author Achim Stefan Reviewers George Murgoci Review Comments 1 Quality Notes 1 visible comment; 1 substantive comment; single reviewer Substantive Comments 1 Reviewer Count 1 Post-Review Pushes 0 MR Created 2026-06-15 10:03 Last Author Commit 2026-06-15 10:01 First Review 2026-06-15 12:50 CR Approved n/a Merged 2026-06-15 10:48 First Review -> CR Approved 0.0h Last Commit -> CR Approved 0.8h Created -> Merged 0.8h | ||||||
| adservio/frontend!1933 | [ADS-7638] Close situation” option is incorrectly visible to teachers when the... | 3/5 Solid | 4.6h | 4.9h | 5.3h | |
Repo adservio/frontend Author Petronel Pavel Reviewers Dragoș Ivan Review Comments 1 Quality Notes 1 visible comment; 1 substantive comment; single reviewer; 1 post-review push Substantive Comments 1 Reviewer Count 1 Post-Review Pushes 1 MR Created 2026-06-17 10:11 Last Author Commit 2026-06-17 09:56 First Review 2026-06-17 14:48 CR Approved 2026-06-17 15:31 Merged 2026-06-17 15:32 First Review -> CR Approved 0.7h Last Commit -> CR Approved 5.6h Created -> Merged 5.3h | ||||||
| adservio/admission/backend!154 | [ADS-7428] - needs modifications flow fixes | 3/5 Solid | 3.1d | 16.7h | 3.1d | |
Repo adservio/admission/backend Author George Murgoci Reviewers Edgar Alexa Review Comments 2 Quality Notes 2 visible comments; 2 substantive comments; single reviewer; 3 post-review pushes Substantive Comments 2 Reviewer Count 1 Post-Review Pushes 3 MR Created 2026-06-08 08:25 Last Author Commit 2026-06-10 16:56 First Review 2026-06-11 09:40 CR Approved 2026-06-11 11:05 Merged 2026-06-15 10:39 First Review -> CR Approved 1.4h Last Commit -> CR Approved 18.1h Created -> Merged 5.1d | ||||||
| adservio/frontend!1900 | [ADS-7428] - needs modifications flow fixes | 3/5 Solid | 3.1d | 18.1h | 3.1d | |
Repo adservio/frontend Author George Murgoci Reviewers Edgar Alexa Review Comments 2 Quality Notes 2 visible comments; 2 substantive comments; single reviewer; 8 post-review pushes Substantive Comments 2 Reviewer Count 1 Post-Review Pushes 8 MR Created 2026-06-08 08:26 Last Author Commit 2026-06-10 16:50 First Review 2026-06-11 10:57 CR Approved 2026-06-11 11:46 Merged 2026-06-15 10:39 First Review -> CR Approved 0.8h Last Commit -> CR Approved 18.9h Created -> Merged 5.1d | ||||||
| adservio/frontend!1921 | [ADS-7239] - Session Labels | 4/5 Strong | 2.1d | 20.1h | 3.0d | |
Repo adservio/frontend Author George Murgoci Reviewers Dragoș Ivan Review Comments 12 Quality Notes 12 visible comments; 10 substantive comments; single reviewer; 4 post-review pushes; multi-point discussion Substantive Comments 10 Reviewer Count 1 Post-Review Pushes 4 MR Created 2026-06-12 11:07 Last Author Commit 2026-06-15 16:54 First Review 2026-06-16 12:58 CR Approved 2026-06-17 11:49 Merged 2026-06-17 13:50 First Review -> CR Approved 22.8h Last Commit -> CR Approved 42.9h Created -> Merged 3.1d | ||||||
| adservio/admission/backend!156 | [ADS-7239] - labels | 4/5 Strong | 2.2d | 23.1h | 3.1d | |
Repo adservio/admission/backend Author George Murgoci Reviewers Dragoș Ivan Review Comments 5 Quality Notes 5 visible comments; 3 substantive comments; single reviewer; 2 post-review pushes; multi-point discussion Substantive Comments 3 Reviewer Count 1 Post-Review Pushes 2 MR Created 2026-06-12 11:07 Last Author Commit 2026-06-15 16:41 First Review 2026-06-16 15:44 CR Approved 2026-06-17 13:50 Merged 2026-06-17 13:50 First Review -> CR Approved 22.1h Last Commit -> CR Approved 45.2h Created -> Merged 3.1d | ||||||
| adservio/social!28 | [ADS-7519][BE] "Afișează noutăți Adservio" toggle is non-functional | 2/5 Light | 24.8h | 24.3h | 24.9h | |
Repo adservio/social Author Alex Murarescu Reviewers Dragoș Ivan Review Comments 1 Quality Notes 1 visible comment; comments look lightweight; single reviewer; 1 post-review push; large MR (25 files) with little visible depth Substantive Comments 0 Reviewer Count 1 Post-Review Pushes 1 MR Created 2026-06-16 15:08 Last Author Commit 2026-06-16 15:38 First Review 2026-06-17 15:57 CR Approved 2026-06-17 16:01 Merged 2026-06-18 11:10 First Review -> CR Approved 0.1h Last Commit -> CR Approved 24.4h Created -> Merged 44.0h | ||||||
| adservio/web!2576 | [ADS-7519][BE] "Afișează noutăți Adservio" toggle is non-functional | 3/5 Solid | 24.9h | 29.8h | 44.0h | |
Repo adservio/web Author Alex Murarescu Reviewers Dragoș Ivan Review Comments 2 Quality Notes 2 visible comments; 2 substantive comments; single reviewer; 1 post-review push Substantive Comments 2 Reviewer Count 1 Post-Review Pushes 1 MR Created 2026-06-16 15:09 Last Author Commit 2026-06-16 10:15 First Review 2026-06-17 16:03 CR Approved 2026-06-18 11:10 Merged 2026-06-18 11:10 First Review -> CR Approved 19.1h Last Commit -> CR Approved 2.0d Created -> Merged 44.0h | ||||||
| adservio/billing/backend!244 | [ADS-1757] [BE] [Financiar] - Plati partiale | 4/5 Strong | 23.7d | 3.0d | 39.8d | |
Repo adservio/billing/backend Author Edgar Alexa Reviewers Daniel Onisoru Review Comments 5 Quality Notes 5 visible comments; 3 substantive comments; single reviewer; 10 post-review pushes; multi-point discussion Substantive Comments 3 Reviewer Count 1 Post-Review Pushes 10 MR Created 2026-04-21 17:40 Last Author Commit 2026-05-20 09:38 First Review 2026-05-25 09:29 CR Approved 2026-06-16 11:55 Merged 2026-06-16 11:58 First Review -> CR Approved 16.1d Last Commit -> CR Approved 19.1d Created -> Merged 39.8d | ||||||
| adservio/services/backend-for-frontend!1 | BFF init | 4/5 Strong | 4.6d | 3.5d | 7.5d | |
Repo adservio/services/backend-for-frontend Author Daniel Onisoru Reviewers Dragoș Ivan, George Murgoci Review Comments 12 Quality Notes 12 visible comments; 8 substantive comments; 2 reviewers; multi-point discussion Substantive Comments 8 Reviewer Count 2 Post-Review Pushes 0 MR Created 2026-06-08 23:19 Last Author Commit 2026-06-10 00:27 First Review 2026-06-15 13:22 CR Approved 2026-06-18 11:09 Merged 2026-06-18 11:10 First Review -> CR Approved 2.9d Last Commit -> CR Approved 6.4d Created -> Merged 7.5d | ||||||
| adservio/frontend!1909 | [ADS-7009]: Do not allow add/edit/delete grade if situation is closed. | 2/5 Light | 4.9d | 4.8d | 4.9d | |
Repo adservio/frontend Author Ioana Ivan Reviewers Dragoș Ivan Review Comments 2 Quality Notes 2 visible comments; 1 substantive comment; single reviewer Substantive Comments 1 Reviewer Count 1 Post-Review Pushes 0 MR Created 2026-06-09 15:38 Last Author Commit 2026-06-09 16:25 First Review 2026-06-16 12:46 CR Approved 2026-06-16 14:07 Merged 2026-06-16 14:07 First Review -> CR Approved 1.4h Last Commit -> CR Approved 4.9d Created -> Merged 4.9d | ||||||
| adservio/frontend!1907 | [ADS-7842]: Add route for View memento for mobile | 2/5 Light | 7.2d | 7.2d | 7.2d | |
Repo adservio/frontend Author Ioana Ivan Reviewers Dragoș Ivan Review Comments 1 Quality Notes 1 visible comment; 1 substantive comment; single reviewer Substantive Comments 1 Reviewer Count 1 Post-Review Pushes 0 MR Created 2026-06-09 11:09 Last Author Commit 2026-06-09 11:07 First Review 2026-06-18 15:39 CR Approved n/a Merged 2026-06-18 15:48 First Review -> CR Approved 0.2h Last Commit -> CR Approved 7.2d Created -> Merged 7.2d | ||||||
| adservio/frontend!1724 | [S: ADS-1757][ADS-7256] [BE - PHP + FE]: Plăți Parțiale | 4/5 Strong | 26.3d | 12.2d | 44.1d | |
Repo adservio/frontend Author Catalin Cojan Reviewers Ioana Ivan, Dragoș Ivan, Edgar Alexa Review Comments 28 Quality Notes 28 visible comments; 13 substantive comments; 3 reviewers; multi-point discussion Substantive Comments 13 Reviewer Count 3 Post-Review Pushes 0 MR Created 2026-04-15 10:10 Last Author Commit 2026-05-05 13:42 First Review 2026-05-21 17:30 CR Approved 2026-06-16 11:54 Merged 2026-06-16 11:58 First Review -> CR Approved 17.8d Last Commit -> CR Approved 29.9d Created -> Merged 44.1d | ||||||
No Visible Review Comment Audit
These merged MRs had no visible non-author GitLab comment under the current heuristic. Use this section to audit whether the gap is a real review-process issue or a GitLab traceability issue.
Total MRs72
With CR Label29
With Approval API9
- Open the MR link and check the Overview tab for
CR - Approvedand any approval widget signal. - Check Activity / Discussions for real human reviewer comments.
- If you only see system notes, merge notes, mentions, or bot activity, it stays in this audit set.
- If you find a genuine reviewer comment from someone other than the author, treat that MR as a false positive of this heuristic.
full audit table (72 MRs). Click again to collapse.
| adservio/docgen2/backend!136 | adservio/docgen2/backend | [ADS-7838] Modificari serii -adaugare dropdown Tip Invatamant | Achim Stefan | 2026-06-15 | ||
| adservio/uni/backend!213 | adservio/uni/backend | [ADS-7838] Nomenclator Forma de Invatamant | Andrei Alexandru | 2026-06-15 | Yes | |
| adservio/admission/backend!158 | adservio/admission/backend | fixed migration | George Murgoci | 2026-06-15 | ||
| adservio/frontend!1925 | adservio/frontend | [ADS-7243] - hide auto populate field if grading sistem is admitted/rejected | George Murgoci | 2026-06-15 | Yes | |
| adservio/billing/backend!252 | adservio/billing/backend | [ads-7284-tests] Fixed failing test | Edgar Alexa | 2026-06-15 | ||
| adservio/docgen2/backend!135 | adservio/docgen2/backend | [ADS-7870] Numarul matricol se suprapune peste numele elevului in Catalogul PDF | Petronel Pavel | 2026-06-15 | Yes | |
| adservio/frontend!1918 | adservio/frontend | [ADS-7062] Calcul si afisare total pentru statistici elevi | Petronel Pavel | 2026-06-15 | Yes | |
| adservio/web!2571 | adservio/web | [ADS-7860] Campul "titulatura" trebuie sa fie optional la introducerea personalului scolar | Petronel Pavel | 2026-06-15 | Yes | |
| adservio/web!2573 | adservio/web | [ADS-7902] Refactoring top elevi | Marian Andrei | 2026-06-15 | ||
| adservio/devops/iac!40 | adservio/devops/iac | Update dev permissions | Rojan Shrestha | 2026-06-15 | ||
| adservio/devops/iac!42 | adservio/devops/iac | ADS-7894: stop backing up EKS EBS volumes (enable_backup_tag=false on prod... | Rojan Shrestha | 2026-06-15 | Yes | Yes |
| adservio/frontend!1889 | adservio/frontend | [ADS-7475]: Plan de Invatamant (FD) - teste e2e | Dana Mihaela Maritca | 2026-06-16 | Yes | |
| adservio/devops/iac!45 | adservio/devops/iac | make adservio-dev-tmp bucket publicly readable | Rojan Shrestha | 2026-06-16 | ||
| adservio/helm2!275 | adservio/helm2 | social-api: enable HPA min 2 and add PDB in production | Rojan Shrestha | 2026-06-16 | ||
| adservio/subscriptions/backend!112 | adservio/subscriptions/backend | [ADS-1757] Plati partiale | Edgar Alexa | 2026-06-16 | Yes | |
| adservio/web!2449 | adservio/web | [S: ADS-1757][ADS-7256] [Financiar] - Plati partiale [BE - PHP + FE]: Notificări web | Catalin Cojan | 2026-06-16 | Yes | |
| adservio/helm2!280 | adservio/helm2 | social-api: set cpu request to 100m so KEDA CPU trigger admission passes | Rojan Shrestha | 2026-06-16 | ||
| adservio/devops/iac!46 | adservio/devops/iac | rename cdn buckets to short-form naming | Rojan Shrestha | 2026-06-16 | ||
| adservio/web!2555 | adservio/web | [ADS-7009]: check if situation is closed before interacting with grades | Ioana Ivan | 2026-06-16 | Yes | Yes |
| adservio/frontend!1930 | adservio/frontend | [ADS-7274] Implementare afisare numar matricol pentru rolurile de elev si parinte | Petronel Pavel | 2026-06-16 | Yes | |
| adservio/devops/iac!47 | adservio/devops/iac | add tuiasi cdn on prod and tmp buckets on prod and staging | Rojan Shrestha | 2026-06-16 | ||
| adservio/uni/backend!215 | adservio/uni/backend | [ADS-7910] Refactor - Catalog - Save grades | Andrei Alexandru | 2026-06-16 | Yes | |
| adservio/frontend!1931 | adservio/frontend | [ads-17570-tests] updated tests | Edgar Alexa | 2026-06-16 | ||
| adservio/docgen2/backend!125 | adservio/docgen2/backend | [ADS-7375]: Suport multi-lang pentru discipline | Dana Mihaela Maritca | 2026-06-16 | Yes | |
| adservio/frontend!1861 | adservio/frontend | [ADS-7375]: Suport multi-lang pentru discipline | Dana Mihaela Maritca | 2026-06-16 | Yes | |
| adservio/uni/backend!195 | adservio/uni/backend | ADS-7375 Disciplinele in EN pentru programele de studiu din EN | Francisc Gabriel Virlan | 2026-06-16 | Yes | Yes |
| adservio/helm2!281 | adservio/helm2 | social-api: bump memory request to 640Mi to stop HPA memory-trigger maxing out | Rojan Shrestha | 2026-06-16 | ||
| adservio/devops/iac!43 | adservio/devops/iac | production: add mysql-catalog3 (db.r8g.2xlarge, 80GB) and bump catalog2 to 2xlarge | Rojan Shrestha | 2026-06-16 | ||
| adservio/devops/iac!44 | adservio/devops/iac | set default RDS maintenance window to Sat 19:00-20:00 UTC | Rojan Shrestha | 2026-06-16 | ||
| adservio/helm2!282 | adservio/helm2 | social-api: bump maxrepl from 4 to 8 for school-hours headroom | Rojan Shrestha | 2026-06-16 | ||
| adservio/devops/iac!41 | adservio/devops/iac | feat: add support for persistent /data EBS volume with configurable size | Daniel Onisoru | 2026-06-16 | ||
| adservio/devops/iac!48 | adservio/devops/iac | lower db_connections_threshold on prod RDS alarms | Rojan Shrestha | 2026-06-16 | ||
| adservio/frontend!1932 | adservio/frontend | Fix uni cohorts automation tests, testId education-form-sync-select missing | Achim Stefan | 2026-06-17 | Yes | |
| adservio/web!2579 | adservio/web | [ADS-7638] msm/mgm validation | Ionut Ciolan | 2026-06-17 | ||
| adservio/frontend!1936 | adservio/frontend | [ADS-7933]: Afisare corecta nume universitate la descarcarea catalogului | Dana Mihaela Maritca | 2026-06-17 | Yes | Yes |
| adservio/frontend!1927 | adservio/frontend | [ADS-6805] - duplicare sesiuni | George Murgoci | 2026-06-17 | Yes | |
| adservio/admission/backend!159 | adservio/admission/backend | [ADS-6805] - duplicare sesiuni | George Murgoci | 2026-06-17 | Yes | |
| adservio/frontend!1934 | adservio/frontend | [ADS-7915] Dirigintelui nu trebuie sa i se permita sa incheie media pe... | Petronel Pavel | 2026-06-17 | Yes | |
| adservio/frontend!1935 | adservio/frontend | [ADS-7930] Setare scutiri materie educatie fizica/Religie | Petronel Pavel | 2026-06-17 | Yes | |
| adservio/web!2581 | adservio/web | [HOTFIX] Activate history | Ionut Ciolan | 2026-06-17 | ||
| adservio/frontend!1928 | adservio/frontend | [ADS-7796] istoric note sterse de catre profesorul ID 2329864 | Petronel Pavel | 2026-06-17 | Yes | Yes |
| adservio/web!2578 | adservio/web | [ADS-7397] Corectarea afisarii claselor cu cifre romane in fisa scolara | Petronel Pavel | 2026-06-17 | Yes | |
| adservio/frontend!1940 | adservio/frontend | [ADS-7939] Incercarea de adaugare a unei suprascrieri in catalogul PDF... | Petronel Pavel | 2026-06-17 | Yes | |
| adservio/docgen2/backend!139 | adservio/docgen2/backend | [ADS-7934] Observatia individuala din avertismente marcata cu "Adauga in... | Petronel Pavel | 2026-06-17 | Yes | |
| adservio/uni/backend!209 | adservio/uni/backend | feat: bootstrap Laravel uni-core component and nginx into the pipeline | Daniel Onisoru | 2026-06-17 | ||
| adservio/helm2!274 | adservio/helm2 | feat(uni): add Laravel uni-core sidecars to uni-api pod | Daniel Onisoru | 2026-06-17 | ||
| adservio/helm2!283 | adservio/helm2 | alertmanager: route every severity=critical alert to PagerDuty | Rojan Shrestha | 2026-06-17 | ||
| adservio/helm2!285 | adservio/helm2 | Add --no-atomic to feature branch dev deploy to prevent rollback on slow rollout | Rojan Shrestha | 2026-06-18 | ||
| adservio/admission/backend!160 | adservio/admission/backend | [ADS-7885]: Session improvements | Dragoș Ivan | 2026-06-18 | Yes | |
| adservio/frontend!1941 | adservio/frontend | [ADS-7885]: Session Improvements | Dragoș Ivan | 2026-06-18 | Yes | |
| adservio/web!2583 | adservio/web | [ADS-7618] Fix chestionare | Marian Andrei | 2026-06-18 | ||
| adservio/web!2577 | adservio/web | [ADS-7913] Exclude non-statistical rows from promotion totals | Valentin Pal | 2026-06-18 | Yes | Yes |
| adservio/frontend!1942 | adservio/frontend | [ADS-7967] Incercarea de accesare "promovabilitate elevi" redirectioneaza... | Petronel Pavel | 2026-06-18 | ||
| adservio/web!2572 | adservio/web | [S: ADS-7896][ADS-7868] Migrare scrieri S3 in contul nou | Catalin Cojan | 2026-06-18 | Yes | |
| adservio/accommodations/backend!71 | adservio/accommodations/backend | [S: ADS-7896][ADS-7868] Migrare scrieri S3 in contul nou | Catalin Cojan | 2026-06-18 | ||
| adservio/billing/backend!253 | adservio/billing/backend | [ADS-7896] S3 Destination key | Edgar Alexa | 2026-06-18 | ||
| adservio/frontend!1937 | adservio/frontend | [S: ADS-7896][ADS-7868] Migrare scrieri S3 in contul nou | Catalin Cojan | 2026-06-18 | ||
| adservio/helm2!278 | adservio/helm2 | [S: ADS-7896][ADS-7868] Migrare scrieri S3 in contul nou | Catalin Cojan | 2026-06-18 | ||
| adservio/helm2!272 | adservio/helm2 | BFF Setup | Daniel Onisoru | 2026-06-18 | ||
| adservio/uni/backend!201 | adservio/uni/backend | [S: ADS-7496][ADS-7601] Situație școlară - Medii Examen finalizare | Andrei Alexandru | 2026-06-18 | Yes | |
| adservio/uni/backend!207 | adservio/uni/backend | [ADS-7534] Generare Adseverinte - Selectare Studenți | Andrei Alexandru | 2026-06-18 | ||
| adservio/library/backend!27 | adservio/library/backend | [ADS-6795] Added support for audit | Edgar Alexa | 2026-06-18 | ||
| adservio/web!2584 | adservio/web | [hotfix-yearbook] Change from anuar to yearbook | Catalin Cojan | 2026-06-18 | ||
| adservio/helm2!288 | adservio/helm2 | billing: remove null global.nodeSelector to fix helm merge conflict | Rojan Shrestha | 2026-06-18 | ||
| adservio/helm2!287 | adservio/helm2 | docgen2: fix HPA over-scaling | Rojan Shrestha | 2026-06-18 | ||
| adservio/helm2!289 | adservio/helm2 | docgen2: bump cpu limit 2000m to 4000m to stop throttling | Rojan Shrestha | 2026-06-19 | ||
| adservio/docgen2/backend!141 | adservio/docgen2/backend | [ADS-7957]: Disciplinele din situatiile scolare apar in engleza | Dana Mihaela Maritca | 2026-06-19 | Yes | |
| adservio/services/backend-for-frontend!2 | adservio/services/backend-for-frontend | chore(logging): add error logging to calendar today handler upstream fetches | Stefan Musat | 2026-06-19 | ||
| adservio/devops/iac!49 | adservio/devops/iac | move adservio-web-user and sentry-user policies out of iam-access module | Rojan Shrestha | 2026-06-19 | ||
| adservio/devops/iac!50 | adservio/devops/iac | Add adservio-social-user with scoped S3 access to news/ prefix | Rojan Shrestha | 2026-06-19 | ||
| adservio/web!2587 | adservio/web | [S: ADS-7896][ADS-7987] Migrare documente S3 | Catalin Cojan | 2026-06-19 | ||
| adservio/services/backend-for-frontend!3 | adservio/services/backend-for-frontend | fix(auth): extract token in User decorator to forward it to upstream requests | Stefan Musat | 2026-06-19 | Yes |
Oldest Active Open MRs (> 48 Working Hours)
| adservio/web!2091 | adservio/web | Andrei Dragan | 241.5d | 5 | |
| adservio/helm2!137 | adservio/helm2 | Raul Popovici | 217.6d | 0 | |
| adservio/services/websocket!8 | adservio/services/websocket | Raul Popovici | 214.5d | 0 | |
| adservio/services/videothumb!5 | adservio/services/videothumb | Raul Popovici | 214.5d | 0 | |
| adservio/services/fet-generator!9 | adservio/services/fet-generator | Raul Popovici | 214.5d | 0 | |
| adservio/ai/backend!62 | adservio/ai/backend | Robert | 115.6d | 0 | |
| adservio/frontend!1556 | adservio/frontend | George Murgoci | 106.3d | 0 | |
| adservio/web!2344 | adservio/web | George Murgoci | 105.4d | 0 | |
| adservio/library/backend!26 | adservio/library/backend | Raul Popovici | 85.4d | 0 | |
| adservio/subscriptions/backend!103 | adservio/subscriptions/backend | Edgar Alexa | 83.4d | 4 |
Longest Merged Cycles
| adservio/library/backend!27 | adservio/library/backend | Edgar Alexa | 82.0d | 7 | 0 |
| adservio/web!2449 | adservio/web | Catalin Cojan | 44.1d | 8 | 0 |
| adservio/frontend!1724 | adservio/frontend | Catalin Cojan | 44.1d | 85 | 28 |
| adservio/billing/backend!244 | adservio/billing/backend | Edgar Alexa | 39.8d | 47 | 5 |
| adservio/subscriptions/backend!112 | adservio/subscriptions/backend | Edgar Alexa | 25.1d | 1 | 0 |
| adservio/uni/backend!195 | adservio/uni/backend | Francisc Gabriel Virlan | 21.2d | 45 | 0 |
| adservio/uni/backend!201 | adservio/uni/backend | Andrei Alexandru | 17.1d | 22 | 0 |
| adservio/frontend!1861 | adservio/frontend | Dana Mihaela Maritca | 15.1d | 29 | 0 |
| adservio/docgen2/backend!125 | adservio/docgen2/backend | Dana Mihaela Maritca | 15.1d | 15 | 0 |
| adservio/frontend!1889 | adservio/frontend | Dana Mihaela Maritca | 8.9d | 54 | 0 |
Methodology And Limits
- This report uses visible non-author GitLab comments as the strongest review-start proxy.
- GitLab approval API coverage is shown separately; missing approval API data should be read as traceability gap, not proof that no approval happened.
- Review completion uses the first `CR - Approved` label-add event after review starts when GitLab label history provides one; otherwise it falls back to merge time.
- Last-author-commit metrics use the latest non-merge MR commit authored by the MR author strictly before review starts.
- Timing metrics in this report exclude Saturday and Sunday hours using the Europe/Bucharest calendar.
- Review-quality scoring is heuristic: it rewards visible substantive comments, multiple reviewers, and post-review pushes, and it penalizes large MRs with shallow visible discussion.
- `Active open backlog rate` excludes draft MRs and uses a > 48 working-hour age threshold.
- Open MR backlog age is a queue-health signal, not proof that every old open MR is actively waiting for review right now.