-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Allow to disable jarhell check for tests on 2.2 with test.jarhell.check
#16174
Conversation
…eck` This relates to elastic#16042 where we agreed on adding an opt-out on 2.2 for test to disable jarhell checks in the BootstrapForTesting.java - for other branches we will use different solutoins.
@clintongormley can you take a look? |
LGTM |
JarHell.checkJarHell(); | ||
} catch (Exception e) { | ||
throw new RuntimeException("found jar hell in test classpath", e); | ||
} |
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.
Should we log that jar hell has been deactivated?
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.
there is no logger here anyway... hot topic :)
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.
haha! I was pretty sure you were going to answer that :)
Allow to disable jarhell check for tests on 2.2 with `test.jarhell.check`
Otherwise it will not be picked up when running the tests from command line. relates to elastic#16174
Allow to disable jarhell check for tests on 2.2 with `test.jarhell.check`
This relates to #16042 where we agreed on adding an opt-out on 2.2 for test
to disable jarhell checks in the BootstrapForTesting.java - for other branches we
will use different solutions.