You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/04/19 07:03:12 UTC

[GitHub] [pulsar] lhotari opened a new pull request #10260: Upgrade k8s java-client to fix dependency resolution

lhotari opened a new pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260


   ### Motivation 
   
   - sundr library has been upgraded to 0.22.1 version which contains a fix a dependency resolution issue, https://github.com/sundrio/sundrio/issues/168
     - various workarounds for sundr library dependency issues can be removed
   
   - newer version of k8s java-client has fewer dependencies
   
   ### Modifications
   
   - upgrade io.kubernetes:java-client version from 10.0.0 to 12.0.0
   - add dependency management rules for com.google.errorprone:error_prone_annotations and com.google.j2objc:j2objc-annotations so that the dependency versions are consistent with the other dependencies.
   - adapt license files to the modifications
   - adapt usage of `coreClient.listNamespacedPod` since the api has changed (1 new parameter, passes as null)
   - remove workarounds that have been in place to address issues with sundr library dependency resolution issue
   - reformat `pulsar-sql/presto-distribution/pom.xml`


-- 
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

[GitHub] [pulsar] lhotari commented on pull request #10260: Upgrade k8s java-client to fix dependency resolution

Posted by GitBox <gi...@apache.org>.
lhotari commented on pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260#issuecomment-822262825


   /pulsarbot run-failure-checks


-- 
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

[GitHub] [pulsar] lhotari commented on pull request #10260: Upgrade k8s java-client to fix dependency resolution

Posted by GitBox <gi...@apache.org>.
lhotari commented on pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260#issuecomment-822547486


   @jerrypeng What would be the best way to verify this PR?


-- 
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

[GitHub] [pulsar] lhotari commented on pull request #10260: Upgrade k8s java-client to fix dependency resolution

Posted by GitBox <gi...@apache.org>.
lhotari commented on pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260#issuecomment-822398417


   /pulsarbot run-failure-checks


-- 
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

[GitHub] [pulsar] lhotari commented on pull request #10260: Upgrade k8s java-client to fix dependency resolution

Posted by GitBox <gi...@apache.org>.
lhotari commented on pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260#issuecomment-822274521


   @jerrypeng @david-streamlio Please review this change.


-- 
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

[GitHub] [pulsar] lhotari commented on pull request #10260: Upgrade k8s java-client to fix dependency resolution

Posted by GitBox <gi...@apache.org>.
lhotari commented on pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260#issuecomment-822436160


   /pulsarbot run-failure-checks


-- 
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

[GitHub] [pulsar] sijie merged pull request #10260: Upgrade k8s java-client to fix dependency resolution

Posted by GitBox <gi...@apache.org>.
sijie merged pull request #10260:
URL: https://github.com/apache/pulsar/pull/10260


   


-- 
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