Skip to content

set P_SEND_ANONYMOUS_USAGE_DATA=false for docker compose tests - #1825

Merged
nikhilsinhaparseable merged 1 commit into
parseablehq:mainfrom
nikhilsinhaparseable:no-analytics-test
Oct 15, 2026
Merged

nikhilsinhaparseable merged 1 commit into
parseablehq:mainfrom
nikhilsinhaparseable:no-analytics-test

Conversation

@nikhilsinhaparseable

@nikhilsinhaparseable nikhilsinhaparseable commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Configuration
    • Anonymous usage data collection is disabled across the included test deployment setups.
    • The Google Cloud Storage distributed test setup now limits query-service restart attempts to three.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: dfc6e7cd-213f-496b-901d-b3f5e505ef61
📥 Commits

Reviewing files that changed from the base of the PR and between c94bd14 and d856a7d.

📒 Files selected for processing: 5
  • docker-compose-distributed-test-with-kafka.yaml
  • docker-compose-distributed-test.yaml
  • docker-compose-gcs-distributed-test.yaml
  • docker-compose-test-with-kafka.yaml
  • docker-compose-test.yaml

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

Walkthrough

Compose test configurations now disable anonymous usage-data sending for the listed Parseable services. The GCS distributed test configuration also limits the query service to three restart attempts.

Changes

Compose test configuration

Layer / File(s) Summary
Test service settings
docker-compose-distributed-test-with-kafka.yaml, docker-compose-distributed-test.yaml, docker-compose-gcs-distributed-test.yaml, docker-compose-test-with-kafka.yaml, docker-compose-test.yaml
The listed services set P_SEND_ANONYMOUS_USAGE_DATA=false. The GCS query service sets max_attempts to 3.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to d856a

The listed test services will skip anonymous-usage scheduling. No material merge-blocking risk was established, so the change appears ready to merge.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The pull request has no description. It does not include the template's description, rationale, key changes, testing status, or documentation status. Add a pull request description with the goal, solution and rationale, key changes, issue reference if applicable, and completed checklist items for testing, comments, and documentation.
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly identifies the main change: disabling anonymous usage data for Docker Compose tests.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit checks the test configs at night
Usage-data sending is switched off right
Three restart tries for GCS query
The compose files now agree
Then hops away beneath the moonlight

Comment @coderabbitai help to get the list of available commands.

@nikhilsinhaparseable
nikhilsinhaparseable merged commit 9baabc5 into parseablehq:main Oct 15, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants