Skip to content

fix(service): revive posthog template#10845

Closed
Prains wants to merge 1 commit into
coollabsio:nextfrom
Prains:fix/posthog-template-review-followup
Closed

fix(service): revive posthog template#10845
Prains wants to merge 1 commit into
coollabsio:nextfrom
Prains:fix/posthog-template-review-followup

Conversation

@Prains

@Prains Prains commented Jul 3, 2026

Copy link
Copy Markdown

Changes

  • Revives the abandoned PostHog template fix from fix(service): posthog template #8739 on the current next branch.
  • Re-enables the PostHog one-click template.
  • Uses Coolify service URL labels instead of an internal proxy container.
  • Keeps named volumes out of the source template so Coolify can generate them.
  • Adds permissions-init to prepare bind directories for ClickHouse UID 101 and livestream/Temporal UID 1000 before those services start.
  • Updates livestream with the shared GeoIP DB, configs.yml, and event/session-recording consumers from the pending fix(service): posthog template #8739 review suggestion.
  • Hardens geoip-init to replace corrupt GeoIP files, skip valid files, fail closed on invalid downloads, and preserve shell variables after Compose interpolation.

Issues

Category

  • Bug fix
  • Improvement
  • New feature
  • Adding new one click service
  • Fixing or updating existing one click service

Preview

N/A — service template change. Verified with local generator, rendered Docker Compose, and throwaway Docker Compose smoke tests.

AI Assistance

  • AI was NOT used to create this PR
  • AI was used (please describe below)

If AI was used:

  • Tools used: OpenAI Codex
  • How extensively: Used for PR/review inspection, implementation, local validation, and independent review/QA.

Testing

  • Generated service templates locally and verified PostHog includes permissions-init, livestream, and hardened geoip-init.
  • Targeted service-template tests passed: 1 test / 2 assertions and 7 tests / 15 assertions.
  • Rendered the template to plain Docker Compose and ran Docker Compose config validation.
  • Ran throwaway smoke tests for permissions ownership, fresh GeoIP download, corrupt GeoIP replacement, and valid GeoIP skip.
  • Ran git diff whitespace validation.
  • Confirmed only the PostHog compose template is tracked in the final commit.

Contributor Agreement

Important

  • I have read and understood the contributor guidelines. If I have failed to follow any guideline, I understand that this PR may be closed without review.
  • I have searched existing issues and pull requests (including closed ones) to ensure this isn't a duplicate.
  • I have tested all the changes thoroughly with a local development instance of Coolify and I am confident that they will work as expected when a maintainer tests them.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This PR did not pass quality checks so it will be closed. If you believe this is a mistake please let us know.

@github-actions github-actions Bot closed this Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant