Repository navigation
Report metrics to the stats service over the internal network - #12
Merged
Merged
Conversation
account and front had STATS_URL set to the public URL. Inside a container that resolves to the container itself, so every stats report failed with ECONNREFUSED. Server-side reporting now goes to http://stats:4900 like the other services; front passes the public URL to the browser via STATS_API (the same split the helm chart uses). The smoke test now fails on a steady stream of stats fetch errors. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012bF628vAWVF7d1gqS2bQwG Signed-off-by: Artem Savchenko <armisav@gmail.com>
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.
accountandfronthadSTATS_URLset to the public URL (http://<host>/_stats). Inside a container that address points at the container itself, so every stats report failed withconnect ECONNREFUSED 127.0.0.1:8087every 10s.account:STATS_URL=http://stats:4900, like the other backend servicesfront:STATS_URL=http://stats:4900for the server's own reporting, public URL moved toSTATS_API— front passesSTATS_API ?? STATS_URLto the browser (pods/front/src/__start.ts, present in s0.7.426). Same split as the helm chart (STATS_URL_INTERNAL)traefik/template-compose.yaml: front uses the internalSTATS_URLfromcommon-env, public URL viaSTATS_APItests/smoke.sh: fails on a steady stream of stats fetch errors in the service logs🤖 Generated with Claude Code
https://claude.ai/code/session_012bF628vAWVF7d1gqS2bQwG
Generated by Claude Code