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

Update JDK to OpenJdk8 #43

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update JDK to OpenJdk8 #43

wants to merge 1 commit into from

Conversation

robbavey
Copy link
Contributor

@robbavey robbavey commented May 1, 2019

No description provided.

@robbavey
Copy link
Contributor Author

robbavey commented May 1, 2019

Note to reviewer: This is the result of the script that will be pushed to all plugins to ensure that we use the openjdk8 for plugin builds to ensure that builds will still be able to run after Travis migrates to Ubuntu xenial for their default build image.

This plugin was chosen as the example PR, as the build image for this plugin has already migrated to Xenial - see System Info in the job log for the CI build:

Build system information
Build language: ruby
Build group: stable
Build dist: xenial
Build id: 526905185
Job id: 526905186
Runtime kernel version: 4.15.0-1028-gcp
travis-build version: 800f14f40
Build image provisioning date and time
Mon Mar 25 16:43:24 UTC 2019
Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.6 LTS
Release:	16.04
Codename:	xenial

@elasticsearch-bot elasticsearch-bot self-assigned this May 1, 2019
jsvd
jsvd approved these changes May 6, 2019
View reviewed changes
Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsvd jsvd jsvd 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
@robbavey @jsvd @elasticsearch-bot