Skip to content

fix: increase buffers used in Catalog's gRPC connection to 16MB - #1210

Merged
gustavogama-cll merged 1 commit into
mainfrom
ggama/fix/increase-catalog-grpc-client-maxmsgsize-16mb
Nov 4, 2026
Merged

gustavogama-cll merged 1 commit into
mainfrom
ggama/fix/increase-catalog-grpc-client-maxmsgsize-16mb

Conversation

@gustavogama-cll

@gustavogama-cll gustavogama-cll commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

This pull request increases the buffer sizes used in the Catalog service's gRPC connections to 16MB. This change is intended to allow larger messages to be sent and received between services, improving reliability when handling larger payloads.

ref guardian ticket: https://chainlink-core.slack.com/archives/C0BRDM5J85C/p1793741214875219

@changeset-bot

changeset-bot Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 1e00543

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
chainlink-deployments-framework Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

This comment was marked as resolved.

@gustavogama-cll
gustavogama-cll force-pushed the ggama/fix/increase-catalog-grpc-client-maxmsgsize-16mb branch from 002c456 to 1e00543 Compare November 4, 2026 02:06
@gustavogama-cll
gustavogama-cll marked this pull request as ready for review November 4, 2026 02:13
@gustavogama-cll
gustavogama-cll requested a review from a team as a code owner November 4, 2026 02:13
Copilot AI balanced review requested due to automatic review settings November 4, 2026 02:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Existing tests bypass the production client’s message-size options, leaving successful payloads between 4 MiB and 16 MiB and rejection above 16 MiB without regression coverage.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@gustavogama-cll
gustavogama-cll added this pull request to the merge queue Nov 4, 2026
Merged via the queue into main with commit 5d0e37a Nov 4, 2026
26 of 27 checks passed
@gustavogama-cll
gustavogama-cll deleted the ggama/fix/increase-catalog-grpc-client-maxmsgsize-16mb branch November 4, 2026 02:32
@gati-app gati-app Bot mentioned this pull request Nov 4, 2026
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.

3 participants