Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Confusing UI - "Use secure erase" and "Use quick erase" are supposed to be subsidiary of "Erase nodes' disk" #4802

Open
webteam-app opened this issue Mar 8, 2023 · 2 comments

Comments

@webteam-app
Copy link

Bug originally filed by nobuto at https://bugs.launchpad.net/bugs/2009669

maas: 1:3.3.0-13159-g.1c22f7beb-0ubuntu1~22.04.1
maas-ui: 3.3.0 acde9f1

In WebUI as MAAS/r/settings/storage, there is disk erase related configurations.

  1. Erase nodes' disks prior to releasing (True/False)
  2. Use secure erase by default when erasing disks (True/False)
  3. Use quick erase by default when erasing disks (True/False)

Those 3 are presented equally at the same UI level in the WebUI. However,

2&3 do not take effect unless 1 is True, so it would be nice if

  • 2&3 are grayed out if 1 is False.
  • 2&3 are displayed at a subsidiary level of 1 in the UI (indented for example)

2&3 are mutually exclusive by design and MAAS doesn't not allow to release a node if both 2&3 are enabled so it would be nice if

  • the fact that 2&3 are mutually exclusive is reflected in the UI (radio box or something)
@nobuto-m
Copy link

nobuto-m commented Mar 8, 2023

Screenshot from 2023-03-08 15-08-03

Screenshot from 2023-03-08 15-14-51

@nobuto-m
Copy link

nobuto-m commented Mar 8, 2023

2&3 are mutually exclusive by design and MAAS doesn't not allow to release a node if both 2&3 are enabled

I have a second thought on this. API operations actually allows both to be set. And the API client's help explains the behavior at least from a documentation point of view.

$ maas admin machine release -h
...

If both secure_erase and quick_erase are specified and the
drive does NOT have a secure erase feature, MAAS will behave as if only
quick_erase was specified.

So the mutually-exclusive behavior itself can be the bug in the UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants