Skip to content

fix: image and log banner say riptide-collector, not riptide#53

Merged
trick77 merged 1 commit into
masterfrom
fix/image-name-riptide-collector
May 17, 2026
Merged

fix: image and log banner say riptide-collector, not riptide#53
trick77 merged 1 commit into
masterfrom
fix/image-name-riptide-collector

Conversation

@trick77
Copy link
Copy Markdown
Owner

@trick77 trick77 commented May 17, 2026

Summary

  • First `release.yaml` run pushed to `ghcr.io/trick77/riptide` because the workflow used `${{ github.repository }}`. The image, Python package, module, and historic ci.yaml path all carry the name `riptide-collector`.
  • Pin image to `ghcr.io/${{ github.repository_owner }}/riptide-collector` and align the startup log banner.
  • Stray `0.1.0` git tag deleted out-of-band; orphan `ghcr.io/trick77/riptide` package will be deleted manually. Next release run starts fresh at `0.1.0` under the correct path.

Test plan

  • `pytest` — 189 passed
  • `ruff format --check` — clean
  • After merge: confirm release workflow pushes to `ghcr.io/trick77/riptide-collector:0.1.0`
  • Confirm startup emits `{"msg": "riptide-collector version: 0.1.0"}`

The first release.yaml run pushed to ghcr.io/trick77/riptide because the
workflow used \${{ github.repository }}. The image, Python package, module,
and historic ci.yaml path all carry the name 'riptide-collector', not
'riptide' — the GitHub repo is the odd one out.

Pin the image name explicitly to riptide-collector and align the startup
log banner with the same name so Splunk panels grepping
msg=\"<service> version: …\" match the image tag readers actually pull.

The stray 0.1.0 git tag and the orphan ghcr.io/trick77/riptide package
are deleted out-of-band; the next release run starts fresh at 0.1.0
under the correct path.
@trick77 trick77 merged commit a410e49 into master May 17, 2026
2 checks passed
@trick77 trick77 deleted the fix/image-name-riptide-collector branch May 17, 2026 09:02
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