Repository navigation
docs(experimentation): Make the self-hosted experimentation guide easier to follow - #8801
Conversation
…ier to follow Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 Walkthrough
Merge Risk: 🔵 Low · up to
|
There was a problem hiding this comment.
Actionable comments posted: 2
ℹ️ Review info
⚙️ Run configuration
- Configuration used: Organization UI
- Review profile: ASSERTIVE
- Plan: Advanced
- Run ID:
d5c4308e-4796-4dc7-a25e-8a13836b9990
📒 Files selected for processing: 2
docs/docs/deployment-self-hosting/hosting-guides/kubernetes-openshift.mddocs/docs/experimentation/run-an-experiment.mdx
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.
# Conflicts: # docs/docs/experimentation/run-an-experiment.mdx
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
Walking through the Kubernetes guide to set experimentation up, it was never clear which infrastructure is the
operator's to provide and which the chart stands up, and the values example arrived with no explanation of what it was.
"Requirements" to "Prerequisites" to match the rest of the docs.
environment", which implies one warehouse per environment; one serves them all, as events are scoped by environment
key.
passwordFromExistingSecretadvice.rather than something to provision, and explain why
auto.create.topics.enable=truemeans creating the topicsyourself first.
closing paragraph instead of dangling off a colon.
run-an-experiment: note that the SDKs post to Flagsmith's hosted events endpoint unless told otherwise, soself-hosted installations must point them at their own ingestion service.
The values example uses
experimentation.warehouseCredentialsSecretFromExistingSecret, so it is accurate onceFlagsmith/flagsmith-charts#616 ships. The Chart Values table is untouched here, as #8800 covers it.
How did you test this code?
npm run buildindocs/passes and reports no new broken anchors. Each statement was checked against the charttemplates and the API source rather than the existing prose.