Horizon: incorrect error message when VM deletion failed

Bug #1414164 reported by Danny Choi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Undecided
Wu Wenxiang

Bug Description

nova-manage version: 2014.2.2

Issue: incorrect error message when VM delete failed. It should be "delete", not "launch".

e.g. ERROR: Failed to launch instance XYZ. Please try again later. [Error: delete_port_precommit failed.].

Changed in horizon:
assignee: nobody → Wu Wenxiang (wu-wenxiang)
Revision history for this message
Wu Wenxiang (wu-wenxiang) wrote :

Hello, After checking code, I found scenario maybe as below:
1. "Failed to launch instance" only in get_instance_error() in openstack_dashboard/dashboards/project/instances/tables.py
2. get_instance_error only called in UpdateRow::get_data, also in openstack_dashboard/dashboards/project/instances/tables.py
3. all the operation below related to UpdateRow::get_data, include delete, so "ERROR: Failed to launch instance" should be changed to "Unable to retrieve instance information"
    class Meta:
        name = "instances"
        verbose_name = _("Instances")
        status_columns = ["status", "task"]
        row_class = UpdateRow
        table_actions_menu = (StartInstance, StopInstance, SoftRebootInstance)
        table_actions = (LaunchLink, TerminateInstance, InstancesFilterAction)
        row_actions = (StartInstance, ConfirmResize, RevertResize,
                       CreateSnapshot, SimpleAssociateIP, AssociateIP,
                       SimpleDisassociateIP, EditInstance,
                       DecryptInstancePassword, EditInstanceSecurityGroups,
                       ConsoleLink, LogLink, TogglePause, ToggleSuspend,
                       ResizeLink, LockInstance, UnlockInstance,
                       SoftRebootInstance, RebootInstance,
                       StopInstance, RebuildInstance, TerminateInstance)

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/149881

Changed in horizon:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/149881
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=94ce4953bdbe102018aae51f4c63e9d57555d067
Submitter: Jenkins
Branch: master

commit 94ce4953bdbe102018aae51f4c63e9d57555d067
Author: Wu Wenxiang <wu.wenxiang@99cloud.net>
Date: Sun Jan 25 07:17:59 2015 -0800

    Incorrect error message when VM deletion failed

    Correct error message from "Failed to launch instance" to "Failed to perform
    requested operation on instance, the instance has an error status" in file:
    openstack_dashboard/dashboards/project/instances/tables.py.

    Change-Id: Ieb3f8c83cd55a74b76cd442418a86c53180feed9
    Closes-Bug: #1414164

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → kilo-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-3 → 2015.1.0
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.