gurkerl83 commented on pull request #78:
URL: https://github.com/apache/jclouds/pull/78#issuecomment-711058485


   @geomacy My assessment of whether the change is a critical change is as 
follows. 
   
   The main area of concern is the way Apache JClouds is used within Apache 
Brooklyn?
   
   At compile time, runtime or both?
   
   If the integration is done at compile time, or running tests, there are no 
changes.
   
   Runtime
   No change during normal JClouds startup. However, if Apache Brooklyn and 
JClouds are run in an OSGi runtime environment, and Apache Karaf in particular, 
there may be some kind of incompatibility.
   
   I did a little digging in Apache Brooklyn and found references in one of the 
git submodules, that Apache Karaf-JClouds features are loaded and started.
   
   Here is the reference to the corresponding file
   
   
https://github.com/apache/brooklyn-server/blob/ba7fb9112623872de18bd7b6135fdfc1014228ae/karaf/features/src/main/feature/feature.xml
   
   Per se this is no problem, but Apache Karaf-Jclouds project is on a very old 
state, which does not map dependencies of newer Apache JClouds versions.
   
   The stack and libraries used in Apache JClouds change over time; 
consequently, the feature set provided by Apache Karaf-JClouds becomes 
incompatible over time. The creation of Apache Karaf features is mostly a 
manual process; it simply does not keep runtime dependencies up to date.
   
   Possible solution
   When I made the GSON handling changes in June, I created an automatic 
feature generator and aligned the outdated Apache Karaf-JClouds project with 
it. If I find the time in one of the coming weeks I will continue the approach 
and bring it to finalization, a manual adjustment of features especially in a 
project with extensive dependencies must be done automatically in the future.
   
   The detection and resolution of possible incompatibilities must consider the 
launch of Apache Brooklyn, whether each of the outdated feature set of Apache 
Karaf-JClouds can be resolved.
   
   Thx!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to