-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Cleanup circleci script #8618
Cleanup circleci script #8618
Conversation
✅ Deploy Preview for meta-velox canceled.
|
@gaoyangxiaozhu do you know why abfs library install would end up with temporary paths? |
@gaoyangxiaozhu, the background is that we installed the dependencies in the CI image. But looks like the azure targets are being linked with temporary build paths. It likely worked before since we were installing the dependencies during the test run time and these paths are still present. |
Filed #8622 for the Azure issue. Let's fix that in a separate PR. |
@kgpai has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary: Pull Request resolved: facebookincubator#8618 Reviewed By: amitkdutta Differential Revision: D53286205 Pulled By: kgpai fbshipit-source-id: 120a25c0903d9aaa0ffdf0e127307f09bc9caefb
We now install dependencies in the circle ci image.