marcel

0 Followers · 0 Following

• Joined on 2026-03-17

Repositories

3

Projects Packages Public Activity Starred Repositories

marcel deleted branch fix/issue-601-obs-stack-permanent from marcel/familienarchiv 2026-05-16 09:46:16 +02:00

marcel pushed to main at marcel/familienarchiv 2026-05-16 09:46:14 +02:00

3658733003 fix(obs): add GlitchTip healthcheck on /_health/ (port 8080)

0bb0a314ad ci(obs): add obs-glitchtip to health assertion loop (now has /_health/ healthcheck)

b194b565f6 ci(obs): reference #603 in keep-in-sync comments; add obs-glitchtip to health assertion

6720a5aeb2 chore(obs): improve deploy maintainability from review feedback

a7f60ebed8 docs(obs): add cp-r stale-file cleanup note to DEPLOYMENT.md

Compare 27 commits »

marcel closed issue marcel/familienarchiv#601 2026-05-16 09:46:14 +02:00

devops: move observability stack out of CI workspace into /opt/familienarchiv/

marcel merged pull request marcel/familienarchiv#602 2026-05-16 09:46:12 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:44:30 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🎨 Leonie Voss (@leonievoss) — UX Designer & Accessibility Strategist

Verdict: ✅ Approved

No UI changes in this PR. Every touched file is infrastructure:

• `.gitea/workflows/nightly.yml…

marcel closed issue marcel/familienarchiv#318 2026-05-16 09:44:27 +02:00

feat(mobile): reader surfaces (Home · /documents · /briefwechsel · /persons) pass mobile-first bar at 375 px

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:44:26 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🧪 Sara Holt (@saraholt) — QA Engineer

Verdict: ✅ Approved

No application test code to review. Evaluating the CI verification additions.

What was added for verification

**`Validate…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:44:13 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

📋 Elicit — Requirements Engineer

Verdict: ✅ Approved

Reviewing in Brownfield mode — auditing the PR against issue #601's stated requirements.

PR description quality

Excellent.…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:44:03 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🏛️ Markus Keller (@mkeller) — Application Architect

Verdict: ✅ Approved

ADR quality

ADR-016 is well-constructed. The context explains the specific failure mode (workspace wipe…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:43:52 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🔒 Nora "NullX" Steiner — Application Security Engineer

Verdict: ✅ Approved

What improved

Secrets out of CI workspace files — significant improvement. Previously, `GRAFANA_ADMI…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:43:37 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

👨‍💻 Felix Brandt (@felixbrandt) — Senior Fullstack Developer

Verdict: ✅ Approved

Pure infrastructure — no Java, TypeScript, or Svelte touched. Reviewing the CI scripts on their own…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:43:26 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🔧 Tobias Wendt (@tobiwendt) — DevOps & Platform Engineer

Verdict: ✅ Approved

Well-engineered. Three immediate wins stand out right away:

1. Secrets out of CI env heredocs —…

marcel pushed to fix/issue-601-obs-stack-permanent at marcel/familienarchiv 2026-05-16 09:37:20 +02:00

3658733003 fix(obs): add GlitchTip healthcheck on /_health/ (port 8080)

marcel pushed to fix/issue-601-obs-stack-permanent at marcel/familienarchiv 2026-05-16 09:36:41 +02:00

0bb0a314ad ci(obs): add obs-glitchtip to health assertion loop (now has /_health/ healthcheck)

marcel pushed to fix/issue-601-obs-stack-permanent at marcel/familienarchiv 2026-05-16 09:35:46 +02:00

b194b565f6 ci(obs): reference #603 in keep-in-sync comments; add obs-glitchtip to health assertion

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:26:17 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🎨 Leonie Voss — UI/UX Designer & Accessibility Strategist

Verdict: ✅ Approved — LGTM

This PR contains zero frontend changes. No Svelte components, no routes, no CSS, no ARIA…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:26:12 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

📋 Elicit — Requirements Engineer

Verdict: ✅ Approved

This is a pure infrastructure bug-fix PR with no product-facing requirements implications. The scope is well-contained: three…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:25:59 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🧪 Sara Holt — Senior QA Engineer

Verdict: ⚠️ Approved with concerns

What's good

The PR adds two genuine quality gates that weren't there before:

1. **"Validate observability…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:25:43 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🔐 Nora "NullX" Steiner — Application Security Engineer

Verdict: ⚠️ Approved with concerns

No exploitable vulnerabilities in this PR. The secrets handling is well-designed. My concerns…

marcel commented on pull request marcel/familienarchiv#602 2026-05-16 09:25:24 +02:00

fix(obs): Tempo processors schema fix + configurable Postgres host and Grafana port

🔧 Tobias Wendt — DevOps & Platform Engineer

Verdict: ⚠️ Approved with concerns

This fixes three real production issues cleanly. The overall approach is sound. I have one blocker (CI…