New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WFLY-14971 Singleton deployment tests still failing intermittently #14422
Conversation
…Infinispan TopologyChanged events instead of JGroups view change events.
Adding hold label until verified via sufficient CI runs. |
So far, so good. I will focus rerunning Linux SM - JDK 8 jobs, as these test seem to fail most often with that configuration. |
@rhusar Please review. |
This looks good now. No singleton deployment test failures seen. |
/** | ||
* @TypeId(140) | ||
*/ | ||
message GetLocalServicesCommand { | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am under impression that proto types should be commented out rather than removed so the IDs don't get inadvertently reused?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are not yet guaranteeing serialization compatibility across major releases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
https://issues.redhat.com/browse/WFLY-14971