fix(xo-server/XOSTOR): install or update packages on all pool hosts (#…

…7597)

CHANGELOG.unreleased.md

@@ -15,11 +15,11 @@
 > Users must be able to say: “I had this issue, happy to know it's fixed”
 
 - Fix support of XenServer 6.5 (broken in XO 5.93.0)
-
 - [VMWare/Import] Fix `Cannot create property 'xxx' on string 'yyy' when trying to import from ESXi
 - [Import/VMWare] Fix ERR_PREMATURE_CLOSE error with Xenserver hosts (PR [#7563](https://github.com/vatesfr/xen-orchestra/pull/7563))
 - [VMWare/Migration] Handle multiple datacenters (PR [#7553](https://github.com/vatesfr/xen-orchestra/pull/7553))
 - [XOSTOR/create] In the summary section, the warning message "Hosts do not have the same number of disks" now takes into consideration host without disks (PR [#7572](https://github.com/vatesfr/xen-orchestra/pull/7572))
+- [XOSTOR] Install or update packages on all hosts in the pool rather than just hosts with disks (PR [#7597](https://github.com/vatesfr/xen-orchestra/pull/7597))
 
 ### Packages to release
 

packages/xo-server/src/api/xostor.mjs

@@ -212,6 +212,8 @@ export const create = defer(async function (
 
     const host = hosts[0]
     const xapi = this.getXapi(host)
+    const poolHostIds = Object.keys(xapi.objects.indexes.type.host)
+    const poolHosts = poolHostIds.map(id => this.getObject(id, 'host'))
 
     const handleHostsDependencies = defer(async ($defer, hosts) => {
       const boundInstallDependencies = installDependencies.bind(this)
@@ -248,7 +250,7 @@ export const create = defer(async function (
         }
       )
     })
-    await handleHostsDependencies(hosts)
+    await handleHostsDependencies(poolHosts)
 
     const boundFormatDisks = formatDisks.bind(this)
     await asyncEach(