]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
test: run promtool test without docker on ubuntu/focal
authorKefu Chai <kchai@redhat.com>
Fri, 19 Mar 2021 02:32:16 +0000 (10:32 +0800)
committerAashish Sharma <aashishsharma@localhost.localdomain>
Fri, 23 Apr 2021 11:44:06 +0000 (17:14 +0530)
commit3ad72a522b947809646c0d9f771e291a80acfe2b
treecaa288c85c8311493f33ff7f71b4d99a81e5d71b
parent3e26463094137153e05193038815adad7975f123
test: run promtool test without docker on ubuntu/focal

before this change, we use docker for running promtools offered by
a docker image, but this is not efficient, and quite a few developers
do not want to use docker for running "make check". this change was
introduced by #39246, the reason was that, in Ceph's CI process, we
are using Ubuntu/Bionic for running "make check" jobs, but prometheus
packaged by Bionic does not offer the "test rules" command. so, to
address problem, we are using "dnanexus/promtool:2.9.2" docker image
for verifying monitoring/prometheus/alerts/test_alerts.yml.

after this change, we use prometheus packaged by debian derivatives
instead of pulling a docker image.

* debian/control: add prometheus as a "make check" dependency
* install-deps.sh: partially revert
  53a5816deda0874a3a37e131e9bc22d88bb2a588, as we don't need to
  pull docker or start docker service for using promtool anymore.
* cmake: check if promtool is capable of running "test rules"
  command, bail out if it is not.

see also: https://tracker.ceph.com/issues/49653

Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit f381aa8bf0e175940153975fa1534ef0559ecadd)
debian/control
install-deps.sh
src/test/CMakeLists.txt