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

Use CircleCI template with JDK 11 #619

Merged
merged 3 commits into from
Oct 3, 2022
Merged

Use CircleCI template with JDK 11 #619

merged 3 commits into from
Oct 3, 2022

Conversation

schlosna
Copy link
Contributor

@schlosna schlosna commented Sep 30, 2022

Before this PR

The circleci setup was using a custom JDK 8 container image with openssl added to enable testing with Hadoop native library crypto. Unfortunately, the custom circle config is causing a bunch of stuck excavators due to Gradle infrastructure requiring JDK 11.

See:

After this PR

==COMMIT_MSG==
Use CircleCI template with JDK 11
==COMMIT_MSG==

Possible downsides?

Does not setup & test IBM JDK yet

@changelog-app
Copy link

changelog-app bot commented Sep 30, 2022

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Use CircleCI template

Check the box to generate changelog(s)

  • Generate changelog entry

@schlosna schlosna changed the title Use CircleCI template Use CircleCI template with JDK 11 Sep 30, 2022
Comment on lines -43 to -53
build-ibm:
docker: [{ image: 'ellisjoe/ibm-java-openssl:java-8' }]
resource_class: xlarge
trial-publish:
docker: [{ image: 'cimg/openjdk:11.0.10-node' }]
resource_class: medium
environment:
GRADLE_OPTS: -Dorg.gradle.console=plain -Dorg.gradle.internal.launcher.welcomeMessageEnabled=false

CIRCLE_TEST_REPORTS: /home/circleci/junit
CIRCLE_ARTIFACTS: /home/circleci/artifacts
GRADLE_OPTS: -Dorg.gradle.workers.max=1 -Dorg.gradle.jvmargs='-Xmx2g --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED'
_JAVA_OPTIONS: -XX:ActiveProcessorCount=2 -XX:MaxRAM=4g -XX:ErrorFile=/home/circleci/artifacts/hs_err_pid%p.log -XX:HeapDumpPath=/home/circleci/artifacts
steps:
- checkout
- restore_cache: { key: 'gradle-wrapper-{{ checksum "gradle/wrapper/gradle-wrapper.properties" }}' }
- restore_cache: { key: 'gradle-cache-{{ checksum "versions.props" }}-{{ checksum "build.gradle" }}' }
- run: ./gradlew --no-daemon --parallel --continue crypto-core:test
Copy link
Contributor Author

@schlosna schlosna Sep 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that we previously tested against IBM JDK 8 using the container image from https://github.com/ellisjoe/ibm-java-openssl/blob/develop/Dockerfile, but that has not been migrated over. We could potentially sudo apt install adoptopenjdk-11-openj9 and test against that, but would prefer to defer that for another day.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine no longer testing against the IBM JVM. I'm not aware of any usage and the overhead of maintaining that additional test has been quite high.

@schlosna schlosna marked this pull request as ready for review September 30, 2022 19:11
@schlosna
Copy link
Contributor Author

once this is approved, we'll need to adjust the required checks for repo to ci/circleci: check and ci/circleci: trial-publish and remove the other old tasks

Copy link
Contributor

@carterkozak carterkozak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thank you :-)

@schlosna
Copy link
Contributor Author

schlosna commented Oct 3, 2022

🚀

@bulldozer-bot bulldozer-bot bot merged commit 00aebea into develop Oct 3, 2022
@bulldozer-bot bulldozer-bot bot deleted the ds/circle-setup branch October 3, 2022 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants