Skip to content

Raise memory limit to 4096Mi and heap to 3000m - #958

Merged
henrik242 merged 1 commit into
mainfrom
fix/antu-memory-limit
Sep 14, 2026
Merged

henrik242 merged 1 commit into
mainfrom
fix/antu-memory-limit

Conversation

@henrik242

Copy link
Copy Markdown
Contributor

No description provided.

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.

🟢 Approval recommended

The resource values are consistent with the documented JVM headroom requirement and deployment template.

Pull request overview

Raises Antu pod memory to prevent kernel OOM kills while preserving JVM headroom.

Changes:

  • Raises memory request and limit to 4096Mi.
  • Raises JVM maximum heap to 3000m.
  • Documents the required non-heap memory allowance.
File summaries
File Description
helm/antu/values.yaml Updates pod memory and JVM heap settings.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@henrik242
henrik242 merged commit c582bfb into main Sep 14, 2026
4 checks passed
@henrik242
henrik242 deleted the fix/antu-memory-limit branch September 14, 2026 08:07
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

📄 Terraform Plan DEV

Plan: 2 to add, 0 to change, 0 to destroy. 
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
 + create

Terraform will perform the following actions:

 # google_pubsub_subscription.AntuCommonFilesAggregationQueue will be created
 + resource "google_pubsub_subscription" "AntuCommonFilesAggregationQueue" {
     + ack_deadline_seconds       = (known after apply)
     + id                         = (known after apply)
     + labels                     = {
         + "app"     = "antu"
         + "manager" = "terraform"
         + "slack"   = "talk-ror"
         + "team"    = "ror"
       }
     + message_retention_duration = "3600s"
     + name                       = "AntuCommonFilesAggregationQueue"
     + project                    = "ent-antu-dev"
     + topic                      = "AntuCommonFilesAggregationQueue"

     + retry_policy {
         + maximum_backoff = (known after apply)
         + minimum_backoff = "10s"
       }
   }

 # google_pubsub_subscription.AntuReportAggregationQueue will be created
 + resource "google_pubsub_subscription" "AntuReportAggregationQueue" {
     + ack_deadline_seconds       = (known after apply)
     + id                         = (known after apply)
     + labels                     = {
         + "app"     = "antu"
         + "manager" = "terraform"
         + "slack"   = "talk-ror"
         + "team"    = "ror"
       }
     + message_retention_duration = "3600s"
     + name                       = "AntuReportAggregationQueue"
     + project                    = "ent-antu-dev"
     + topic                      = "AntuReportAggregationQueue"

     + retry_policy {
         + maximum_backoff = (known after apply)
         + minimum_backoff = "10s"
       }
   }

Plan: 2 to add, 0 to change, 0 to destroy.


View full plan

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