increase k8s memory limit to 120% memory request to prevent k8s from …

…killing container (#80) * disable k8s memory limits * increase memory limit to 120% * update changelog Co-authored-by: Raj Poluri <[email protected]>
ExpediaGroup · Jan 27, 2021 · 6b110cc · 6b110cc
1 parent 9883770
commit 6b110cc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,10 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [3.2.2] - 2021-01-27
+### Changed
+- Update k8s memory limit to 120% memory request to prevent k8s from killing waggle-dance containers.
+
 ## [3.2.1] - 2021-01-27
 ### Changed
 - Update terraform required version to include 0.13.x.

diff --git a/k8s.tf b/k8s.tf
@@ -5,7 +5,8 @@
  */
 
 locals {
-  heapsize = ceil((var.memory * 85) / 100)
+  heapsize     = ceil((var.memory * 85) / 100)
+  memory_limit = ceil((var.memory * 120) / 100)
 }
 resource "kubernetes_deployment" "waggle_dance" {
   count = var.wd_instance_type == "k8s" ? 1 : 0
@@ -55,7 +56,7 @@ resource "kubernetes_deployment" "waggle_dance" {
           }
           resources {
             limits {
-              memory = "${var.memory}Mi"
+              memory = "${local.memory_limit}Mi"
             }
             requests {
               memory = "${var.memory}Mi"