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

install-deps.sh: install libpmem libraries if WITH_PMEM is set #40617

Merged
merged 2 commits into from Apr 8, 2021

Conversation

tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Apr 6, 2021

Signed-off-by: Kefu Chai kchai@redhat.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

Signed-off-by: Kefu Chai <kchai@redhat.com>
so we can at least build-test the rwl cache plugin in "make check" run.

this is an opt-in option which is only enabled if WITH_PMEM is "true",
we will set it in the corresponding ceph-build job.

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov
Copy link
Contributor Author

tchaikov commented Apr 8, 2021

tested using

WITH_PMEM=true ./run-make-check.sh

on a focal machine.

@batrick thanks for your review. i just realized that i forgot to update run-make-check.sh. could you please take another look?

@trociny @CongMinYin @cyx1231st hi folks, as Jason pointed out we need to have the pmem cache support by default. but pmem v1.10 is not packaged by ubuntu focal. so i compiled and uploaded the latest libpmem packages and its dependencies to chacra, so we can at least build-test it in "make check".

Copy link
Contributor

@trociny trociny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Thanks!

@tchaikov tchaikov merged commit 8e7e442 into ceph:master Apr 8, 2021
@tchaikov tchaikov deleted the wip-system-pmem branch April 8, 2021 14:57
@djgalloway
Copy link

This is still somehow creating mangled pmem.list files on our Jenkins builders.

Do we need to install a special version built and hosted by us in shaman or is the distro-provided version okay?

@tchaikov
Copy link
Contributor Author

#46773

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants