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

ci: update actions/checkout in GitHub Actions workflows to v4 #739

Merged
merged 1 commit into from
Dec 24, 2023

Conversation

striezel
Copy link
Contributor

Description

This updates actions/checkout to v4, it's current version.

Changelog:

v4.1.0

  • Add support for partial checkout filters

v4.0.0

  • Support fetching without the --progress option
  • Update to node20

Tasklist

  • Ensure all CI builds pass
  • Review and approve

@striezel
Copy link
Contributor Author

This seems to fail on all the jobs using the ubuntu:18.04 Docker container image. :(

@striezel striezel marked this pull request as draft December 23, 2023 10:40
@mloskot
Copy link
Member

mloskot commented Dec 23, 2023

Yeah, this looks like a known issue for Ubuntu 18.04 which is incompatible with Node required by the checkout v4 action:

I can't see what can be done about it, other than postpone v3 to v4 pgrade or remove the jobs based on Ubuntu 18.04 (losing GCC 6, Clang 3.9, 4.0 and 5.0).

@striezel
Copy link
Contributor Author

I can't see what can be done about it, other than postpone v3 to v4 pgrade or remove the jobs based on Ubuntu 18.04 (losing GCC 6, Clang 3.9, 4.0 and 5.0).

I am not sure that losing those CI jobs is acceptable. So I decided not to update the workflows that contain containerized builds to to actions/checkout@v4 and just update the others instead. It may not be the ideal solution, but it's better than nothing, and the jobs should pass now - again.

@striezel striezel marked this pull request as ready for review December 24, 2023 18:34
@mloskot
Copy link
Member

mloskot commented Dec 24, 2023

Thank you @striezel

Another workaround could be to split this posix job

posix:
strategy:
fail-fast: false
matrix:
include:
- toolset: gcc-6
cxxstd: "14,1z"
os: ubuntu-latest
container: ubuntu:18.04

into posix and posix-legacy, and move 18.04-based builds to the later, then the posix could be upgraded to checkout@v4.

This would mean copying some YAML boilerplate, but I personally don't die hard for DRY in YAML-s :)

@striezel
Copy link
Contributor Author

That split would make the YAML file even longer and more complicated than it is now. I don't feel that actions/checkout@v3 vs. v4 is such a big improvement that it is worth the effort to make the CI configuration more complicated and maintain two copies of basically the same CI steps.

@mloskot
Copy link
Member

mloskot commented Dec 24, 2023

Sure. By the way, I kind of steal that from Peter Dimov's https://github.com/boostorg/mp11/blob/develop/.github/workflows/ci.yml 😊

Copy link
Member

@mloskot mloskot left a comment

Choose a reason for hiding this comment

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

Thanks @striezel

@mloskot mloskot merged commit 3a4d454 into boostorg:develop Dec 24, 2023
17 of 18 checks passed
@striezel striezel deleted the actions-checkout-v4 branch December 25, 2023 22:06
@striezel striezel mentioned this pull request Jun 30, 2024
6 tasks
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

Successfully merging this pull request may close these issues.

2 participants