[PR #1954] [MERGED] feat: cli option to enable the new action cache #2207

New issue

Closed

opened 2026-03-01 21:54:27 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 21:54:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/1954
Author: @ChristopherHX
Created: 8/8/2023
Status: ✅ Merged
Merged: 1/19/2024
Merged by: @mergify[bot]

Base: master ← Head: enable-new-action-cache

📝 Commits (10+)

d620ca7 Enable the new action cache
b8a5410 fix
64df8ae fix: CopyTarStream (Docker)
be5d3a4 suppress panic in test
b497266 add a cli option for opt in
7e7f4b2 fixups
13ffce7 add package
5c1c3b7 fix
509df53 rc.Config nil in test???
05c91d6 add feature flag

📊 Changes

10 files changed (+211 additions, -16 deletions)

View changed files

📝 cmd/input.go (+1 -0)
📝 cmd/root.go (+6 -0)
📝 pkg/container/docker_run.go (+19 -1)
📝 pkg/model/planner.go (+40 -6)
📝 pkg/runner/action.go (+21 -1)
📝 pkg/runner/reusable_workflow.go (+40 -0)
📝 pkg/runner/runner.go (+1 -0)
📝 pkg/runner/step.go (+13 -0)
📝 pkg/runner/step_action_local.go (+27 -8)
📝 pkg/runner/step_action_remote.go (+43 -0)

📄 Description

This is my initial code to switch the action cache over to my new implementation

Until all tests are migrated this will be kept as draft to avoid mergify spam.

I would be careful to test this change, to detect known issues of my previous PR's and this change.

Checklist

Test symlinked action.yml file in remote actions
Test symlinked action.yml file in local actions
Test remote docker actions
Test using images with a non root container (I expect we still need to fix permissions)
Test error messages like
- action.yml not found
- Clone failed
Make Tests Green (If I add a Feature Toggle for this change the test don't need to change)

Do you prefer to add a Feature Toggle for this change? Then we can merge this while iron out remaining bugs later and make it possible to make use of this change for people affected by the current problems.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/nektos/act/pull/1954 **Author:** [@ChristopherHX](https://github.com/ChristopherHX) **Created:** 8/8/2023 **Status:** ✅ Merged **Merged:** 1/19/2024 **Merged by:** [@mergify[bot]](https://github.com/apps/mergify) **Base:** `master` ← **Head:** `enable-new-action-cache` --- ### 📝 Commits (10+) - [`d620ca7`](https://github.com/nektos/act/commit/d620ca7d08f607a6a2c6a640669b784a6f288e0e) Enable the new action cache - [`b8a5410`](https://github.com/nektos/act/commit/b8a5410e67453f91a384cc5c3f7156f1307daf69) fix - [`64df8ae`](https://github.com/nektos/act/commit/64df8aeda43d922efc0e78c67c65baeeb146c980) fix: CopyTarStream (Docker) - [`be5d3a4`](https://github.com/nektos/act/commit/be5d3a456224777621b3fb8efdbecc5f9bcd9b9c) suppress panic in test - [`b497266`](https://github.com/nektos/act/commit/b497266bfa6ee2def98767170f937ec7d30dabd0) add a cli option for opt in - [`7e7f4b2`](https://github.com/nektos/act/commit/7e7f4b24846f1152f5e334c17965b86a42f2b3c6) fixups - [`13ffce7`](https://github.com/nektos/act/commit/13ffce71113db8cec8a51b8d0bf426ee06c6f3d4) add package - [`5c1c3b7`](https://github.com/nektos/act/commit/5c1c3b7d8fc2866bf3acba438a61e7721e0f9f32) fix - [`509df53`](https://github.com/nektos/act/commit/509df5302d045370b182a3cc4edb49516966529d) rc.Config nil in test??? - [`05c91d6`](https://github.com/nektos/act/commit/05c91d652a47d1c336dca779c3a744e377582378) add feature flag ### 📊 Changes **10 files changed** (+211 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `cmd/input.go` (+1 -0) 📝 `cmd/root.go` (+6 -0) 📝 `pkg/container/docker_run.go` (+19 -1) 📝 `pkg/model/planner.go` (+40 -6) 📝 `pkg/runner/action.go` (+21 -1) 📝 `pkg/runner/reusable_workflow.go` (+40 -0) 📝 `pkg/runner/runner.go` (+1 -0) 📝 `pkg/runner/step.go` (+13 -0) 📝 `pkg/runner/step_action_local.go` (+27 -8) 📝 `pkg/runner/step_action_remote.go` (+43 -0) </details> ### 📄 Description This is my initial code to switch the action cache over to my new implementation Until all tests are migrated this will be kept as draft to avoid mergify spam. I would be careful to test this change, to detect known issues of my previous PR's and this change. Checklist - [ ] Test symlinked action.yml file in remote actions - [ ] Test symlinked action.yml file in local actions - [ ] Test remote docker actions - [ ] Test using images with a non root container (I expect we still need to fix permissions) - [ ] Test error messages like - [ ] action.yml not found - [ ] Clone failed - [ ] Make Tests Green (If I add a Feature Toggle for this change the test don't need to change) Do you prefer to add a Feature Toggle for this change? Then we can merge this while iron out remaining bugs later and make it possible to make use of this change for people affected by the current problems. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-01 21:54:27 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/act#2207

No description provided.

Rows
Columns