Skip to content

cmake: s/HAVE_MSGHDR/WITH_SYSTEMD/#40658

Merged
tchaikov merged 2 commits intoceph:masterfrom
tchaikov:wip-systemd
Apr 12, 2021
Merged

cmake: s/HAVE_MSGHDR/WITH_SYSTEMD/#40658
tchaikov merged 2 commits intoceph:masterfrom
tchaikov:wip-systemd

Conversation

@tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Apr 8, 2021

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

tchaikov added 2 commits April 8, 2021 16:58
as FreeBSD also has msghdr but it does not have systemd, or
flags like MFD_ALLOW_SEALING, O_TMPFILE or F_SEAL_GROW. so
use WITH_SYSTEMD for enabling journald backend of logging system.

also move the option of "WITH_SYSTEMD" up so that the src/CMakeLists.txt
is able to see the variable of WITH_SYSTEMD defined by it.

Signed-off-by: Kefu Chai <[email protected]>
as FreeBSD does not has systemd at the time of writing, so it does not
make sense to build or install systemd support.

Signed-off-by: Kefu Chai <[email protected]>
Copy link
Contributor

@wjwithagen wjwithagen left a comment

Choose a reason for hiding this comment

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

Jenkins build is back to normal : ceph-master-compile #6416

@tchaikov
Copy link
Contributor Author

tchaikov commented Apr 8, 2021

22 - run-rbd-unit-tests-61.sh (Timeout)

@tchaikov
Copy link
Contributor Author

tchaikov commented Apr 8, 2021

jenkins test make check

@tchaikov tchaikov merged commit 7a158fc into ceph:master Apr 12, 2021
@tchaikov tchaikov deleted the wip-systemd branch April 12, 2021 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants