-
Notifications
You must be signed in to change notification settings - Fork 18
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
Onboard jenkins prod docker images to github actions #76
Onboard jenkins prod docker images to github actions #76
Conversation
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
.github/workflows/check.yml
Outdated
|
||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: Set up JDK 11 |
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.
Do we need this step? Our CI image has all JDKs pre-installed
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.
Yes, but it is easier just to use this installation, than setting an env var here.
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.
Discussed offline: we should no alter our CI image since the goal is to use the same env
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.
Adding a matrix as well and will use -Druntime.java=
* Onboard jenkins prod docker images to github actions Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> --------- Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> (cherry picked from commit c85334e) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
) * Onboard jenkins prod docker images to github actions (#76) * Onboard jenkins prod docker images to github actions Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Add more Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> --------- Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> (cherry picked from commit c85334e) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * remove jdk21 Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Explicitly set JAVA_HOME to JDK-11 for Gradle build Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Update OpenSearch core version to 2.12.0-SNAPSHOT Signed-off-by: Andriy Redko <andriy.redko@aiven.io> * Removed explicit -Druntime.java version specifier for Gradle check Signed-off-by: Andriy Redko <andriy.redko@aiven.io> --------- Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Andriy Redko <andriy.redko@aiven.io> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Peter Zhu <zhujiaxi@amazon.com> Co-authored-by: Andriy Redko <andriy.redko@aiven.io>
Description
Onboard jenkins prod docker images to github actions
Issues Resolved
#75
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.