[2.5b3, UI] UI incorrect action notifications that machine cannot be commissioned, deployed, etc.

Bug #1797896 reported by Andres Rodriguez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
High
Caleb Ellis

Bug Description

The UI is now showing a notification incorrectly indicating that machines cannot be deployed (or commissioned, as it is a similar notification), while machines are correctly being deployed/commissioned.

See attached screenshot.

Tags: ui regression

Related branches

Revision history for this message
Andres Rodriguez (andreserl) wrote :
tags: added: ui
tags: added: regression
Changed in maas:
status: New → Triaged
importance: Undecided → High
milestone: none → 2.5.0rc1
assignee: nobody → Anthony Dillon (ya-bo-ng)
Changed in maas:
assignee: Anthony Dillon (ya-bo-ng) → Caleb Ellis (caleb-ellis)
status: Triaged → In Progress
Revision history for this message
Lilyana Videnova (lilyanavidenova) wrote :

The notification should suggest that these machines cannot be deployed/commissioned again right now as they are in the middle of the process, which is not going to be interrupted, thus I suggest the notification should read:

"N nodes are currently being deployed | commissioned. To proceed, update your selection."

Which would correctly explain what is happening and why the user's action cannot be completed.

Revision history for this message
Caleb Ellis (caleb-ellis) wrote :

I've gone with adding a clause to the current message rather than changing the entire message: ", N of which is | are currently in the process". Otherwise, since you can select a few nodes, some of which are deploying, some of which aren't and then choose Deploy from the action menu, the message "N nodes are currently being deployed | commissioned" no longer makes sense. I tried adding two separate warnings, one for normal action errors and one for in progress errors but thought it looked a bit busy.

Changed in maas:
milestone: 2.5.0rc1 → 2.5.0rc2
Revision history for this message
Andres Rodriguez (andreserl) wrote :

@Lilyana,

The error is not that machines cannot be deployed or commissioned. The error is that the message is being shown when the machines *can* be deployed and commissioned.

In other words, you can select 50 machines in 'Ready' state, all of which can be commissioned or deployed, and when you execute the action it *incorrectly* shows the message. The message should only be shown if one or more of the 50 selected machines are in a state that prevents them from being deployed/commissioned.

Revision history for this message
Lilyana Videnova (lilyanavidenova) wrote :

Just to make sure everyone is on the same page, there was a misunderstanding and the fix proposed is not required.

In order to fix this bug, we need to hide the warning message that is displayed _while_ one or more nodes are commissioned/deployed correctly and only display it if there is an actual problem commissioning/deploying a node.

Changed in maas:
milestone: 2.5.0rc2 → 2.5.0rc1
Changed in maas:
status: In Progress → Fix Committed
Changed in maas:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.