Merge pull request #1401 from zeari/container_definition_limits

Add container limits to container summary page
ManageIQ · Jul 31, 2017 · 144e0ca · 144e0ca
2 parents 8eb9a27 + f40111d
commit 144e0ca
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/app/controllers/container_controller.rb b/app/controllers/container_controller.rb
@@ -13,7 +13,7 @@ def show_list
   private
 
   def textual_group_list
-    [%i(properties relationships smart_management), %i(env)]
+    [%i(properties relationships smart_management), %i(env limits)]
   end
   helper_method :textual_group_list
 

diff --git a/app/helpers/container_helper/textual_summary.rb b/app/helpers/container_helper/textual_summary.rb
@@ -140,4 +140,14 @@ def collect_env_variables
       [var.name, var.value, var.field_path]
     end
   end
+
+  def textual_group_limits
+    TextualMultilabel.new(
+      _("Request and Limits"),
+      :labels => [_("Resource"), _("Value")],
+      :values => [[_("Limit CPU cores"), @record.limit_cpu_cores],
+                  [_("Limit CPU cores"), @record.limit_memory_bytes],
+                  [_("Request CPU cores"), @record.request_cpu_cores],
+                  [_("Request Memory"), @record.request_memory_bytes]])
+  end
 end