Skip to content

Conversation

@DTTerastar
Copy link
Contributor

@DTTerastar DTTerastar commented Jan 4, 2026

  • Enabled full observability and logging for both artifact-proxy and release-proxy workers
  • Configured 100% request sampling for comprehensive monitoring
  • Enabled worker invocation tracking and persistent log storage

Summary by CodeRabbit

  • Chores
    • Enhanced logging and observability configuration across proxy services with persistent logging and full request sampling to improve system monitoring and diagnostics.

✏️ Tip: You can customize this high-level summary in your review settings.

- Enable full request logging with 100% sampling rate
- Enable worker invocation tracking
- Enable persistent log storage
- View logs in Cloudflare Workers dashboard

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Jan 4, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

Configuration updates to two Cloudflare Workers wrangler.toml files, adding observability.logs settings with enabled=true, head_sampling_rate=1, invocation_logs=true, and persist=true to enable centralized logging.

Changes

Cohort / File(s) Summary
Observability Configuration
espresense-artifact-proxy/wrangler.toml, espresense-release-proxy/wrangler.toml
Added identical observability.logs configuration block to both wrangler.toml files, enabling logs, sampling, invocation tracking, and persistence for Cloudflare Workers observability.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Logs now flow like carrots in the garden,
Two workers singing in observability's arden,
Head sampling set to one, persistence true,
Invocation whispers captured just for you! 🌟

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-wrangler-config

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e8e78a2 and 5d3eaa1.

📒 Files selected for processing (2)
  • espresense-artifact-proxy/wrangler.toml
  • espresense-release-proxy/wrangler.toml

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

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

@DTTerastar DTTerastar merged commit 837f07e into master Jan 4, 2026
1 of 2 checks passed
@DTTerastar DTTerastar deleted the update-wrangler-config branch January 4, 2026 07:21
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