Point the worker's agent at the web service inside the compose network - #130
Closed
xrendan wants to merge 2 commits into
Closed
Point the worker's agent at the web service inside the compose network#130xrendan wants to merge 2 commits into
xrendan wants to merge 2 commits into
Conversation
AgentEvaluateCommitmentJob and AgentProcessEntryJob shell out to the claude CLI, which writes results back via RAILS_API_URL. The worker container never had that variable, so the subprocess defaulted to http://localhost:3000 and could not reach the web service.
GoodJob drops enqueues beyond enqueue_limit without raising. With 598 non-broken commitments and a limit of 550, AgentWeeklyScanJob silently skipped the tail of every full scan.
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.
Summary
The agent jobs run in the
workercontainer and shell out to theclaudeCLI, which writes evidence, criteria and evaluation runs back through the Rails agent API usingRAILS_API_URL. That variable was never set for the worker, so the subprocess fell back tohttp://localhost:3000, which inside the worker container is nothing. Every production agent run since the agent shipped, including the SundayAgentWeeklyScanJob, would have failed to write results.Sets
RAILS_API_URL=http://web:3000on the worker service. The web service listens on 3000 in-container and is reachable by service name on the default compose network.Testing
Static change only. To confirm after deploy, from the host:
should print
200.