chore: ship click worker logs to axiom - #217
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideConfigure Vector to collect and parse logs from the new Flow diagram for Vector log shipping including spoo_click_workerflowchart LR
spoo_app[spoo_app container]
spoo_qr[spoo_qr container]
spoo_click_worker[spoo_click_worker container]
docker_source[sources.docker]
parse_and_clean[transforms.parse_and_clean]
axiom_sink[axiom sink]
spoo_app --> docker_source
spoo_qr --> docker_source
spoo_click_worker --> docker_source
docker_source --> parse_and_clean
parse_and_clean --> axiom_sink
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughModified vector.toml to add the spoo_click_worker container to the Docker logs source's include_containers list and to the conditional logic in the parse_and_clean transform that gates JSON parsing for specific containers. ChangesVector logging config
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The container name list is duplicated between
include_containersand theparse_and_cleantransform condition; consider centralizing or pattern-based matching (e.g., checking a prefix or using a shared list) to avoid future drift when adding/removing containers.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The container name list is duplicated between `include_containers` and the `parse_and_clean` transform condition; consider centralizing or pattern-based matching (e.g., checking a prefix or using a shared list) to avoid future drift when adding/removing containers.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Pull request overview
This PR updates the Vector log shipping configuration so logs from the spoo_click_worker Docker container are collected and (when applicable) JSON-parsed before being shipped to Axiom.
Changes:
- Add
spoo_click_workerto the Docker log source container allowlist. - Extend the VRL remap transform’s JSON parsing condition to include
spoo_click_worker.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .service = .container_name | ||
|
|
||
| if .container_name == "spoo_app" || .container_name == "spoo_qr" { | ||
| if .container_name == "spoo_app" || .container_name == "spoo_qr" || .container_name == "spoo_click_worker" { |
Summary by Sourcery
Enhancements:
Summary by CodeRabbit