pjfanning opened a new pull request, #1877: URL: https://github.com/apache/pekko/pull/1877
relates to #1866 I've been focusing on the InitJoin message but @regiskuckaertz has pointed out that configs are also exchanged in the InitJoinAck message. I'm going to try to add test coverage for this change. I think I need to get some of our tests to do Akka-only checks in the JoinCluster checks when certain configs are set up. At the moment the Pekko JoinCluster checks expect joining nodes to have pekko configs and in mixed cluster setup, this is relaxes to allow pekko or akka. It is probably useful in some of our cluster tests to make this stricter and only accept akka configs. In this case, we would be more likely to catch issues with us not adjusting our configs when interacting with Akka nodes. -- 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. To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org For additional commands, e-mail: notifications-h...@pekko.apache.org
