-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Speed up installation of a large number of packages #155
Comments
thanks for reporting, will be implemented within this month ;) |
when deploying a package that is changing some configuration of |
thinking about sth like that: then such @tomasz-sobczyk-wttech , @maciej-majchrzak-wttech WDYT? or maybe just there should be 2 different done_thresholds
|
This
|
regarding second point - will be addressed by #159 |
Overview
Today every package installation has it's own health checks - there is a param called done_threshold that defines how many times we check if system is stable before announcing package is installed.
Reducing the value of this parameter to sth like 2 is a good idea to speed things up. But!
There are cases where you might want to double-triple-quadruple check the stability - example is when your package contaings Sling Maping changes. Such a change can cause an asynchronously invoked (event driven) restart of most-if-not-all AEM bundles.
How to eat the cake and have the cake? A.k.a. potential solution:
Introduce (apart from existing global configuration for done_threshold) a new configuration where I will be able to:
This should be an array type of data (multiple configurations allowed)
Code should verify the package being installed against the list provided, first matched configuration should be picked up.
If not matching configuration is found - it should pick up the value from global config.
This configuration can be still overwritten from the command line (CLI direct params should always take precedense)
Regexp?
Why regexp for package name: Package names have versions in them often etc - let's keep it open ended.
The text was updated successfully, but these errors were encountered: