feat(xo-web,xo-server): expose VM Xen Tools version (#2838)

Fixes #2650

packages/xo-server/src/xapi-object-to-xo.js

@@ -232,13 +232,11 @@ const TRANSFORMS = {
       }
 
       const { major, minor } = guestMetrics.PV_drivers_version
-      const [hostMajor, hostMinor] = (
-        obj.$resident_on || obj.$pool.$master
-      ).software_version.product_version.split('.')
 
-      return major >= hostMajor && minor >= hostMinor
+      return {
-        ? 'up to date'
+        major,
-        : 'out of date'
+        minor,
+      }
     })()
 
     let resourceSet = otherConfig['xo:resource_set']

packages/xo-web/src/common/intl/messages.js

@@ -894,12 +894,7 @@ const messages = {
   cpuCapLabel: 'CPU cap',
   defaultCpuCap: 'Default ({value, number})',
   pvArgsLabel: 'PV args',
-  xenToolsStatus: 'Xen tools status',
+  xenToolsStatus: 'Xen tools version',
-  xenToolsStatusValue: {
-    defaultMessage: '{status}',
-    description:
-      'status can be `not-installed`, `unknown`, `out-of-date` & `up-to-date`',
-  },
   osName: 'OS name',
   osKernel: 'OS kernel',
   autoPowerOn: 'Auto power on',

packages/xo-web/src/common/utils.js

@@ -219,21 +219,6 @@ export const parseSize = size => {
 
 // -------------------------------------------------------------------
 
-export const normalizeXenToolsStatus = status => {
-  if (status === false) {
-    return 'not-installed'
-  }
-  if (status === undefined) {
-    return 'unknown'
-  }
-  if (status === 'up to date') {
-    return 'up-to-date'
-  }
-  return 'out-of-date'
-}
-
-// -------------------------------------------------------------------
-
 const _NotFound = () => <h1>{_('errorPageNotFound')}</h1>
 
 // Decorator to declare routes on a component.

packages/xo-web/src/xo-app/vm/tab-advanced.js

@@ -20,7 +20,6 @@ import {
   connectStore,
   formatSize,
   getCoresPerSocketPossibilities,
-  normalizeXenToolsStatus,
   osFamily,
 } from 'utils'
 import {
@@ -561,9 +560,7 @@ export default connectStore(() => {
             <tr>
               <th>{_('xenToolsStatus')}</th>
               <td>
-                {_('xenToolsStatusValue', {
+                {vm.xenTools && `${vm.xenTools.major}.${vm.xenTools.minor}`}
-                  status: normalizeXenToolsStatus(vm.xenTools),
-                })}
               </td>
             </tr>
             <tr>