VmHeader: show templates as such

fixes #205
Icinga · Sep 20, 2020 · eb5afd8 · eb5afd8
1 parent 8213692
commit eb5afd8
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 4 deletions.
diff --git a/doc/84-Changelog.md b/doc/84-Changelog.md
@@ -41,6 +41,7 @@ restart the *Background Daemon*.
 * FEATURE: VM details now show host resource usage (#120)
 * FEATURE: Provide suggestions in case there is no vCenter (#189)
 * FEATURE: allow to define a custom vCenter display name (#121)
+* FEATURE: show VM templates as such to make them distinguishable (#205)
 
 ### CLI
 * FIX: vm check now also looks for object name (#109)

diff --git a/library/Vspheredb/Web/Table/Object/VmExtraInfoTable.php b/library/Vspheredb/Web/Table/Object/VmExtraInfoTable.php
@@ -45,9 +45,6 @@ protected function assemble()
             $this->translate('UUID') => Html::tag('pre', $vm->get('bios_uuid')),
             $this->translate('Instance UUID') => Html::tag('pre', $vm->get('instance_uuid')),
             $this->translate('CPUs')   => $vm->get('hardware_numcpu'),
-            $this->translate('Is Template') => $vm->get('template') === 'y'
-                ? $this->translate('true')
-                : $this->translate('false'),
             $this->translate('Version') => $vm->get('version'),
         ]);
     }

diff --git a/library/Vspheredb/Web/Widget/VmHeader.php b/library/Vspheredb/Web/Widget/VmHeader.php
@@ -2,6 +2,7 @@
 
 namespace Icinga\Module\Vspheredb\Web\Widget;
 
+use gipfl\IcingaWeb2\Icon;
 use gipfl\Translation\TranslationHelper;
 use Icinga\Module\Vspheredb\DbObject\VirtualMachine;
 use ipl\Html\Html;
@@ -27,7 +28,16 @@ protected function assemble()
         $vm = $this->vm;
         $powerState = $vm->get('runtime_power_state');
         $renderer = new PowerStateRenderer();
-        if ($powerState !== 'poweredOn') {
+        if ($vm->get('template') === 'y') {
+            $cpu = Html::tag('div', [
+                'class' => 'vm-template'
+            ], Icon::create('upload', [
+                'title' => $this->translate('This is a template'),
+                'class' => [ 'state' ]
+            ]));
+
+            $mem = $this->translate('This is a template');
+        } elseif ($powerState !== 'poweredOn') {
             $cpu = Html::tag('div', [
                 'class' => 'cpu off',
                 // 'style' => 'font-size: 3em; width: 1em; height: 1em; display: inline-block;',

diff --git a/public/css/module.less b/public/css/module.less
@@ -1129,6 +1129,27 @@ pre.logOutput {
 }
 
 /** End logfile */
+div.vm-template {
+  float: left;
+  border: 0.75em solid @text-color;
+  height: 6em;
+  box-sizing: border-box;
+  width: 6em;
+  border-radius: 1em;
+  text-align: center;
+  display: inline-block;
+  background-color: @gray;
+  margin-right: 1em;
+  color: @text-color;
+  padding: 0.2em 0;
+  i.state {
+    &:before {
+      color: @text-color;
+    }
+    font-size: 2.2em;
+  }
+}
+
 .cpu {
   border: 0.75em solid @text-color;
   height: 6em;