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

Cleanup circleci script #8618

Closed

Conversation

majetideepak
Copy link
Collaborator

@majetideepak majetideepak commented Jan 31, 2024

We now install dependencies in the circle ci image.

Copy link

netlify bot commented Jan 31, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 988d44f
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/65baa80b60402f0008342217

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 31, 2024
@majetideepak majetideepak marked this pull request as draft January 31, 2024 13:13
@majetideepak
Copy link
Collaborator Author

majetideepak commented Jan 31, 2024

ninja: error: '/build/azure-sdk-for-cpp/sdk/storage/azure-storage-common/_build/vcpkg_installed/x64-linux/lib/libxml2.a', needed by 'velox/common/memory/tests/velox_memory_test', missing and no known rule to make it

@gaoyangxiaozhu do you know why abfs library install would end up with temporary paths?

@majetideepak
Copy link
Collaborator Author

@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.

@majetideepak majetideepak marked this pull request as ready for review January 31, 2024 21:11
@majetideepak
Copy link
Collaborator Author

Filed #8622 for the Azure issue. Let's fix that in a separate PR.

@facebook-github-bot
Copy link
Contributor

@kgpai has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@kgpai merged this pull request in eb6b585.

Copy link

Conbench analyzed the 1 benchmark run on commit eb6b585b.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

@majetideepak majetideepak deleted the cleanup-circleci branch February 2, 2024 16:16
FelixYBW pushed a commit to FelixYBW/velox that referenced this pull request Feb 12, 2024
Summary: Pull Request resolved: facebookincubator#8618

Reviewed By: amitkdutta

Differential Revision: D53286205

Pulled By: kgpai

fbshipit-source-id: 120a25c0903d9aaa0ffdf0e127307f09bc9caefb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants