fix(telemetry): document the opt-out for both collectors - #651
Open
Andrei Kvapil (kvaps) wants to merge 2 commits into
Open
fix(telemetry): document the opt-out for both collectors#651Andrei Kvapil (kvaps) wants to merge 2 commits into
Andrei Kvapil (kvaps) wants to merge 2 commits into
Conversation
The page names two telemetry collectors, cozystack-operator and cozystack-controller, and then gave a single opt-out that only reaches the operator. cozystack-controller is deployed by the platform, not by the cozy-installer chart, so it kept reporting application counts after a reader had followed the documented procedure. Both steps are now given. On next, the second step uses the platform's new telemetry.disabled key; on v1.6, which has no such key, it overrides the component values directly. Assisted-By: Claude Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
✅ Deploy Preview for cozystack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
📝 WalkthroughWalkthroughThe telemetry documentation for the next, v1.5, and v1.6 versions now explains separate operator and controller telemetry settings. It adds controller opt-out configuration through the ChangesTelemetry documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
11 tasks
v1.5 is still installed and carries the same incomplete instruction. It gets the v1.6 form, which overrides the component values directly, since neither version has the platform-level telemetry.disabled key. Earlier documentation versions are left alone. Assisted-By: Claude Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
Andrei Kvapil (kvaps)
marked this pull request as ready for review
August 12, 2026 20:58
Andrei Kvapil (kvaps)
requested review from
Timofei Larkin (lllamnyp),
myasnikovdaniil and
Timur Tukaev (tym83)
as code owners
August 12, 2026 20:58
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@content/en/docs/next/operations/configuration/telemetry.md`:
- Around line 102-103: Update the re-enable instructions in
content/en/docs/next/operations/configuration/telemetry.md:102-103 to explicitly
run Helm with cozystackOperator.disableTelemetry=false, then set
telemetry.disabled: false in the Platform Package and apply it. Apply the
corresponding update in
content/en/docs/v1.5/operations/configuration/telemetry.md:103-104 and
content/en/docs/v1.6/operations/configuration/telemetry.md:103-104, using
cozystackController.disableTelemetry: false in the engine Package and
instructing the operator to apply it.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 0e3816bd-262c-413c-8c09-cb78e0664ea6
📒 Files selected for processing (3)
content/en/docs/next/operations/configuration/telemetry.mdcontent/en/docs/v1.5/operations/configuration/telemetry.mdcontent/en/docs/v1.6/operations/configuration/telemetry.md
Comment on lines
+102
to
+103
| To re-enable telemetry later, run the same command with | ||
| `disableTelemetry=false` and set `telemetry.disabled: false`. |
Contributor
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Document separate re-enable actions for the operator and controller.
Each page uses two configuration mechanisms, but the re-enable text can lead users to run only the Helm command and omit the Package update.
content/en/docs/next/operations/configuration/telemetry.md#L102-L103: run Helm withcozystackOperator.disableTelemetry=false, then settelemetry.disabled: falsein the Platform Package and apply it.content/en/docs/v1.5/operations/configuration/telemetry.md#L103-L104: run Helm withcozystackOperator.disableTelemetry=false, then setcozystackController.disableTelemetry: falsein the engine Package and apply it.content/en/docs/v1.6/operations/configuration/telemetry.md#L103-L104: run Helm withcozystackOperator.disableTelemetry=false, then setcozystackController.disableTelemetry: falsein the engine Package and apply it.
📍 Affects 3 files
content/en/docs/next/operations/configuration/telemetry.md#L102-L103(this comment)content/en/docs/v1.5/operations/configuration/telemetry.md#L103-L104content/en/docs/v1.6/operations/configuration/telemetry.md#L103-L104
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@content/en/docs/next/operations/configuration/telemetry.md` around lines 102
- 103, Update the re-enable instructions in
content/en/docs/next/operations/configuration/telemetry.md:102-103 to explicitly
run Helm with cozystackOperator.disableTelemetry=false, then set
telemetry.disabled: false in the Platform Package and apply it. Apply the
corresponding update in
content/en/docs/v1.5/operations/configuration/telemetry.md:103-104 and
content/en/docs/v1.6/operations/configuration/telemetry.md:103-104, using
cozystackController.disableTelemetry: false in the engine Package and
instructing the operator to apply it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does
The telemetry page names two collectors —
cozystack-operatorandcozystack-controller— and then gives a single opt-out that only reaches the operator.cozystack-controlleris deployed by the platform, not by thecozy-installerchart, so--set cozystackOperator.disableTelemetry=truenever reaches it: a reader who follows the documented procedure keeps reportingcozy_application_countevery 15 minutes, with nothing to indicate the opt-out was partial.Both steps are now documented:
nextuses the platform's newtelemetry.disabledkey.v1.6has no such key, so it overrides thecozystack-controllercomponent values on thecozystack.cozystack-enginePackage directly.Earlier documentation versions carry the same incomplete instruction. They are left alone here, on the usual "backport only if it matters for a released version" rule — say the word if v1.5 and below should get the workaround too.
Summary by CodeRabbit