Skip to content

Commit bc76004

Browse files
authored
Merge pull request ceph#64069 from phlogistonjohn/jjm-bwc-test-tweak
test: improvments for running make check test suite in containers
2 parents a217c0e + f80dad2 commit bc76004

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

src/pybind/mgr/dashboard/run-frontend-unittests.sh

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,16 @@ else
4141
fi
4242
fi
4343

44+
# The coverage tool embeds an absolute path in the coverage xml file.
45+
# This causes issues when the suite is run within a container and external
46+
# tools
47+
cov_xml="coverage/cobertura-coverage.xml"
48+
if [ -f "${cov_xml}" -a -n "$REWRITE_COVERAGE_ROOTDIR" ]; then
49+
echo "Updating ${cov_xml}"
50+
cp "${cov_xml}" "${cov_xml}.orig"
51+
xmlstarlet ed --inplace -u /coverage/sources/source -v "${REWRITE_COVERAGE_ROOTDIR}" "${cov_xml}"
52+
fi
53+
4454
if [ `uname` != "FreeBSD" ]; then
4555
deactivate
4656
fi

src/script/build-with-container.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -534,7 +534,6 @@ def build_container(ctx):
534534
"-t",
535535
ctx.image_name,
536536
f"--label=io.ceph.build-with-container.src={_hash_sources()}",
537-
f"--build-arg=JENKINS_HOME={ctx.cli.homedir}",
538537
f"--build-arg=CEPH_BASE_BRANCH={ctx.base_branch()}",
539538
]
540539
if ctx.cli.distro:

0 commit comments

Comments
 (0)