-
Notifications
You must be signed in to change notification settings - Fork 79
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
fix: Make Java, Python wheel artifacts have same dependencies #5850
Conversation
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.
Suggest backporting this to 0.35.x.
@@ -27,6 +27,10 @@ dependencies { | |||
runtimeOnly project(':hotspot-impl') | |||
} | |||
|
|||
if (!hasProperty('excludeClockImpl')) { |
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.
This was also missing, but the server impl explicitly assumes it is there in its jvm args.
8aae914
to
dc40ddb
Compare
Added a test that will fail without the fix, so that we get early warnings about this kind of problem in the future. This ends up downloading a few wheels when the test runs, not unlike how the py-client tests behave - we may want to see about pinning those in our base images, so that the tests run much faster. This also required a fix to stream.py, as the xmlrunner provides a stream for stdout/err that apparently includes an |
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.
The Python changes LGTM
Fixes #5848