Skip to content

chore(tra-963): remove dead Redpanda Connect ingester source artifacts#474

Merged
mikestankavich merged 2 commits into
mainfrom
chore/tra-963-remove-redpanda-ingester-source
Jun 9, 2026
Merged

chore(tra-963): remove dead Redpanda Connect ingester source artifacts#474
mikestankavich merged 2 commits into
mainfrom
chore/tra-963-remove-redpanda-ingester-source

Conversation

@mikestankavich

Copy link
Copy Markdown
Contributor

Closes TRA-963. Follow-up to TRA-907 (RC runtime already decommissioned in infra — helm release removed, GKE + preview on the in-backend Go subscriber, Railway dead). This deletes the leftover RC source-tree artifacts.

Removed (dead RC runtime)

  • ingester/connect.yaml — RC pipeline config (MQTT → raw INSERT into tag_scans)
  • ingester/justfile — RC docker-compose recipes
  • ingester/acceptance/capture-cs463.sh — sourced the now-dead EMQX Cloud (TRA-828); no longer runnable
  • docker-compose.yaml — the ingester: redpanda-connect service
  • root justfile — the ingester delegate recipe + alias ing

Relocated & kept (NOT dead)

The CS463 broker-acceptance deck (TRA-834) is not RC-specific — it mosquitto_pubs a corpus and asserts rows land in tag_scans, so it validates whatever is subscribed. It now exercises the in-backend Go subscriber (TRA-900) unchanged. The corpus (cs463.tsv, 2521 real messages / 4 capture points / 71 EPCs) was captured from EMQX Cloud and is not recapturable.

  • moved ingester/acceptance/{replay-cs463.sh,README.md,corpus/cs463.tsv}test/acceptance/cs463-replay/
  • reworded README + script comments off "Redpanda Connect" → "Go subscriber"; fixed default CORPUS path

Deliberately untouched

  • backend/internal/ingest/*, backend/internal/cmd/serve/serve.go — the live Go path (only historical comments mention RC)
  • spec/, docs/superpowers/ — historical record
  • docs/architecture-decisions.md — a frozen historical planning snapshot (still lists "Railway initially", "Defer RLS", etc.); selectively patching one stale RC line would make it less coherent, so left as-is

Verification

  • just --list parses; ingester recipe/alias gone, all other recipes intact
  • docker compose config -q exits 0; no ingester service remains
  • grep -rn ingester|redpanda over the active tree (excl. historical + live Go path) is clean
  • relocated corpus intact (2521 lines)

Internal-only, no API/docs surface → no docs PR.

🤖 Generated with Claude Code

The RC ingester runtime was decommissioned in TRA-907 (helm release removed,
GKE + preview on the in-backend Go subscriber, Railway dead). This removes the
now-dead source-tree artifacts:

- delete ingester/connect.yaml, ingester/justfile (RC config + recipes)
- delete ingester/acceptance/capture-cs463.sh (sourced the dead EMQX Cloud)
- drop the redpanda-connect `ingester` service from docker-compose.yaml
- drop the `ingester` delegate recipe + `ing` alias from the root justfile

Preserve the still-useful CS463 broker-acceptance deck (TRA-834): it publishes a
frozen, non-recapturable corpus over MQTT and asserts rows land in tag_scans, so
it now validates the in-backend Go subscriber unchanged. Relocated to
test/acceptance/cs463-replay/ with README/comments reworded off Redpanda Connect.

docs/architecture-decisions.md left as-is: it's a frozen historical planning
snapshot (still lists Railway / deferred-RLS), not living docs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

🚀 Preview Deployment Update

✅ This PR has been successfully merged into the preview branch.

The preview environment will update shortly at: https://app.preview.trakrf.id

trakrf-preview-sync Bot pushed a commit that referenced this pull request Jun 9, 2026
trakrf-preview-sync Bot pushed a commit that referenced this pull request Jun 9, 2026
@mikestankavich mikestankavich merged commit 3fe04ae into main Jun 9, 2026
14 checks passed
@mikestankavich mikestankavich deleted the chore/tra-963-remove-redpanda-ingester-source branch June 9, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant