jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [jclouds] gurkerl83 commented on pull request #78: Replace embedded and repackaged GSON library
Date Sat, 17 Oct 2020 18:04:37 GMT

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:
users@infra.apache.org



Mime
View raw message