[GitHub] [openwhisk] vrann commented on pull request #5065: Upgrade to Akka 2.6.12

[GitBox <gi...@apache.org>]

vrann commented on pull request #5065:
URL: https://github.com/apache/openwhisk/pull/5065#issuecomment-790240892


   @bdoyle0182 @rabbah I have few more questions regarding the upgrade.
   1. Upgrade to 2.6.x brings also the usage of artery for the networking. Per akka documentation, when upgrading live systems, if you can't stop the cluster, upgrade should happen in two stages: first -- upgrade to 2.6 with Artery disabled and 2nd -- enable artery. I think this is what you @bdoyle0182 also mentioned? How do we want to proceed with it in this PR? I can see one option is to have first PR with artery disabled released in the minor version and then another PR which enables artery to release in patch version. In this way you can perform two-step upgrade on your end. But looking into release history of the openwhisk I'm not sure what is the general strategy towards releases, would it be really something community would use.
   2. I have strange issue with the Java Flight Recorder. It complains for the missing class in classpath -- NoClassDefFoundError: jdk.jfr.Event. I disabled flight recording in the akka and now it works, but would appreciate any comment on the better fix. I understand that JFR being proprietary tool might not exist on my image, but why would it then be expected to present in class path by akka?


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