Can't delete allocated node even if owned by self

Bug #997092 reported by Gavin Panella
42
This bug affects 10 people
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
High
Raphaël Badin
1.2
Fix Released
High
Julian Edwards

Bug Description

Once lp:~allenap/maas/delete-failing-commissioning-node-bug-994781 lands it won't be possible to delete a node if it's allocated. However, Jeroen points out a valid use-case for this:

> I wonder if we shouldn't also allow a user to delete nodes allocated
> _to themselves_.  “There.  It's running fine now.  I want to keep it
> like this, without further control from MAAS.”

Workaround:

Use the MAAS shell to de-allocate the node. Then you can delete it from the UI.

15:18 <roaksoax> from maasserver.models import Node
15:18 <roaksoax> node = Node.objects.get(hostname='XYZ')
15:18 <roaksoax> node.release()
15:18 <roaksoax> that should do it

Tags: ui qateam

Related branches

Gavin Panella (allenap)
summary: - Can't delete node if allocated even if owned by self
+ Can't delete allocated node even if owned by self
Revision history for this message
Julian Edwards (julian-edwards) wrote :

Given the potential upcoming "all users are admin" change, make it easy to change it so that anyone to delete if necessary.

Changed in maas:
status: New → Triaged
importance: Undecided → High
tags: added: ui
tags: added: qateam
Revision history for this message
Raphaël Badin (rvb) wrote :

As a workaround, one can use the API/CLI to 'release' that node and *then* delete it (from the UI or the API/CLI).

Revision history for this message
Julian Edwards (julian-edwards) wrote :

So, given that we're sticking with multi-tenancy, some points of note:

 - We need to add a "stop" button in the UI.
 - Only admins should be able to delete a node

As a workaround, you can use maas-cli to release a node that was started in the UI, before deleting it as admin.

Robie Basak (racb)
description: updated
Revision history for this message
Andres Rodriguez (andreserl) wrote :

Hi Julian!

Thank you! I have been asking for that 'Stop' button in the UI for so long and nobody paid attention to my request.

And I agree, only admins should be able to delete a node.

Raphaël Badin (rvb)
Changed in maas:
assignee: nobody → Raphaël Badin (rvb)
status: Triaged → Fix Committed
Revision history for this message
Julian Edwards (julian-edwards) wrote :

Anything landing in trunk can go fix-released as soon as the branch lands.

Changed in maas:
status: Fix Committed → Fix Released
Revision history for this message
Adam Stokes (adam-stokes) wrote :

Is this being pushed to precise as well?

Changed in maas:
milestone: none → 13.10
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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