Bug 1300899 - Still show "Cancelling" message while the build/deployment has be cancelled
Summary: Still show "Cancelling" message while the build/deployment has be cancelled
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Management Console
Version: 3.x
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Jessica Forrester
QA Contact: Yadan Pei
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-22 02:38 UTC by Xingxing Xia
Modified: 2016-12-09 21:50 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-12-09 21:50:50 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
An error occurred while cancelling the deployment (118.89 KB, image/png)
2016-11-15 10:12 UTC, Xingxing Xia
no flags Details

Description Xingxing Xia 2016-01-22 02:38:34 UTC
Description of problem:
Cancel a running build/deployment, then it will show message like "Cancelling build ruby-sample-build-2 of ruby-sample-build". While it has been cancelled, the message should turn to "cancelled" instead of keep "Cancelling".

Version-Release number of selected component (if applicable):
openshift/oc v1.1.1-93-g6a3bd56
kubernetes v1.1.0-origin-1107-g4c8e6f4

How reproducible:
Always

Steps to Reproduce:
1. Login into web console, create project and new-app containing bc/dc
2. Restart a build, and cancel it, then check the bar message
3. Deploy a deployment, and cancel it, then check the bar message

Actual results:
2. The bar message shows "Cancelling build ruby-sample-build-2 of ruby-sample-build", but after it has been cancelled, the message still shows "Cancelling"
3. Same issue for cancelling deployment

Expected results:
2 and 3, after it has been cancelled, the message should turn to "Cancelled" instead of keeping "Cancelling"

Additional info:

Comment 1 Jessica Forrester 2016-11-14 16:10:31 UTC
Switching to always say Cancelled since by the time we show the message we know the API request to cancel completed successfully.

Comment 2 Xingxing Xia 2016-11-15 10:12:01 UTC
Created attachment 1220796 [details]
An error occurred while cancelling the deployment

Comment 3 Xingxing Xia 2016-11-15 10:13:40 UTC
(In reply to Jessica Forrester from comment #1)
> Switching to always say Cancelled since by the time we show the message we
> know the API request to cancel completed successfully.

FYI, not sure if it will be affected in the situation when cancellation sometimes shows error like:

An error occurred while cancelling the deployment.Operation cannot be fulfilled on replicationcontrollers "hooks-7": the object has been modified; please apply your changes to the latest version and try again 

See attachment.

Comment 4 Jessica Forrester 2016-11-15 14:18:32 UTC
The error messages are worded correctly, we can leave those.

Comment 5 Jessica Forrester 2016-11-15 14:19:07 UTC
The fix is in https://github.com/openshift/origin-web-console/pull/867

just waiting for everything to open up for 1.5 to merge it

Comment 6 openshift-github-bot 2016-11-17 17:01:37 UTC
Commit pushed to master at https://github.com/openshift/origin-web-console

https://github.com/openshift/origin-web-console/commit/d8ffe9df78c119778a230560f2e1ece9aec39bc4
Bug 1300899 Switch success messages from saying Cancelling to Cancelled

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1300899

Comment 7 Yanping Zhang 2016-11-18 04:51:10 UTC
Checked on openshift v1.4.0-alpha.1+63e3797-264, now cancel running build/deployment it will show a cancelled info, such as "Cancelled build eap-app-3 of eap-app.","Cancelled deployment eap-app-4 of eap-app."
The bug has been fixed, so move it to Verified.


Note You need to log in before you can comment on or make changes to this bug.