Skip to content

Update github actions versions#110

Merged
dflook merged 1 commit into
mainfrom
checkout
Oct 3, 2024
Merged

Update github actions versions#110
dflook merged 1 commit into
mainfrom
checkout

Conversation

@dflook

@dflook dflook commented Oct 1, 2024

Copy link
Copy Markdown
Owner

GitHub decided to use nodejs for their github actions, such as checkout/upload-artifact/download-artifact etc. This is problematic when running actions jobs inside containers, as node needs to get in there somehow. The actions runner rams a nodejs runtime into your containers, whether it works there or not.

GitHub are threatening to use a newer nodejs runtime, which certainly does NOT work in some of the older python containers.

To address this there is a new composite action called 'run-in-container', which tries to approximate the same behaviour as running jobs inside containers, but for a single step. This means that the GitHub provided actions in the job can now run outside the container.

@dflook dflook force-pushed the checkout branch 2 times, most recently from c9a8e5e to 80c0a89 Compare October 2, 2024 16:26
GitHub decided to use nodejs for their github actions, such as checkout/upload-artifact/download-artifact etc.
This is problematic when running actions jobs inside containers, as node needs to get in there somehow.
The actions runner rams a nodejs runtime into your containers, whether it works there or not.

GitHub are threatening to use a newer nodejs runtime, which certainly does NOT work in some of the older python containers.

To address this there is a new composite action called 'run-in-container', which tries to approximate the same behaviour
as running jobs inside containers, but for a single step. This means that the GitHub provided actions in the job can now
run outside the test container.
@dflook dflook merged commit 66b0858 into main Oct 3, 2024
@dflook dflook deleted the checkout branch October 3, 2024 07:57
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.

1 participant