Github user rdowner commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/975#discussion_r206111336
  
    --- Diff: karaf/features/src/main/feature/feature.xml ---
    @@ -274,8 +275,40 @@
         <feature name="brooklyn-container-service" 
version="${project.version}" description="Brooklyn Container Service and 
Location Targets">
             <bundle 
start-level="85">mvn:org.apache.brooklyn/brooklyn-locations-container/${project.version}</bundle>
     
    +        <feature>brooklyn-core</feature>
    +<!-- previously we used these, but they pull in wrong versions - jackson 
2.7.5 and snakeyaml 1.17 
    +        <feature>kubernetes-client</feature>
    +        <feature>openshift-client</feature>
    +-->
    +<!-- these are pulled in elsewhere
    +        <bundle 
dependency='true'>mvn:javax.validation/validation-api/1.1.0.Final</bundle>
    +        <bundle 
dependency='true'>mvn:javax.servlet/javax.servlet-api/3.1.0</bundle>
    +-->
    +<!-- these are wrong version, right version pulled in elsewhere
    --- End diff --
    
    Why leave them here, commented out, if they're wrong? Just delete?


---

Reply via email to