feat(xo-web/pool): disable Rolling Pool Update if pool has 1 host

CHANGELOG.unreleased.md

@@ -23,6 +23,7 @@
 - Avoid unnecessary `pool.add_to_other_config: Duplicate key` error in XAPI log [Forum#68761](https://xcp-ng.org/forum/post/68761)
 - [Jobs] Reset parameters when editing method to avoid invalid parameters on execution [Forum#69299](https://xcp-ng.org/forum/post/69299)
 - [Metadata Backup] Fix `ENOENT` error when restoring an _XO Config_ backup [Forum#68999](https://xcp-ng.org/forum/post/68999)
+- [Pool/patches] Disable Rolling Pool Update button if host is alone in its pool [#6415](https://github.com/vatesfr/xen-orchestra/issues/6415) (PR [#7286](https://github.com/vatesfr/xen-orchestra/pull/7286))
 
 ### Packages to release
 

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

@@ -937,6 +937,8 @@ const messages = {
   noNbdConnection: 'No NBD Connection',
   nbdConnection: 'NBD Connection',
   insecureNbdConnection: 'Insecure NBD Connection (not allowed through XO)',
+  // ----- Pool patches tab -----
+  multiHostPoolUpdate: "Rolling pool update can only work when there's multiple hosts in a pool with a shared storage",
   // ----- Pool stats tab -----
   poolNoStats: 'No stats',
   poolAllHosts: 'All hosts',

packages/xo-web/src/xo-app/pool/tab-patches.js

@@ -6,7 +6,7 @@ import Upgrade from 'xoa-upgrade'
 import { addSubscriptions, connectStore, formatSize } from 'utils'
 import { alert } from 'modal'
 import { Col, Container, Row } from 'grid'
-import { createGetObjectsOfType } from 'selectors'
+import { createGetObjectsOfType, createSelector } from 'selectors'
 import { FormattedRelative, FormattedTime } from 'react-intl'
 import { getXoaPlan, ENTERPRISE } from 'xoa-plans'
 import {
@@ -17,6 +17,7 @@ import {
   subscribeHostMissingPatches,
 } from 'xo'
 import isEmpty from 'lodash/isEmpty.js'
+import size from 'lodash/size.js'
 
 const ROLLING_POOL_UPDATES_AVAILABLE = getXoaPlan().value >= ENTERPRISE.value
 
@@ -166,6 +167,12 @@ const INSTALLED_PATCH_COLUMNS = [
 }))
 @connectStore({
   hostPatches: createGetObjectsOfType('patch').pick((_, { master }) => master.patches),
+  poolHosts: createGetObjectsOfType('host').filter(
+    createSelector(
+      (_, props) => props.pool.id,
+      poolId => host => host.$pool === poolId
+    )
+  ),
 })
 export default class TabPatches extends Component {
   render() {
@@ -174,11 +181,14 @@ export default class TabPatches extends Component {
       master: { productBrand },
       missingPatches = [],
       pool,
+      poolHosts,
       userPreferences,
     } = this.props
 
     const needsCredentials = productBrand !== 'XCP-ng' && userPreferences.xsCredentials === undefined
 
+    const isSingleHost = size(poolHosts) < 2
+
     return (
       <Upgrade place='poolPatches' required={2}>
         <Container>
@@ -187,11 +197,12 @@ export default class TabPatches extends Component {
               {ROLLING_POOL_UPDATES_AVAILABLE && (
                 <TabButton
                   btnStyle='primary'
-                  disabled={isEmpty(missingPatches)}
+                  disabled={isEmpty(missingPatches) || isSingleHost}
                   handler={rollingPoolUpdate}
                   handlerParam={pool.id}
                   icon='pool-rolling-update'
                   labelId='rollingPoolUpdate'
+                  tooltip={isSingleHost ? _('multiHostPoolUpdate') : undefined}
                 />
               )}
               <TabButton