Repository navigation
Conversation
In single-binary mode the querier hands each result back to the query frontend over gRPC, and Loki's server kept the 4MiB message default. A filter query over a busy workload's logs failed with "received message larger than max (31010467 vs. 4194304)". Raise the server's send and receive limits to 100MiB, the querier's own client limit.
Ravion Module Publish PlanDry run only. No Ravion API mutations were made.
Diffsrvn-eks-addons 0.23.0 -> 0.23.2--- remote
+++ compiled
- Each load balancer's security group is automatically allowed to reach pods through the EKS cluster security group (all TCP ports).
- Because `TargetGroupBinding` depends on the controller, enabling any load balancer automatically installs the **load balancer controller** - there is nothing extra to configure.
- Terraform source: [ravionhq/modules/compute/eks/addons](https://github.com/ravionhq/modules/tree/rvn-eks-addons@0.23.0/compute/eks/addons)
+ Terraform source: [ravionhq/modules/compute/eks/addons](https://github.com/ravionhq/modules/tree/rvn-eks-addons@0.23.2/compute/eks/addons)
## Prerequisites
@@
execution_environment_overrides:
security_group: << module.input.ravion_runner_security_group_id >>
plan_permissions: '<< module.input.terraform_plan_default_policies_enabled == false ? {"replace": module.input.terraform_plan_iam_policy_arns} : (module.input.terraform_plan_iam_policy_arns ? {"attach": module.input.terraform_plan_iam_policy_arns} : nil) >>'
- ref: rvn-eks-addons@0.23.0
+ ref: rvn-eks-addons@0.23.2
repo: https://github.com/ravionhq/modules
stack_id: <<stack.id>>
terraform_variables: |
flybayer
approved these changes
Oct 17, 2026
Contributor
|
@mabadir @flybayer This PR could not be combined with the queued merge tree. Failed operation: merge #220 at How to resolve
No conflict diff was captured, and assembly stopped before a candidate commit was created. Use the exact commit comparisons and conflicting files above; there is no candidate-commit diff to link. |
Contributor
|
Removed from the queue. Re-add the label to queue it again. |
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
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.
Why
In prod, Loki's logs show log searches failing with:
In single-binary mode the querier returns each result to the query frontend over gRPC, and Loki's server keeps the 4MiB message default. A filter over a busy workload's logs (e.g.
{namespace="ravion-prod", workload="api"} |= "rate_limit"over a few hours) produces results well past that, so the search fails.What
loki.server.grpc_server_max_recv_msg_size/grpc_server_max_send_msg_size= 100MiB (104857600), matching the querier's own client limit.tests/logs.tftest.hcl. Verified withhelm templateagainst grafana/loki 7.3.0 that both keys render into Loki'sserverconfig.Tests
tofu testincompute/eks/addons: 263 passedcharts/test.sh: 390 passedThe PR appears safe to merge; no actionable issues were found.
Summary
Raises Loki's gRPC send and receive limits to 100MiB so larger log-query results can pass.
Reviews (1) · Last reviewed commit: "fix(eks-addons): let Loki return query r..." · Reviewed by Greptile