Bug 971280

Summary: "{"exit_code"=>nil, "field"=>nil, "severity"=>"info", "text"=>"Application 'zendtest' found"}" is shown when restart app
Product: OpenShift Online Reporter: Wei Sun <wsun>
Component: WebsiteAssignee: Jordan Liggitt <jliggitt>
Status: CLOSED CURRENTRELEASE QA Contact: libra bugs <libra-bugs>
Severity: medium Docs Contact:
Priority: medium    
Version: 2.xCC: ccoleman, jliggitt, xtian, yujzhang
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-06-24 14:51:51 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
restartapp none

Description Wei Sun 2013-06-06 07:49:27 UTC
Description of problem:
When restart the app,some messages will be shown:
{"exit_code"=>nil, "field"=>nil, "severity"=>"info", "text"=>"Application 'zendtest' found"}
please refer to attachment

Version-Release number of selected component (if applicable):
devenv_3324

How reproducible:
Always

Steps to Reproduce:
1.Create an application
2.Restart the app
3.

Actual results:
"{"exit_code"=>nil, "field"=>nil, "severity"=>"info", "text"=>"Application 'zendtest' found"}" is shown

Expected results:
Don't have this message

Additional info:

Comment 1 Wei Sun 2013-06-06 08:52:48 UTC
Created attachment 757554 [details]
restartapp

Comment 2 Jordan Liggitt 2013-06-06 14:28:55 UTC
same issue exists when deleting an app alias

Comment 3 Clayton Coleman 2013-06-07 16:14:59 UTC
This is due to the 1.5 change to return 200 content on deletes.  This seems wrong though - as in either the broker is returning poorly formatted JSON, or a bad message.

Comment 4 Jordan Liggitt 2013-06-07 17:00:29 UTC
The broker is fine. We're reading the messages off the application object, which isn't getting updated from the remove_alias or restart! methods

Comment 5 Jordan Liggitt 2013-06-13 20:10:45 UTC
Marked as [merge] in https://github.com/openshift/origin-server/pull/2775

Comment 6 openshift-github-bot 2013-06-14 00:29:55 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/49862a32804b5645f95bc12419d5f2f88e8f59e7
Fix bug 971280 - make app restart message correct

Update restart controller test

Comment 7 Yujie Zhang 2013-06-14 06:22:10 UTC
Tested on devenv_3360, the return message after restarting app is correct now, so verify this bug, thanks.