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

[internal] Remove test cache isolation #13337

Merged

Conversation

stuhood
Copy link
Member

@stuhood stuhood commented Oct 22, 2021

With #13336 fixed, we should be able to remove RuleRunner cache directory isolation.

Thanks a lot to @patricklaw for the upstream fix in coursier/coursier#2197, and @alexarchambault for his work getting out a new coursier release!

[ci skip-rust]
[ci skip-build-wheels]

@stuhood
Copy link
Member Author

stuhood commented Oct 25, 2021

One Two Three green runs here: will rebase it a few more times before bringing it out of drafts.

@stuhood stuhood force-pushed the stuhood/remove-test-cache-isolation branch 2 times, most recently from f43c76e to b70453d Compare October 25, 2021 23:19
[ci skip-build-wheels]

[ci skip-rust]
@stuhood stuhood force-pushed the stuhood/remove-test-cache-isolation branch from b70453d to 7afc8b5 Compare October 25, 2021 23:20
@stuhood stuhood marked this pull request as ready for review October 25, 2021 23:57
@stuhood
Copy link
Member Author

stuhood commented Oct 25, 2021

This is ready for review: it's run three times without any flakes of the form that we used to see before #13046.

@stuhood stuhood enabled auto-merge (squash) October 25, 2021 23:59
@stuhood stuhood disabled auto-merge October 26, 2021 00:00
@stuhood stuhood merged commit f498538 into pantsbuild:main Oct 26, 2021
@stuhood stuhood deleted the stuhood/remove-test-cache-isolation branch October 26, 2021 00:00
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