storage: Disable "Apply" in Teardown dialogs after error

We want the user to cancel and start over.

pkg/storaged/content-views.jsx

@@ -433,6 +433,7 @@ function create_tabs(client, target, options) {
                 Action: {
                     Danger: danger,
                     Title: _("Delete"),
+                    disable_on_error: usage.Teardown,
                     action: function () {
                         return utils.teardown_active_usage(client, usage)
                                 .then(function () {
@@ -752,6 +753,7 @@ function format_disk(client, block) {
             Title: _("Initialize"),
             Danger: _("Initializing erases all data on a disk."),
             wrapper: job_progress_wrapper(client, block.path),
+            disable_on_error: usage.Teardown,
             action: function (vals) {
                 const options = {
                     'tear-down': { t: 'b', v: true }

pkg/storaged/dialog.jsx

@@ -400,6 +400,7 @@ export const dialog_open = (def) => {
                         }
                         errors = errs;
                         update();
+                        update_footer();
                         return Promise.reject();
                     });
         };
@@ -414,7 +415,7 @@ export const dialog_open = (def) => {
                     caption: def.Action.Title,
                     style: "primary",
                     danger: def.Action.Danger || def.Action.DangerButton,
-                    disabled: running_promise != null,
+                    disabled: running_promise != null || (def.Action.disable_on_error && errors),
                     clicked: progress_callback => run_action(progress_callback, null),
                 }
             ];

pkg/storaged/format-dialog.jsx

@@ -396,6 +396,7 @@ function format_dialog_internal(client, path, start, size, enable_dos_extended,
             Danger: (create_partition ? null : _("Formatting erases all data on a storage device.")),
             Variants: [action_variant],
             wrapper: job_progress_wrapper(client, block.path, client.blocks_cleartext[block.path]?.path),
+            disable_on_error: usage.Teardown,
             action: function (vals) {
                 const mount_now = vals.variant != "nomount";
 

pkg/storaged/fsys-tab.jsx

@@ -414,6 +414,7 @@ export function mounting_dialog(client, block, mode, forced_options) {
         },
         Action: {
             Title: mode_action[mode],
+            disable_on_error: usage.Teardown,
             action: function (vals) {
                 if (mode == "unmount") {
                     return do_unmount();

pkg/storaged/mdraid-details.jsx

@@ -180,6 +180,7 @@ function mdraid_stop(client, mdraid) {
             Teardown: TeardownMessage(usage),
             Action: {
                 Title: _("Stop device"),
+                disable_on_error: true,
                 action: function () {
                     return utils.teardown_active_usage(client, usage)
                             .then(function () {
@@ -235,6 +236,7 @@ function mdraid_delete(client, mdraid) {
         Action: {
             Title: _("Delete"),
             Danger: _("Deleting erases all data on a RAID device."),
+            disable_on_error: usage.Teardown,
             action: function () {
                 return utils.teardown_active_usage(client, usage)
                         .then(delete_)

pkg/storaged/resize.jsx

@@ -428,6 +428,7 @@ export function grow_dialog(client, lvol_or_part, info, to_fit) {
         },
         Action: {
             Title: _("Grow"),
+            disable_on_error: usage.Teardown,
             action: function (vals) {
                 return teardown_active_usage(client, usage)
                         .then(function () {
@@ -541,6 +542,7 @@ export function shrink_dialog(client, lvol_or_part, info, to_fit) {
         Fields: size_fields.concat(passphrase_fields),
         Action: {
             Title: _("Shrink"),
+            disable_on_error: usage.Teardown,
             action: function (vals) {
                 return teardown_active_usage(client, usage)
                         .then(function () {

pkg/storaged/stratis-details.jsx

@@ -408,6 +408,7 @@ export function stratis_content_rows(client, pool, options) {
                 Action: {
                     Danger: _("Deleting a filesystem will delete all data in it."),
                     Title: _("Delete"),
+                    disable_on_error: usage.Teardown,
                     action: function () {
                         return teardown_active_usage(client, usage)
                                 .then(() => destroy_filesystem(client, fsys));
@@ -594,6 +595,7 @@ function delete_pool(client, pool) {
         Action: {
             Danger: _("Deleting a Stratis pool will erase all data it contains."),
             Title: _("Delete"),
+            disable_on_error: usage.Teardown,
             action: function () {
                 return teardown_active_usage(client, usage)
                         .then(() => destroy_pool(client, pool))

pkg/storaged/vdo-details.jsx

@@ -138,6 +138,7 @@ export class VDODetails extends React.Component {
                     Teardown: TeardownMessage(usage),
                     Action: {
                         Title: _("Stop"),
+                        disable_on_error: usage.Teardown,
                         action: function () {
                             return teardown_active_usage(client, usage)
                                     .then(function () {
@@ -199,6 +200,7 @@ export class VDODetails extends React.Component {
                 Action: {
                     Title: _("Delete"),
                     Danger: _("Deleting erases all data on a VDO device."),
+                    disable_on_error: usage.Teardown,
                     action: function () {
                         return (teardown_active_usage(client, usage)
                                 .then(teardown_configs)

pkg/storaged/vgroup-details.jsx

@@ -173,6 +173,7 @@ export function vgroup_delete(client, vgroup) {
         Action: {
             Danger: _("Deleting erases all data on a volume group."),
             Title: _("Delete"),
+            disable_on_error: usage.Teardown,
             action: function () {
                 return utils.teardown_active_usage(client, usage)
                         .then(function () {
@@ -284,6 +285,7 @@ export class VGroupDetails extends React.Component {
                 Teardown: TeardownMessage(usage),
                 Action: {
                     Title: _("Remove"),
+                    disable_on_error: usage.Teardown,
                     action: function () {
                         return utils.teardown_active_usage(client, usage)
                                 .then(function () {

test/common/storagelib.py

@@ -317,6 +317,9 @@ def dialog_wait_not_present(self, field):
     def dialog_wait_apply_enabled(self):
         self.browser.wait_attr('#dialog button.apply:nth-of-type(1)', "disabled", None)
 
+    def dialog_wait_apply_disabled(self):
+        self.browser.wait_visible('#dialog button.apply:nth-of-type(1)[disabled]')
+
     def dialog_apply(self):
         self.browser.click('#dialog button.apply:nth-of-type(1)')
 

test/verify/check-storage-used

@@ -133,6 +133,39 @@ ExecStart=/usr/bin/sleep infinity
 
         self.assertEqual(m.execute("vgs TEST1 || echo GONE").strip(), "GONE")
 
+    def testTeardownRetry(self):
+        m = self.machine
+        b = self.browser
+
+        self.login_and_go("/storage")
+
+        disk = self.add_ram_disk()
+        b.wait_in_text("#drives", disk)
+        m.execute(f"mke2fs -q -L TEST {disk}")
+        m.execute(f"mount {disk} /mnt")
+
+        b.click(f'.sidepanel-row:contains("{disk}")')
+        b.wait_visible("#storage-detail")
+
+        # Start formatting, and while the dialog is open, make the
+        # filesystem unmountable.
+
+        b.click('button:contains(Create partition table)')
+        self.dialog_wait_open()
+        self.dialog_wait_apply_enabled()
+        sleep_pid = m.spawn("cd /mnt; sleep infinity; true", "sleep")
+        self.dialog_apply()
+        b.wait_in_text("#dialog", "umount: /mnt: target is busy")
+        self.dialog_wait_apply_disabled()
+        self.dialog_cancel()
+        self.dialog_wait_close()
+
+        b.click('button:contains(Create partition table)')
+        self.dialog_wait_open()
+        b.wait_visible("#dialog button:contains(Currently in use)")
+        self.dialog_apply()
+        self.dialog_wait_close()
+
 
 if __name__ == '__main__':
     testlib.test_main()