pantsbuild / pants

The Pants Build System
https://www.pantsbuild.org
Apache License 2.0
3.34k stars 638 forks source link

actions/upload-artifacts@v3 is being removed (5 Dec 2024), affects Linux build-wheels jobs #21616

Open huonw opened 2 weeks ago

huonw commented 2 weeks ago

Describe the bug

According to https://github.blog/changelog/2024-11-05-notice-of-breaking-changes-for-github-actions/#artifacts-v3-brownouts, the actions/upload-artifacts@v3 behaviour is being deprecated and removed. Timeline:

This affects the Linux build-wheels jobs which run in a container, and require node16 compatibility (only available in v3, not v4, of this action).

https://github.com/pantsbuild/pants/blob/9244fcc8530ac93fcc7a8e0d1fdcbabe61090f36/src/python/pants_release/generate_github_workflows.py#L902

We'll need to remove the usage of this from all active branches, or CI will fail.

Options:

  1. quick-hack: stop uploading the logs at all
  2. better: use the container only for the build, either by running the whole pants invocation in it, or using a docker environment within pants
  3. ???

Pants version N/A

OS N/A

Additional info Related discussion happened in: https://github.com/pantsbuild/pants/pull/21133

tdyas commented 2 weeks ago

Can we just upgrade to v4 of actions/upload-artifact?

huonw commented 2 weeks ago

I think that requires node20 which isn't compatible with the container used (see #21133 and #21172), although feel free to dig into the git history and/or experiment more!