Bug 889947 - Remove current scale from cartridges
Summary: Remove current scale from cartridges
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Master
Version: 1.x
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Lili Nader
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-24 06:10 UTC by Mengjiao Gao
Modified: 2014-10-26 20:41 UTC (History)
6 users (show)

Fixed In Version: fork_ami_refctr1_386
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-05-15 15:28:02 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
The info in the app creation page said that the app cannot be scaled. (171.49 KB, image/png)
2012-12-24 06:10 UTC, Mengjiao Gao
no flags Details

Description Mengjiao Gao 2012-12-24 06:10:14 UTC
Created attachment 668318 [details]
The info in the app creation page said that the app cannot be scaled.

Description of problem:
  In the app creation page, the message "This application shares resources and can't be scaled." is shown in the Scaling part, and there's no choice list for user to choose to create a scalable app. The app creation page is shown as the attachment.

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

How reproducible:
always

Steps to Reproduce:
1.Go to website and login.
2.Check the app's creation page.
3.Try to create a scalable app.
  
Actual results:
At step 2, the message "This application shares resources and can't be scaled." will be shown.
Then user cannot be able to create a scalable app at step 3.

Expected results:
User should be able to choose to create a scalable app.

Additional info:

Comment 1 Dan McPherson 2013-01-04 01:35:49 UTC
Lili, can you take a look at this one.  It appears none of the scaling properties are being returned with the rest_cartridge (broker/rest/cartridges.json) call.

Comment 2 Mengjiao Gao 2013-01-05 05:56:22 UTC
Tested this issue on fork_ami_refctr1_381, I still couldn't create all scalable non-templates applications. 
But I can create scalable template apps (including spring, rubyonrail, wordpress, django, capedrawf, cakephp and kit) .

Comment 3 Lili Nader 2013-01-07 19:14:39 UTC
fork_ami_refctr1_386

Comment 4 Mengjiao Gao 2013-01-08 02:37:23 UTC
Verified this issue on fork_ami_refctr1_387, user can change the scaling item of non-template app to create a scalable application now. Thanks!

Comment 5 Clayton Coleman 2014-01-17 16:15:38 UTC
There are some minor issues with the fix.  A new value "current_scale" has shown up in /broker/rest/cartridges that should not be there.

Comment 7 Dan McPherson 2014-04-25 21:49:41 UTC
Moving back to modified until it's at least being merged.

Comment 8 openshift-github-bot 2014-04-29 18:40:58 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/1ce59d082956e446554ac75c0798431ab30fcb48
Bugs 1084980 and 889947

Comment 9 Lili Nader 2014-04-29 20:43:22 UTC
Note current_scale is only removed from GET /cartridges and GET /cartridge/id.  It still exists for GET /application/id/cartridges and GET /application/id/cartridge/id

Comment 10 Jianwei Hou 2014-04-30 03:22:51 UTC
Verified on devenv_4739

The 'current_scale' is removed from GET /cartridges and GET /cartridge/id
The 'current_scale' remains in GET /application/id/cartridges and GET /application/id/cartridge/id


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