Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: add workaround for nested composite actions issue #11490

Merged
merged 2 commits into from
Nov 30, 2022
Merged

Conversation

redsun82
Copy link
Contributor

Because of actions/runner#2009 the deeply nested action cache was failing to save the cache in the post run phase.

For the moment we just avoid the nesting with a copy-pasted action snippet.

@redsun82 redsun82 force-pushed the redsun82/cache branch 2 times, most recently from 8563859 to 42ac89e Compare November 30, 2022 08:55
@redsun82
Copy link
Contributor Author

Here one can see a successful cache save on a test PR that exercises the push workflow: #11491

redsun82 and others added 2 commits November 30, 2022 10:47
Because of actions/runner#2009 the deeply
nested action cache was failing to save the cache in the post run phase.

For the moment we just avoid the nesting with a copy-pasted action
snippet.
Co-authored-by: Erik Krogh Kristensen <[email protected]>
@redsun82 redsun82 marked this pull request as ready for review November 30, 2022 09:47
@redsun82 redsun82 requested a review from a team as a code owner November 30, 2022 09:47
@redsun82 redsun82 requested a review from erik-krogh November 30, 2022 09:48
@redsun82 redsun82 merged commit 2f4cf59 into main Nov 30, 2022
@redsun82 redsun82 deleted the redsun82/cache branch November 30, 2022 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants