feat(xo-web/host): ask for confirmation to reboot the updated slave host if the master is not

CHANGELOG.unreleased.md

@@ -8,6 +8,7 @@
 > Users must be able to say: “Nice enhancement, I'm eager to test it”
 
 - [Settings/Logs] Use GitHub issue form with pre-filled fields when reporting a bug [#7142](https://github.com/vatesfr/xen-orchestra/issues/7142) (PR [#7274](https://github.com/vatesfr/xen-orchestra/pull/7274))
+- [Host/Reboot] Confirmation modal to reboot an updated slave host if the master is not [#7059](https://github.com/vatesfr/xen-orchestra/issues/7059) (PR [#7293](https://github.com/vatesfr/xen-orchestra/pull/7293))
 
 ### Bug fixes
 

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

@@ -128,7 +128,25 @@ export async function restart({
 
   bypassBlockedSuspend = force,
   bypassCurrentVmCheck = force,
+  bypassVersionCheck = force,
 }) {
+
+  if (bypassVersionCheck) {
+    log.warn('host.restart with argument "bypassVersionCheck" set to true', { hostId: host.id })
+  } else {
+    const pool = this.getObject(host.$poolId, 'pool')
+    const master = this.getObject(pool.master, 'host')
+    const hostRebootRequired = host.rebootRequired
+    if (hostRebootRequired && host.id !== master.id && host.version === master.version) {
+      throw incorrectState({
+        actual: hostRebootRequired,
+        expected: false,
+        object: host.id,
+        property: 'rebootRequired',
+      })
+    }
+  }
+
   if (bypassBackupCheck) {
     log.warn('host.restart with argument "bypassBackupCheck" set to true', { hostId: host.id })
   } else {
@@ -165,6 +183,10 @@ restart.params = {
     type: 'boolean',
     default: false,
   },
+  bypassVersionCheck: {
+    type: 'boolean',
+    optional: true,
+  },
 }
 
 restart.resolve = {

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

@@ -966,6 +966,8 @@ const messages = {
   // ----- Host actions ------
   disableMaintenanceMode: 'Disable maintenance mode',
   enableMaintenanceMode: 'Enable maintenance mode',
+  slaveHostMoreUpToDateThanMasterAfterReboot:
+    'It appears that your slave host will be more up-to-date than the master after the reboot. This will result in the slave being unable to contact the pool master. Please update and reboot your master node first.',
   startHostLabel: 'Start',
   stopHostLabel: 'Stop',
   enableHostLabel: 'Enable',
@@ -978,6 +980,7 @@ const messages = {
   forceRebootHostLabel: 'Force reboot',
   forceSmartRebootHost:
     'Smart Reboot failed because {nVms, number} VM{nVms, plural, one {} other {s}} ha{nVms, plural, one {s} other {ve}} {nVms, plural, one {its} other {their}} Suspend operation blocked. Would you like to force?',
+  rebootAnyway: 'Reboot anyway',
   rebootHostLabel: 'Reboot',
   noHostsAvailableErrorTitle: 'Error while restarting host',
   noHostsAvailableErrorMessage:

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

@@ -900,6 +900,21 @@ const _restartHost = async ({ host, ...opts }) => {
       return _restartHost({ ...opts, host, bypassBackupCheck: true })
     }
 
+    if (masterNeedUpdate(error)) {
+      await chooseAction({
+        body: (
+          <p>
+            <Icon icon='alarm' /> {_('slaveHostMoreUpToDateThanMasterAfterReboot')}
+          </p>
+        ),
+        buttons: [{ label: _('rebootAnyway'), btnStyle: 'warning' }],
+        icon: 'alarm',
+        title: _('restartHostModalTitle'),
+      })
+
+      return _restartHost({ ...opts, host, bypassVersionCheck: true })
+    }
+
     if (noHostsAvailableErrCheck(error)) {
       alert(_('noHostsAvailableErrorTitle'), _('noHostsAvailableErrorMessage'))
     }
@@ -926,6 +941,12 @@ const backupIsRunning = (err, poolId) =>
     forbiddenOperation.is(err, {
       reason: `A backup is running on the pool: ${poolId}`,
     }))
+const masterNeedUpdate = err =>
+  err !== undefined &&
+  incorrectState.is(err, {
+    property: 'rebootRequired',
+  })
+
 const noHostsAvailableErrCheck = err => err !== undefined && noHostsAvailable.is(err)
 
 export const restartHosts = (hosts, force = false) => {