Skip to content

ci: Fix build in databend-docker #3394

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

Merged
merged 3 commits into from
Dec 12, 2021
Merged

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Dec 12, 2021

Signed-off-by: Xuanwo [email protected]

I hereby agree to the terms of the CLA available at: https://databend.rs/policies/cla/

Summary

This PR is an extension of #3391, fix an extra place of cross.

Changelog

  • Build/Testing/CI

@Xuanwo Xuanwo requested a review from BohuTANG as a code owner December 12, 2021 05:16
@databend-bot databend-bot added the pr-build this PR changes build/testing/ci steps label Dec 12, 2021
@databend-bot
Copy link
Member

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

1 similar comment
@databend-bot
Copy link
Member

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@vercel
Copy link

vercel bot commented Dec 12, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/databend/databend/HtrqJ9LvrsczjbK1pGaY6z4ABfVr
✅ Preview: Canceled

[Deployment for 9a0d554 canceled]

@codecov-commenter
Copy link

codecov-commenter commented Dec 12, 2021

Codecov Report

Merging #3394 (9a0d554) into main (cd650c9) will decrease coverage by 0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #3394   +/-   ##
=====================================
- Coverage     61%     61%   -1%     
=====================================
  Files        608     608           
  Lines      34052   34050    -2     
=====================================
- Hits       20917   20910    -7     
- Misses     13135   13140    +5     
Impacted Files Coverage Δ
metasrv/src/errors.rs 23% <0%> (-8%) ⬇️
common/management/src/cluster/cluster_mgr.rs 79% <0%> (-2%) ⬇️
metasrv/src/meta_service/network.rs 76% <0%> (-2%) ⬇️
metasrv/src/meta_service/meta_service_impl.rs 76% <0%> (-1%) ⬇️
metasrv/src/meta_service/message.rs 72% <0%> (ø)
common/meta/types/src/cmd.rs 93% <0%> (+6%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cd650c9...9a0d554. Read the comment docs.

@databend-bot
Copy link
Member

Wait for another reviewer approval

@BohuTANG BohuTANG merged commit ef57d8b into databendlabs:main Dec 12, 2021
@BohuTANG
Copy link
Member

Build perf tool failed, seems related to OpenSSL :

https://github.com/datafuselabs/databend/runs/4496828786?check_suite_focus=true

run pkg_config fail: "pkg-config has not been configured to support cross-compilation.\n\nInstall a sysroot for the target platform and configure it via\nPKG_CONFIG_SYSROOT_DIR and PKG_CONFIG_PATH, or install a\ncross-compiling wrapper for pkg-config and set it via\nPKG_CONFIG environment variable."

  --- stderr
  thread 'main' panicked at '

  Could not find directory of OpenSSL installation, and this `-sys` crate cannot
  proceed without this knowledge. If OpenSSL is installed and this crate had
  trouble finding it,  you can set the `OPENSSL_DIR` environment variable for the
  compilation process.

  Make sure you also have the development packages of openssl installed.
  For example, `libssl-dev` on Ubuntu or `openssl-devel` on Fedora.

  If you're in a situation where you think the directory *should* be found
  automatically, please open a bug at https://github.com/sfackler/rust-openssl
  and include information about your system as well as this message.

  $HOST = x86_64-unknown-linux-gnu
  $TARGET = aarch64-unknown-linux-gnu
  openssl-sys = 0.9.71

  ', /cargo/registry/src/mygithub.libinneed.workers.dev-1ecc6299db9ec823/openssl-sys-0.9.71/build/find_normal.rs:180:5
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: build failed
Error: Process completed with exit code 101.


@Xuanwo
Copy link
Member Author

Xuanwo commented Dec 12, 2021

Wired, I will figure it out tomorrow.

@Xuanwo Xuanwo deleted the fix-build branch December 13, 2021 02:25
@Xuanwo
Copy link
Member Author

Xuanwo commented Dec 13, 2021

The problem is gone on the main branch, I believe there is something wrong with the ~/.cargo/bin cache that failed workflows use the wrong cross binary.

Now all the build is OK and I'm working #3344 so that we can avoid those problems again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-build this PR changes build/testing/ci steps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants