Skip to content

Conversation

@mhashizume
Copy link
Contributor

This PR does the following:

GitHub is deprecating Actions running Node12 this year:
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

The Checkout Action uses Node12 in v2 and Node16 in v3. This commit
updates all instances of the Checkout Action to use v3.
This commit updates all of the Ruby versions from end-of-life
versions to Ruby 2.7.
@mhashizume mhashizume requested a review from a team as a code owner January 4, 2023 00:23
@mhashizume mhashizume force-pushed the maint/main/checkout_action branch from 7714ef7 to e6e9d4b Compare January 4, 2023 00:48
The install_puppet_agent method in the Presuite Action blindly
passes Bundler output as a file path to Beaker. However, the
Bundler command can sometimes include other output (e.g. deprecation
warnings) and cause Beaker to fail.

This commit adds simple error handling to the install_puppet_agent
method to deal with this condition.
On running the acceptance workflow, we get a warning about a bug
in the currently running version of Rubygems (3.0.3.1) that affects
other code in the workflow.

This commit sets the Rubygems version in the acceptance workflow to
3.3.26, the same version of Rubygems that we use in Puppet GitHub
Actions.
Prior to this commit, the monkeypatching for mocks was overwriting
the win32/registry class but ommited the API module. This commit
adds that module back.
@mhashizume mhashizume force-pushed the maint/main/checkout_action branch from 115bb2c to 0532109 Compare January 5, 2023 19:44
@mhashizume
Copy link
Contributor Author

Added a few more commits to address a failing acceptance test that was blocking checks from passing and to fix the discrepancy between the macOS GitHub Actions runner (latest is shifting from macOS 11 to macOS 12).

@joshcooper joshcooper merged commit d3447d7 into puppetlabs:main Jan 5, 2023
@mhashizume mhashizume deleted the maint/main/checkout_action branch January 9, 2023 21:33
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