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.

Sign up for GitHub

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

Jump to bottom

[Backport 2.x] Added jenkinsfile to run gradle check in OpenSearch #2628

Merged
merged 1 commit into from
Mar 30, 2022
Merged

[Backport 2.x] Added jenkinsfile to run gradle check in OpenSearch #2628

merged 1 commit into from
Mar 30, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport f1d35d0 from #2166

@owaiskazi19 @github-actions
Added jenkinsfile to run gradle check in OpenSearch (#2166)
a9a31e4 
* Added jenkinsfile for gradle check

Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>

* Added jenkinsfile to run gradle check

Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>

* PR comment

Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>
(cherry picked from commit f1d35d0)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner March 28, 2022 18:43
@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure a9a31e4
Log 3818

Reports 3818

@owaiskazi19
Copy link
Member

#1703

@owaiskazi19
Copy link
Member

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success a9a31e4
Log 3824

Reports 3824

dblock
dblock reviewed Mar 29, 2022
View reviewed changes
jenkins/jenkinsfile
JAVA14_HOME="/opt/java/openjdk-14"
JAVA17_HOME="/opt/java/openjdk-17"
JAVA8_HOME="/opt/java/openjdk-8"
JAVA_HOME="/opt/java/openjdk-14"
Copy link
Member

@dblock dblock Mar 29, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

For 2.0 this needs to be switched to JDK 11 or 17, no?

Copy link
Member

Choose a reason for hiding this comment

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

Created #2656 for the same.

Copy link
Member

Choose a reason for hiding this comment

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

We can merge this if it looks fine and then I can backport #2656 to 2.0 and 2.x.

@dblock dblock merged commit 18c14e2 into 2.x Mar 30, 2022
@github-actions github-actions bot deleted the backport/backport-2166-to-2.x branch March 30, 2022 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@owaiskazi19 owaiskazi19 owaiskazi19 left review comments

@dblock dblock dblock approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@opensearch-ci-bot @owaiskazi19 @dblock