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

[#1817] Actions: update list of job runners environments for CI #1811

Merged
merged 4 commits into from
Jul 31, 2022

Conversation

dcshzj
Copy link
Member

@dcshzj dcshzj commented Jul 23, 2022

Fixes #1817.

Proposed commit message

The virtual environments for the GitHub Actions runners have been
updated, with the introduction of macos-12 and ubuntu-22.04, and the
deprecation of macos-10.15.

Let's update the list of environments used for running the CI jobs.

Other information

List of available environments are given here: https://github.com/actions/virtual-environments#available-environments

The virtual environments for the GitHub Actions runners have been
updated, with the introduction of macos-12 and ubuntu-22.04, and the
deprecation of macos-10.15.

Let's update the list of environments used for running the CI jobs.
@dcshzj dcshzj requested a review from a team July 23, 2022 15:33
@@ -15,7 +15,7 @@ jobs:
name: ${{ matrix.os }} JDK 8
strategy:
matrix:
os: [ubuntu-18.04, ubuntu-20.04]
os: [ubuntu-18.04, ubuntu-20.04, ubuntu-22.04]
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure if we should adopt ubuntu-22.04 since it's still in beta according to the actions/virtual-environments page.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yup, it is in beta currently. I added this such that we can run on this version, but I will not be marking the run as a required check (i.e. it is allowed to fail but will not block the PR from being merged).

@@ -173,7 +173,7 @@ jobs:
name: ${{ matrix.os }} JDK 8
strategy:
matrix:
os: [windows-2022, windows-2019]
os: [windows-2016, windows-2019, windows-2022]
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure if we should add another Windows os for CI since this may impose additional load. Particularly because Windows CI is slower than that of Ubuntu.

Copy link
Member Author

Choose a reason for hiding this comment

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

The jobs will run in parallel, so I think it does not cause the CI builds to take longer. Nonetheless, I removed it because I realized it was supposed to be deprecated by now.

@dcshzj dcshzj changed the title Actions: update list of job runners environments for CI [#1817] Actions: update list of job runners environments for CI Jul 29, 2022
Copy link
Contributor

@yhtMinceraft1010X yhtMinceraft1010X left a comment

Choose a reason for hiding this comment

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

LGTM

@dcshzj dcshzj merged commit a528076 into reposense:master Jul 31, 2022
@github-actions
Copy link
Contributor

The following links are for previewing this pull request:

@dcshzj dcshzj deleted the update-actions-runners branch August 9, 2022 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Continuous Integration for macos-10.15 is always cancelled
2 participants