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

change our CI (org-wide) to use AdoptOpenJDK #587

Closed
SethTisue opened this issue Nov 27, 2018 · 7 comments
Closed

change our CI (org-wide) to use AdoptOpenJDK #587

SethTisue opened this issue Nov 27, 2018 · 7 comments
Assignees

Comments

@SethTisue
Copy link
Member

@adriaanm I wrote down "create an Adopt OpenJDK ticket, assign it to myself, say I plan to do it in January" but now I can't remember what this meant exactly... was it to add AdoptOpenJDK to our scala/scala CI matrix, or the community build, or what? if you can't remember either I can review the team meeting video and find out

@adriaanm
Copy link
Contributor

adriaanm commented Nov 27, 2018

it was originally about standardizing on adoptopenjdk hotspot on travis ci (installed using jabba), but in the mean time we're no longer sure because they have not yet been able to license the TCK

@SethTisue SethTisue changed the title (this is the AdoptOpenJDK ticket) change our CI (org-wide) to use AdoptOpenJDK? Nov 27, 2018
@SethTisue SethTisue self-assigned this Nov 27, 2018
@SethTisue
Copy link
Member Author

okay, we'll do nothing for now, and re-sync with others at Lightbend on this in January or whenever. it's mainly downstream projects (Akka, Play, etc) where it might really matter actually, so we can wait and follow their lead.

@SethTisue
Copy link
Member Author

note to self, since I already forgot once: the issue is that Oracle OpenJDK, which we're currently using on our own machines and on Jenkins and Travis, will eventually stop receiving security updates. that's the reason we will ultimately want to be using AdoptOpenJDK (or something)

@dwijnand
Copy link
Member

dwijnand commented Mar 9, 2019

Setting up Travis CI to use AdoptOpenJDK is quite simple:

before_install: curl -Ls https://git.io/jabba | bash && . ~/.jabba/jabba.sh
install: jabba install adopt@1.8.202-08 && jabba use "$_" && java -version

An example of a matrix using v8 and v11, with v11 being allowed to fail:

before_install: curl -Ls https://git.io/jabba | bash && . ~/.jabba/jabba.sh
install: jabba install "$TRAVIS_JDK" && jabba use "$_" && java -version

env:
  - TRAVIS_JDK=adopt@1.8.202-08
  - TRAVIS_JDK=adopt@1.11.0-2

matrix:
  fast_finish: true
  allow_failures:
    - env: TRAVIS_JDK=adopt@1.11.0-2 # not fully supported but allows problem discovery

And don't forget to cache the installed jdks:

cache:
  directories:
    - $HOME/.jabba/jdk

@SethTisue
Copy link
Member Author

@eed3si9n is working on this

@SethTisue SethTisue assigned eed3si9n and unassigned SethTisue Jul 8, 2019
@SethTisue SethTisue changed the title change our CI (org-wide) to use AdoptOpenJDK? change our CI (org-wide) to use AdoptOpenJDK Jul 8, 2019
eed3si9n added a commit to eed3si9n/scala-parser-combinators that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-parser-combinators that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-parser-combinators that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-parser-combinators that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-collection-compat that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-java8-compat that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-parallel-collections that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/slick that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala that referenced this issue Jul 9, 2019
eed3si9n added a commit to eed3si9n/scala-xml that referenced this issue Jul 10, 2019
eed3si9n added a commit to eed3si9n/scala-xml that referenced this issue Jul 10, 2019
eed3si9n added a commit to eed3si9n/scala that referenced this issue Jul 10, 2019
Ref scala/scala-dev#587

This is a continuation of scala#7931 to check AdoptOpenJDK JDK 11.
eed3si9n added a commit to eed3si9n/scala-collection-contrib that referenced this issue Jul 24, 2019
eed3si9n added a commit to eed3si9n/slick that referenced this issue Aug 7, 2019
@SethTisue
Copy link
Member Author

SethTisue commented Feb 4, 2020

I think this is done, or done enough to close the ticket anyway, any missing repo would be a very obscure one, I think. all the modules repos are using https://github.com/scala/scala-dev/blob/scala-dev/travis/default.yml. the community build is on AdoptOpenJDK (for 11+ anyway, not sure about 8, but whatevs)

nafg added a commit to slick/slick that referenced this issue Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants