Document applying config changes on self-managed (restart pods or Reloader) - #2872
pnalubandhu-a11y wants to merge 4 commits into
Conversation
…ader) Config/auth changes update the ConfigMaps but don't restart the running pods, so changes (SSO/OIDC especially) look like they didn't take effect. Add a section explaining you need to restart app/api or install Stakater Reloader.
|
Narrowing the ping. cc @wandb/delivery-tooling-team 🙂 Context: docs PR in the self-managed OpenShift series. Documents that config/auth changes (e.g. SSO/OIDC) need a pod restart to take effect — the deployments carry the Stakater Reloader annotation but Reloader isn't installed by default, so nothing restarts them. Companions: #2869, #2870, #2874. @wandb/docs-team is the reviewer. |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
📚 Mintlify Preview Links📝 Changed (1 total)📄 Pages (1)
🤖 Generated automatically when Mintlify deployment succeeds |
🔗 Link Checker Results✅ All links are valid! No broken links were detected. Preview: https://wb-21fd5541-sa-self-managed-config-change-restart.mintlify.site |
📋 Validation of technical assertionsEvery claim below was checked against the truth on the ground using Sourcegraph (the implementing
Open items — recommended next steps
Scope notes
🤖 Generated by the CoreWeave Docs Team's Validator ( |
| - **Restart the affected deployments manually.** Restart whichever components consume the changed config; `app` and `api` cover most auth/SSO changes: | ||
|
|
||
| ```bash | ||
| kubectl -n <namespace> rollout restart deployment/<release>-app deployment/<release>-api |
There was a problem hiding this comment.
🟡 Partially confirmed — the approach (restart the affected deployments) is confirmed, but these deployment names don't match the operator's default chart, so this command errors with NotFound on a default install.
On the current operator-wandb chart the app deployment renders as <release>-app-bc (deploymentPostfix: "bc" is the default), and there is no standalone <release>-api deployment — api is a ConfigMap that the app (wandb/local) deployment consumes, so restarting app applies the API/SSO config too. Names also vary by install method/version.
| kubectl -n <namespace> rollout restart deployment/<release>-app deployment/<release>-api | |
| # Deployment names vary by chart version/install. On the operator's default chart the | |
| # app deployment is <release>-app-bc, and there is no separate <release>-api deployment | |
| # (the app deployment serves the API and consumes the SSO/OIDC config). List them first: | |
| kubectl -n <namespace> get deployments | |
| kubectl -n <namespace> rollout restart deployment/<release>-app-bc |
Source: operator-wandb/values.yaml#L748-L750, wandb-base/_helpers.tpl#L26-L29, default.snap#L2830-L2834 · Recommended: confirm the canonical operator deployment names with @pnalubandhu-a11y (the OCP 4.21 test — were they <release>-app/<release>-api or -bc?).
🤖 CoreWeave Docs Team's Validator — partially confirmed
Review feedback: hardcoded names (<release>-app / <release>-api) don't match the default chart, where the app deployment carries a -bc suffix and names vary by version/install. List deployments first, then restart the right ones.
|
Good catch on #3 — fixed in 6c9470f. Since deployment names vary by version/install, the command now lists deployments first, then restarts by name (noting the To answer the question: on the OCP 4.21 install I tested, the deployments were |
Readability impactWord-weighted Flesch-Kincaid grade change across 1 changed page: +0.0 (unchanged). Lower Flesch-Kincaid grade and higher reading ease both mean easier to read. This check is informational and never blocks a PR. Human readability
AI agent comprehensionRated 0-3 (higher is easier for an agent to parse and act on).
Curated-docs baseline median FK grade by type: conceptual 10.5, procedural 8.8, reference 9.4. |
There was a problem hiding this comment.
🟡 Changes recommended
Clarify configuration refresh behavior and document the supported Reloader setup.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds self-managed documentation for applying configuration changes, including manual restarts and optional Stakater Reloader integration.
Changes:
- Documents restarting
appandapideployments. - Adds SSO/OIDC troubleshooting guidance.
- Describes automatic restart options.
File summaries
| File | Summary |
|---|---|
platform/hosting/self-managed/operator.mdx |
Adds configuration-change instructions; clarify ConfigMap refresh behavior and document spec.values.reloader.install: true. |
Review details
Suppressed comments (1)
platform/hosting/self-managed/operator.mdx:254
- The
operator-wandbchart includes the Reloader controller as an optionalreloadersubchart, withreloader.installdisabled by default. This section omits the supported operator path and only tells users to deploy another controller; document addingreloader.install: trueunderspec.valuesand reapplying the CR (with a separate installation as an alternative), otherwise the automatic option is not actionable and can lead to a redundant controller.
- **Install [Stakater Reloader](https://github.com/stakater/Reloader).** The chart already annotates the deployments with `reloader.stakater.com/auto: "true"`. If the Reloader controller is running in your cluster, it restarts the affected pods automatically when their ConfigMap or Secret changes. Reloader isn't installed by default, so on a fresh cluster you must deploy it yourself for this to work.
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| ## Apply configuration changes | ||
|
|
||
| After the initial install, you can change SSO/OIDC settings, environment variables, or other values in the custom resource (CR). When you do, the operator updates the underlying ConfigMaps, but the running pods don't automatically pick up the change. Kubernetes doesn't restart pods when a mounted ConfigMap changes. |
What
Adds an "Applying configuration changes" section to the self-managed operator page.
Why
After install, changing config in the CR (especially SSO/OIDC) updates the ConfigMaps but doesn't restart the running pods, so the change looks like it had no effect. The most common symptom is "I changed SSO settings but login still behaves the same."
The chart annotates deployments with
reloader.stakater.com/auto: "true", but that only works if the Stakater Reloader controller is installed — which isn't the case on a default cluster.Change
New section with the two ways to apply a config change:
Testing
On OCP 4.21, an OIDC config change didn't take effect until app/api were restarted. The deployments carried the reloader annotation but no Reloader controller was installed, so nothing restarted them automatically.