Bug 969876 - Setting the scale info on a cartridge to the same value results in connection hooks being run
Summary: Setting the scale info on a cartridge to the same value results in connection...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Pod
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Ravi Sankar
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-02 21:08 UTC by Clayton Coleman
Modified: 2015-05-15 00:17 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-22 15:15:25 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Clayton Coleman 2013-06-02 21:08:05 UTC
Created a scaled 3rd party cartridge, and then set the minimum scale to 2.  Operation succeeded.  Set minimum scale to 2 again, and noticed that all connection hooks were rerun.  If the scale value doesn't change (if scale up is not required) the second operation should be a no-op.

1) rhc create-app foo php-5.3 -s
2) rhc add-cartridge "http://cartreflect-claytondev.rhcloud.com/reflect?github=smarterclayton/openshift-redis-cart&commit=6dca180bb06412a261d7211d9d8fe1211afe32d8" -a foo
3) rhc cartridge-scale redis -a foo --min 2

<creates a second gear>

4) rhc cartridge-scale redis -a foo --min 2

<runs publish hooks again, should do nothing>

Does not block sprint exit.  Is a performance and potential app availability issue (if connection hooks have side effects).

Comment 1 Xiaoli Tian 2013-06-03 01:21:32 UTC
Move it to Online product according to recent workflow about filing bugs, and the release blocker list may only count Online bugs as well.

Comment 2 Xiaoli Tian 2013-06-03 04:07:56 UTC
(In reply to Clayton Coleman from comment #0)
> <runs publish hooks again, should do nothing>
> 
> Does not block sprint exit.  Is a performance and potential app availability
> issue (if connection hooks have side effects).

Add a futurefeature keyword for now to remove it from block list according to comment 0.

Comment 3 openshift-github-bot 2013-07-15 18:18:48 UTC
Commits pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/0d2f10a3baf6e632b7eede1839ea93ebde5ce3b4
Bug 969876 - Don't run execute_connections if there are no add/remove component ops

https://github.com/openshift/origin-server/commit/aa14b659e97504822d0424c6346ceb7b39074b38
Merge pull request #3072 from pravisankar/dev/ravi/bug969876

Merged by openshift-bot

Comment 4 Jianwei Hou 2013-07-16 12:06:14 UTC
Verified on devenv_3508

1) rhc create-app foo php-5.3 -s
2) rhc add-cartridge "http://cartreflect-claytondev.rhcloud.com/reflect?github=smarterclayton/openshift-redis-cart&commit=6dca180bb06412a261d7211d9d8fe1211afe32d8" -a foo
3) rhc cartridge-scale redis -a foo --min 2
4) rhc cartridge-scale redis -a foo --min 2

The publisher hooks won't be called on the second time


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