Skip to content

feat(docker): Run experimentation on the Flagsmith Postgres database in docker compose - #8884

Merged
gagantrivedi merged 2 commits into
mainfrom
feat/docker-compose-postgres-experimentation
Nov 6, 2026
Merged

gagantrivedi merged 2 commits into
mainfrom
feat/docker-compose-postgres-experimentation

Conversation

@gagantrivedi

@gagantrivedi gagantrivedi commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Thanks for submitting a PR! Please check the boxes below:

  • I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

Contributes to #8864

The API now stores SDK events in its own Postgres, so docker compose no longer needs a separate experimentation stack.

  • Drop Kafka, ClickHouse, the ingestion server and warehouse delivery from docker-compose.yml; experimentation stays enabled.
  • Remove docker-compose.flags-only.yml and the flags-only Makefile switch.
  • Update the Docker hosting guide: pick Flagsmith database under Environment Settings > Warehouse to try out experimentation; production should use a dedicated warehouse.

How did you test this code?

Built the image from this branch and ran the compose file from a fresh volume, then in the dashboard: onboarding, enabled the Flagsmith database warehouse, sent a test event, created a metric and experiment. Sent exposures and purchases for 60 identities; results and exposure charts rendered per variant.

@vercel

vercel Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Nov 5, 2026 10:52am UTC
2 Skipped Deployments
Project Deployment Actions Updated
flagsmith-frontend-preview Ignored Ignored Preview Nov 5, 2026 10:52am UTC
flagsmith-frontend-staging Ignored Ignored Preview Nov 5, 2026 10:52am UTC

Request Review

@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Explain Change →

Walkthrough

The Makefile now uses Docker Compose’s default file selection. The flags-only Compose file and the experimentation services in the main Compose file were removed. EXPERIMENTATION_ENABLED remains set to 'true', and the warehouse credential value is now literal. The Docker guide removes the flags-only instructions and describes using the Flagsmith database to store experimentation events in PostgreSQL.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: Moderate · up to d94f6

Document the upgrade path and dedicated-warehouse delivery requirements before merging. Existing installations can retain unusable warehouse connections, while users following the production guidance can configure a warehouse that receives no events.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@github-actions github-actions Bot added the docs Documentation updates label Nov 5, 2026
@github-actions github-actions Bot added feature New feature or request and removed docs Documentation updates labels Nov 5, 2026
@gagantrivedi
gagantrivedi marked this pull request as ready for review November 5, 2026 10:52
@github-actions github-actions Bot added the docs Documentation updates label Nov 5, 2026
@gagantrivedi
gagantrivedi requested review from a team as code owners November 5, 2026 10:52
@gagantrivedi
gagantrivedi requested review from bakirFS and removed request for a team November 5, 2026 10:52
@github-actions github-actions Bot removed the feature New feature or request label Nov 5, 2026
@github-actions

github-actions Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-8884 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-8884 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-8884 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-api:pr-8884 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith:pr-8884 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-8884 Finished ✅ Results ✅

@github-actions github-actions Bot added feature New feature or request and removed docs Documentation updates feature New feature or request labels Nov 5, 2026
@gagantrivedi
gagantrivedi requested a review from Zaimwa9 November 5, 2026 10:54
@github-actions

github-actions Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor
✅ private-cloud · depot-ubuntu-latest-arm-16 — run #22889 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  46.5 seconds
commit  d94f657
info  🔄 Run: #22889 (attempt 1)

🗂️ Previous results
✅ private-cloud · depot-ubuntu-latest-16 — run #22889 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  3 passed

Details

stats  3 tests across 3 suites
duration  34.7 seconds
commit  d94f657
info  🔄 Run: #22889 (attempt 1)

✅ oss · depot-ubuntu-latest-arm-16 — run #22889 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  36.2 seconds
commit  d94f657
info  🔄 Run: #22889 (attempt 1)

✅ oss · depot-ubuntu-latest-16 — run #22889 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  31.3 seconds
commit  d94f657
info  🔄 Run: #22889 (attempt 1)

@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Visual Regression

24 screenshots compared. See report for details.
View full report

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

Review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 56ca781d-702d-47c4-9d87-8149e1844b45
Commits

Reviewing files that changed from the base of the PR and between 253d313 and d94f657.

Files selected for processing: 4

  • Makefile
  • docker-compose.flags-only.yml
  • docker-compose.yml
  • docs/docs/deployment-self-hosting/hosting-guides/docker.md
Files with no reviewable changes: 1

  • docker-compose.flags-only.yml

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

Comment thread docker-compose.yml Resolved
@gagantrivedi
gagantrivedi merged commit 1ec1049 into main Nov 6, 2026
41 checks passed
@gagantrivedi
gagantrivedi deleted the feat/docker-compose-postgres-experimentation branch November 6, 2026 02:56

This branch was successfully deployed

1 active deployment
Preview – docs — d94f657f Deployed Nov 5, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants