Skip to content

Onboard jenkins prod docker images to github actions#76

Merged
peterzhuamazon merged 5 commits into
opensearch-project:mainfrom
peterzhuamazon:issues-75-1
Nov 1, 2023
Merged

Onboard jenkins prod docker images to github actions#76
peterzhuamazon merged 5 commits into
opensearch-project:mainfrom
peterzhuamazon:issues-75-1

Conversation

@peterzhuamazon
Copy link
Copy Markdown
Member

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.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Comment thread .github/workflows/check.yml Outdated

steps:
- uses: actions/checkout@v3
- name: Set up JDK 11
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member Author

@peterzhuamazon peterzhuamazon Nov 1, 2023

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.

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member Author

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=

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon merged commit c85334e into opensearch-project:main Nov 1, 2023
@peterzhuamazon peterzhuamazon deleted the issues-75-1 branch November 1, 2023 20:19
opensearch-trigger-bot Bot pushed a commit that referenced this pull request Nov 1, 2023
* 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>
reta added a commit that referenced this pull request Nov 1, 2023
)

* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.x enhancement New feature or request

Projects

Development

Successfully merging this pull request may close these issues.

2 participants