Success message shown when deleting a volume /volume snapshot

Bug #1291681 reported by mariam john
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Wishlist
Adrian Turjak

Bug Description

Click on Project -> Volumes. Create a new volume and volume snapshot. Click on the Delete Volume/Delete Snapshot button and it will display the following message:

Success: Schedule deletion of volume/volume snapshot {name}

Since the deletion is not complete at that point, isnt it appropriate to show this as an 'Info' message instead of 'Success' (like how it shows when a volume/volume snapshot is created)

Changed in horizon:
assignee: nobody → Santiago Baldassin (santiago-b-baldassin)
Changed in horizon:
assignee: Santiago Baldassin (santiago-b-baldassin) → nobody
mariam john (mariamj)
Changed in horizon:
assignee: nobody → mariam john (mariamj)
Revision history for this message
Akihiro Motoki (amotoki) wrote :

The same thing applies to the message when deleting an instance. It seems better to check which is better INFO message or SUCCESS message throughout OpenStack Dashboard.

Revision history for this message
mariam john (mariamj) wrote :

Thank you Akihiro for your comments. Personally I think this should be an INFO message when deleting a volume/instance since the actual deletion process is not complete at that point. What does the SUCCESS message in here stand for - that the deletion process has begun successfully? To me it looks a little misleading especially when I see this success message and then a delete actually fails. Any thoughts?

Revision history for this message
Julie Pichon (jpichon) wrote :

FWIW we've worked around this so far by wording the success messages to be about "scheduling" rather than the action itself. I agree it's not ideal. I suspect fixing it to display messages at the info level when appropriate may require a blueprint, though I haven't looked into that part of the code for a while.

Revision history for this message
Akihiro Motoki (amotoki) wrote :

bug 1286418 is related.

tags: added: ux
tags: added: error-reporting
Changed in horizon:
status: New → Confirmed
importance: Undecided → Wishlist
Varsha R (varshar186)
Changed in horizon:
assignee: mariam john (mariamj) → Varsha R (varshar186)
Revision history for this message
Sunkara Ramya Sree (ramyasunkara) wrote :

Can I work on this bug?

Changed in horizon:
assignee: Varsha R (varshar186) → Simon Alexander Kerr Collins (bird-jesus)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/534586

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Ivan Kolodyazhny (<email address hidden>) on branch: master
Review: https://review.openstack.org/534586
Reason: This review is > 4 months without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/628103

Changed in horizon:
assignee: Simon Alexander Kerr Collins (bird-jesus) → Adrian Turjak (adriant-y)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/628103
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=2a3b04ce6cf4e3a48b893a4ee2dc9f8f246938b1
Submitter: Zuul
Branch: master

commit 2a3b04ce6cf4e3a48b893a4ee2dc9f8f246938b1
Author: Simon Collins <email address hidden>
Date: Wed Jan 17 15:08:34 2018 +1300

    Changed the message level for deleting some resources to info

    Added functionality to the handle function in the BatchAction class
    in action.py and allowed the easy changing between the alert types
    and made deleting images, instances, and volumes give an info alert
    rather than a success message and altered the testing parameters

    Change-Id: I93251d6d12cf89a331b5c051bf4fe10a6e470f51
    Closes-Bug: #1291681

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 15.0.0.0b2

This issue was fixed in the openstack/horizon 15.0.0.0b2 development milestone.

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.