Skip to content
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

OpenJ9 smoke test Docker images #127

Merged
merged 7 commits into from
Feb 27, 2021

Conversation

agoallikmaa
Copy link
Contributor

  • Adds OpenJ9 images for Windows images and JBoss images.
  • The logic to define images to create in matrix/build.gradle has been changed to be as close as possible to the logic in OpenTelemetry - only difference currently is that this one has logic to make Windows images, add pull tasks, and can be called for multiple sets of targets as it is wrapped in a function with targets as a parameter.
  • Added a more concise way to list test configurations in smoke tests now to remove duplication and to merge adding multiple JDKs and VMs of the same version into one line.
  • Updated junit-jupiter and mockito dependency versions to the latest which also makes them all depend on exactly the same jupiter version. This was done because for some reason after jcenter removal, without redownloading dependencies it tried to use a mix of 5.7.0 and 5.6.2 packages which caused an AbstractMethodError during build. Did not investigate the root cause further as making sure everything requires the same version was an easy fix.

@agoallikmaa agoallikmaa changed the title Openj9 smoke images OpenJ9 smoke test Docker images Feb 25, 2021
matrix/build.gradle Show resolved Hide resolved
@iNikem iNikem merged commit e2d715d into signalfx:main Feb 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants