Enhancement at startup of gears and restart of applications (cgroup)#6261
Enhancement at startup of gears and restart of applications (cgroup)#6261IshentRas wants to merge 1 commit intoopenshift:masterfrom IshentRas:enhancement_containers-boost
Conversation
|
Looks reasonable to me, although I could be missing some reason we were not already boosting these operations. @ironcladlou, I think you might have some familiarity with the boost code, right? Also, you authored 8bee9db, which changed |
|
I can't think right now offhand why restart couldn't also be boosted. As far as unidling via the REST API, I remember there was a very good reason that it's necessary, but I'm struggling to remember the details. There might be an old bugzilla issue about it. @jwhonce, I know you and I devised the unidling fix way back... do you happen to remember any more detail? |
|
Yeah, your commit links to https://bugzilla.redhat.com/show_bug.cgi?id=1093776 "Idler doesn't unidle secondary gears in a scaled application". Your change makes sense, but I think it had the side effect of removing boost for unidle; I thought it was worth mentioning the commit because this PR restores that boost. |
|
History: V1 only idled the first gear in an application. That was ported to V2 but Online found a lot of scaled applications where node capacity could be increased if we idled all the gears in the application. Ie, a scaled database application that was created but never used. Pre @ironcladlou 's PR the database gear never idled. To idle and then unidle the whole application required orchestration from the Broker. A potential issue with this PR is boosting a boosted gear during gear create and updates. The two test cases I see are that: 1) boosting a boosted gear is restored to default after the outer-most boosted block is finished, and 2) that the gear doesn't loose boost after the embedded boost block is finished. |
|
Thanks all for the comments so far. As far as I can tell, it has been a real issue at the client's side (starting,restarting JEE application) and I testing the below has definitely boosted the platform. |
cg_template_boosted was originally created for handling application creation events (Which require extra resources to start) for instance JEE apps.
This PR is an enhancement for handling applications restart or gears start/restart actions as they also require the same boost.
Seen at customer's side (PM if you require more info)