add sync backup volume option to allow user manually synchronizing ba…

…ckup volumes Signed-off-by: andy.lee <andy.lee@suse.com>
longhorn · May 10, 2024 · 53b3b88 · 53b3b88
1 parent 638c918
commit 53b3b88
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 1 deletion.
diff --git a/src/models/backup.js b/src/models/backup.js
@@ -1,4 +1,4 @@
-import { query, queryBackupList, execAction, restore, deleteBackup, createVolume, deleteAllBackups, getNodeTags, getDiskTags, queryTarget } from '../services/backup'
+import { query, queryBackupList, execAction, restore, deleteBackup, syncVolume, createVolume, deleteAllBackups, getNodeTags, getDiskTags, queryTarget } from '../services/backup'
 import { message } from 'antd'
 import { wsChanges } from '../utils/websocket'
 import queryString from 'query-string'
@@ -246,6 +246,15 @@ export default {
       const search = yield select(store => { return store.backup.search })
       yield put({ type: 'query', payload: { ...search } })
     },
+    *syncBackupVolume({
+      payload,
+    }, { call }) {
+      const resp = yield call(syncVolume, payload.name)
+      if (resp && resp.status === 200) {
+        // console.log('backup model call syncVolume, got resp = ', resp)
+        // do something after got 200.
+      }
+    },
     *bulkCreateVolume({
       payload,
     }, { call, put, select }) {

diff --git a/src/routes/backup/BackupVolumeList.js b/src/routes/backup/BackupVolumeList.js
@@ -87,6 +87,8 @@ class List extends React.Component {
       this.props.DeleteAllBackups(record)
     } else if (e.key === 'restoreLatestBackup') {
       this.props.restoreLatestBackup(record)
+    } else if (e.key === 'syncBackupVolume') {
+      this.props.syncBackupVolume(record)
     } else if (e.key === 'backingImageInfo') {
       this.props.showBackingImageInfo(record)
     }
@@ -248,6 +250,7 @@ class List extends React.Component {
             <DropOption menuOptions={[
               { key: 'recovery', name: 'Create Disaster Recovery Volume', disabled: !record.lastBackupName || (record.messages && record.messages.error) },
               { key: 'restoreLatestBackup', name: 'Restore Latest Backup', disabled: !record.lastBackupName || (record.messages && record.messages.error) },
+              { key: 'syncBackupVolume', name: 'Sync Backup Volume' },
               { key: 'deleteAll', name: 'Delete All Backups' },
               { key: 'backingImageInfo', name: 'Backing Image Info', disabled: !hasBackingImage, tooltip: hasBackingImage ? '' : 'No backing image is used' },
             ]}
@@ -313,6 +316,7 @@ List.propTypes = {
   DeleteAllBackups: PropTypes.func,
   dispatch: PropTypes.func,
   restoreLatestBackup: PropTypes.func,
+  syncBackupVolume: PropTypes.func,
   showBackingImageInfo: PropTypes.func,
   showWorkloadsStatusDetail: PropTypes.func,
 }

diff --git a/src/routes/backup/index.js b/src/routes/backup/index.js
@@ -82,6 +82,12 @@ function Backup({ host, backup, loading, setting, backingImage, dispatch, locati
     showBackingImageInfo(record) {
       showBackingImageInfo(record)
     },
+    syncBackupVolume(record) {
+      dispatch({
+        type: 'backup/syncBackupVolume',
+        payload: record,
+      })
+    },
     restoreLatestBackup(record) {
       dispatch({
         type: 'backup/restoreLatestBackup',

diff --git a/src/services/backup.js b/src/services/backup.js
@@ -31,6 +31,16 @@ export async function createVolume(params) {
   })
 }
 
+export async function syncVolume(volumeName) {
+  return request({
+    url: `/v1/backupvolumes/${volumeName}?action=backupVolumeSync`,
+    method: 'put',
+    data: {
+      syncBackupVolume: true,
+    },
+  })
+}
+
 export async function execAction(url, params) {
   return request({
     url,