mgr/dashboard: Fix missing failed restore notification

Fixes: http://tracker.ceph.com/issues/36513
Signed-off-by: Tiago Melo <tmelo@suse.com>

diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/block/rbd-trash-restore-modal/rbd-trash-restore-modal.component.ts b/src/pybind/mgr/dashboard/frontend/src/app/ceph/block/rbd-trash-restore-modal/rbd-trash-restore-modal.component.ts
index c471a0151e8fea19e8c4c54513ea21f80a0d6899..42794419821324fd20af575cec626ceae4a2ba8d 100644 (file)
--- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/block/rbd-trash-restore-modal/rbd-trash-restore-modal.component.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/block/rbd-trash-restore-modal/rbd-trash-restore-modal.component.ts
@@ -44,7 +44,6 @@ export class RbdTrashRestoreModalComponent implements OnInit {
         task: new FinishedTask('rbd/trash/restore', {
           pool_name: this.poolName,
           image_id: this.imageId,
-          image_name: this.imageName,
           new_image_name: name
         }),
         call: this.rbdService.restoreTrash(this.poolName, this.imageId, name)

diff --git a/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.spec.ts b/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.spec.ts
index f2615248eecf2e2d69fc57787c9ab6c3947844bd..c8f0ae67706cf87a7eba345cc675ab26755d57a8 100644 (file)
--- a/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.spec.ts
@@ -91,19 +91,22 @@ describe('TaskManagerMessageService', () => {
     });
 
     describe('rbd tasks', () => {
+      let metadata;
       let childMsg: string;
       let destinationMsg: string;
       let snapMsg: string;
 
       beforeEach(() => {
-        const metadata = {
+        metadata = {
           pool_name: 'somePool',
           image_name: 'someImage',
           snapshot_name: 'someSnapShot',
           dest_pool_name: 'someDestinationPool',
           dest_image_name: 'someDestinationImage',
           child_pool_name: 'someChildPool',
-          child_image_name: 'someChildImage'
+          child_image_name: 'someChildImage',
+          new_image_name: 'newImage',
+          image_id: '12345'
         };
         defaultMsg = `RBD '${metadata.pool_name}/${metadata.image_name}'`;
         childMsg = `RBD '${metadata.child_pool_name}/${metadata.child_image_name}'`;
@@ -172,6 +175,41 @@ describe('TaskManagerMessageService', () => {
         finishedTask.name = 'rbd/snap/rollback';
         testMessages(new TaskMessageOperation('Rolling back', 'rollback', 'Rolled back'), snapMsg);
       });
+
+      it('tests rbd/trash/move messages', () => {
+        finishedTask.name = 'rbd/trash/move';
+        testMessages(
+          new TaskMessageOperation('Moving', 'move', 'Moved'),
+          `image '${metadata.pool_name}/${metadata.image_name}' to trash`
+        );
+        testErrorCode(2, `Could not find image.`);
+      });
+
+      it('tests rbd/trash/restore messages', () => {
+        finishedTask.name = 'rbd/trash/restore';
+        testMessages(
+          new TaskMessageOperation('Restoring', 'restore', 'Restored'),
+          `image '${metadata.pool_name}@${metadata.image_id}' \
+        into '${metadata.pool_name}/${metadata.new_image_name}'`
+        );
+        testErrorCode(
+          17,
+          `Image name '${metadata.pool_name}/${metadata.new_image_name}' is already in use.`
+        );
+      });
+
+      it('tests rbd/trash/remove messages', () => {
+        finishedTask.name = 'rbd/trash/remove';
+        testDelete(`image '${metadata.pool_name}/${metadata.image_name}@${metadata.image_id}'`);
+      });
+
+      it('tests rbd/trash/purge messages', () => {
+        finishedTask.name = 'rbd/trash/purge';
+        testMessages(
+          new TaskMessageOperation('Purging', 'purge', 'Purged'),
+          `images from '${metadata.pool_name}'`
+        );
+      });
     });
   });
 });

diff --git a/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.ts b/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.ts
index 4f42e37be6836af1df45438d24e3200eb652fcf5..9cea92c3c0520aaca7fb1340d4f9a74419913f99 100644 (file)
--- a/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/shared/services/task-message.service.ts
@@ -146,7 +146,7 @@ export class TaskMessageService {
     'rbd/trash/restore': new TaskMessage(
       new TaskMessageOperation('Restoring', 'restore', 'Restored'),
       (metadata) =>
-        `image '${metadata.pool_name}/${metadata.image_name}@${metadata.image_id}' \
+        `image '${metadata.pool_name}@${metadata.image_id}' \
         into '${metadata.pool_name}/${metadata.new_image_name}'`,
       (metadata) => ({
         17: `Image name '${metadata.pool_name}/${metadata.new_image_name}' is already in use.`