Skip to content

Conversation

@dobrac
Copy link
Contributor

@dobrac dobrac commented Oct 18, 2025

Stop returning unused duplicit logs. Limit the log entries count (more should be requested by setting offset).

Should improve the following error:

polling received unrecoverable error: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (4258626 vs. 4194304)


Note

Adds a capped limit to TemplateStatusRequest and drops legacy logs strings, returning only structured logEntries with offset/level filtering.

  • Protocol (packages/orchestrator/template-manager.proto):
    • Add optional limit to TemplateStatusRequest for log pagination.
    • Remove repeated string logs from TemplateBuildStatusResponse; reserve fields 3,4.
  • Server (internal/template/server/template_status.go):
    • Enforce per-request log limit (max 100), respecting offset and level; stop formatting/returning string logs.
    • Return only logEntries.
  • Generated code (template-manager.pb.go):
    • Reflects new limit field and removal of logs field; updates descriptors and accessors accordingly.

Written by Cursor Bugbot for commit a2b8d8f. This will update automatically on new commits. Configure here.

@dobrac dobrac added the bug Something isn't working label Oct 18, 2025
@linear
Copy link

linear bot commented Oct 18, 2025

@dobrac dobrac force-pushed the minimize-template-build-log-size-returned-eng-3196 branch from aa59498 to 3f78fa2 Compare October 18, 2025 09:08
@dobrac dobrac force-pushed the minimize-template-build-log-size-returned-eng-3196 branch from 3f78fa2 to b802682 Compare October 18, 2025 09:37
cursor[bot]

This comment was marked as outdated.

@dobrac dobrac force-pushed the minimize-template-build-log-size-returned-eng-3196 branch from 7a7f9e0 to a2b8d8f Compare October 18, 2025 09:44
@dobrac dobrac merged commit 0cd8c98 into main Oct 18, 2025
27 checks passed
@dobrac dobrac deleted the minimize-template-build-log-size-returned-eng-3196 branch October 18, 2025 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants