dkms icon indicating copy to clipboard operation
dkms copied to clipboard

build(deps): bump actions/checkout from 3 to 4

Open dependabot[bot] opened this issue 2 years ago • 6 comments

Bumps actions/checkout from 3 to 4.

Release notes

Sourced from actions/checkout's releases.

v4.0.0

What's Changed

New Contributors

Full Changelog: https://github.com/actions/checkout/compare/v3...v4.0.0

v3.6.0

What's Changed

New Contributors

Full Changelog: https://github.com/actions/checkout/compare/v3.5.3...v3.6.0

v3.5.3

What's Changed

New Contributors

Full Changelog: https://github.com/actions/checkout/compare/v3...v3.5.3

v3.5.2

What's Changed

Full Changelog: https://github.com/actions/checkout/compare/v3.5.1...v3.5.2

v3.5.1

What's Changed

New Contributors

... (truncated)

Changelog

Sourced from actions/checkout's changelog.

Changelog

v4.1.0

v4.0.0

v3.6.0

v3.5.3

v3.5.2

v3.5.1

v3.5.0

v3.4.0

v3.3.0

v3.2.0

v3.1.0

v3.0.2

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

dependabot[bot] avatar Oct 11 '23 11:10 dependabot[bot]

Looking through the failures are reasons behind them:

The v4 action switches from node.js 16 (EOL a month ago) to 20, which n newer glibc/c++ that is missing on CentOS 7 and Ubuntu 18.04.

The latter has gone EOL a few months ago - Ubuntu are providing extra (read $$$$$) HWE support. The former is still supported until mid 2024, although with the overall CentOS future looking fuzzy I'm inclined to drop it earlier than later. Unless we have someone to semi-actively maintain it of course.

So I'm inclined to simply drop the two - CentOS 7 and Ubuntu 18.04 - and merge this.

@scaronni any preferences, suggestions?

@fishilico you've added the initial CI (huge thanks for which). Did you add CentOS with a particular use-case in mind? If so would you be interested in maintaining it or any other version?

evelikov avatar Oct 11 '23 11:10 evelikov

Looking through the failures are reasons behind them:

The v4 action switches from node.js 16 (EOL a month ago) to 20, which n newer glibc/c++ that is missing on CentOS 7 and Ubuntu 18.04.

@scaronni any preferences, suggestions?

I'm still packaging DKMS for CentOS 7 in EPEL until the EOL date (2024.06), but I think you can remove it anyway from the tests.

scaronni avatar Oct 11 '23 20:10 scaronni

Ouch - are you still packaging the latest or you're using a fixed older version? AFAICT there is nothing security-wise in the diff log in the action, so there's no need to actually update it.

Edit: Considering the EOL numbers - Stream 8 is EOL in April 2024, while 7 is July 2024 - I'm inclined that we can leave this PR open until then ;-)

evelikov avatar Oct 11 '23 22:10 evelikov

Ouch - are you still packaging the latest or you're using a fixed older version? AFAICT there is nothing security-wise in the diff log in the action, so there's no need to actually update it.

I'm updating to the latest on all supported EPEL/Fedora releases. The DKMS package is then picked up by Nvidia and AMD for bundling in their RPM based installers (graphic card drivers from AMD, DGX packages from Nvidia, etc.).

scaronni avatar Oct 12 '23 05:10 scaronni

Plus we use them at work :)

scaronni avatar Oct 12 '23 05:10 scaronni

Thanks - flipping to draft.

This can sit until some point next year (edit: or later) IMHO. I really don't want to make your life harder than needed. Especially since there is no obvious benefit of using the newer actions.

evelikov avatar Oct 12 '23 09:10 evelikov