MINOR: Apply extra serialized rsync step to both parallel and serial paths#3546
MINOR: Apply extra serialized rsync step to both parallel and serial paths#3546ewencp wants to merge 1 commit intoapache:trunkfrom
Conversation
|
@enothereska @ijuma @hachikuji @dguy Pinging all the folks annoyed by environmental issues. We should still fix the nightlies to do parallel vagrant up and I'm not sure why it's not enabled yet, but this makes the rsync fix work for both paths. I tested myself on AWS with both parallel and serial cluster up. |
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
ijuma
left a comment
ijuma
left a comment
There was a problem hiding this comment.
LGTM if the system tests pass with this change.
|
Fired off a job here, though given the massive failures from https://issues.apache.org/jira/browse/KAFKA-5608, who knows whether it'll be meaningful or not... |
|
The system test didn't go through, but the patch looks reasonable. |
|
The system test run failed during rsync so that's a bit concerning. I restarted it. |
|
I believe this is the job you started. All the failures appear to be related to the JmxTool issues. However, now that that issue has been resolved, I've restarted the job once again after rebasing against the latest trunk, running how here. |
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
|
System tests have passed now that the other patch went through, so committing and cherry-picking all the way back to 0.10.0 |
4 participants
No description provided.