Increase max RPC size to 64MB

elastic / elastic-agent-shipper

Data shipper for the Elastic Agent - single, unified way to add monitoring for logs, metrics, and other types of data to a host.

Other

9 stars 17 forks source link

Increase max RPC size to 64MB #281

Closed faec closed 1 year ago

faec commented 1 year ago

Increase the size limit of RPC calls to the shipper from its default of 4MB to 64MB.

This PR mitigates but does not fix https://github.com/elastic/beats/issues/34695.

[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[ ] ~~I have made corresponding changes to the documentation~~
[ ] ~~I have made corresponding change to the default configuration files~~
[ ] ~~I have added tests that prove my fix is effective or that my feature works~~
[ ] ~~I have added an entry in CHANGELOG.md or CHANGELOG-developer.md.~~

mergify[bot] commented 1 year ago

This pull request does not have a backport label. If this is a bug or security fix, could you label this PR @faec? 🙏. For such, you'll need to label your PR with:

The upcoming major version of the Elastic Stack
The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed branches, such as:

backport-v8./d.0 is the label to automatically backport to the 8./d branch. /d is the digit

elasticmachine commented 1 year ago

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

#### Build stats * Start Time: 2023-03-07T16:55:52.706+0000 * Duration: 17 min 29 sec

:grey_exclamation: Flaky test report

No test was executed to be analysed.

:robot: GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with: - `/test` : Re-trigger the build.

faec commented 1 year ago

/test