-
Notifications
You must be signed in to change notification settings - Fork 603
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
Remove temporary cache cleaning fix from OS smoke tests #8943
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
bors merge |
8943: Remove temporary cache cleaning fix from OS smoke tests r=npepinpe a=npepinpe ## Description This PR removes the temporary cache cleaning actions from the OS smoke tests. It doesn't seem to be doing much at this point, as we still regularly run into issues with it. Let's go with the "standard" cache action and start again when we see issues. Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
Build failed: |
Git push to origin failed for stable/1.2 with exitcode 1 |
Git push to origin failed for stable/1.3 with exitcode 1 |
That's a new one. I didn't even get the commands I need to paste :( @korthout |
Yeah this is a permissions topic. If we want it to be able to backport changes to workflow files, then we need to explicitly give it the permission to do so. We could consider that 🤔 What do you think? @npepinpe |
Description
This PR removes the temporary cache cleaning actions from the OS smoke tests. It doesn't seem to be doing much at this point, as we still regularly run into issues with it. Let's go with the "standard" cache action and start again when we see issues.
Definition of Done
Not all items need to be done depending on the issue and the pull request.
Code changes:
backport stable/1.3
) to the PR, in case that fails you need to create backports manually.Testing:
Documentation: