From: Alfonso Martínez <almartin@redhat.com>
Date: Wed, 11 Aug 2021 08:25:42 +0000 (+0200)
Subject: mgr/dashboard: run-cephadm-e2e-tests.sh improvements
X-Git-Tag: v16.2.6~62^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F42789%2Fhead;p=ceph.git

mgr/dashboard: run-cephadm-e2e-tests.sh improvements

- Jenkins env.: make sure the cluster is always started.
- PR template: add trigger phrase to the jenkins commands list.
- Cypress: add --no-install flag; clean previous reports.

Fixes: https://tracker.ceph.com/issues/52082
Signed-off-by: Alfonso Martínez <almartin@redhat.com>
(cherry picked from commit cadf8c75ac0266e42c1ea0b4d6733590ea3dac21)
---

diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
index 4d8d4d41a0d2..46ad1113ddd1 100644
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -48,6 +48,7 @@ https://raw.githubusercontent.com/ceph/ceph/master/SubmittingPatches.rst
 - `jenkins test make check arm64`
 - `jenkins test submodules`
 - `jenkins test dashboard`
+- `jenkins test dashboard cephadm`
 - `jenkins test api`
 - `jenkins test docs`
 - `jenkins render docs`
diff --git a/src/pybind/mgr/dashboard/ci/cephadm/run-cephadm-e2e-tests.sh b/src/pybind/mgr/dashboard/ci/cephadm/run-cephadm-e2e-tests.sh
index 178c89f5ba60..e10929716089 100755
--- a/src/pybind/mgr/dashboard/ci/cephadm/run-cephadm-e2e-tests.sh
+++ b/src/pybind/mgr/dashboard/ci/cephadm/run-cephadm-e2e-tests.sh
@@ -13,11 +13,9 @@ get_vm_ip () {
     echo -n $ip
 }
 
-if [[ -z "${CYPRESS_BASE_URL}" ]]; then
-    CEPH_NODE_00_IP="$(get_vm_ip ceph-node-00)"
-    if [[ -z "${CEPH_NODE_00_IP}" ]]; then
-        . "$(dirname $0)"/start-cluster.sh
-    fi
+if [[ -n "${JENKINS_HOME}" || (-z "${CYPRESS_BASE_URL}" && -z "$(get_vm_ip ceph-node-00)") ]]; then
+    . "$(dirname $0)"/start-cluster.sh
+
     CYPRESS_BASE_URL="https://$(get_vm_ip ceph-node-00):${DASHBOARD_PORT}"
 fi
 
@@ -27,11 +25,13 @@ cypress_run () {
     local specs="$1"
     local timeout="$2"
     local override_config="ignoreTestFiles=*.po.ts,retries=0,testFiles=${specs}"
-
     if [[ -n "$timeout" ]]; then
         override_config="${override_config},defaultCommandTimeout=${timeout}"
     fi
-    npx cypress run ${CYPRESS_ARGS} --browser chrome --headless --config "$override_config"
+
+    rm -f cypress/reports/results-*.xml || true
+
+    npx --no-install cypress run ${CYPRESS_ARGS} --browser chrome --headless --config "$override_config"
 }
 
 : ${CEPH_DEV_FOLDER:=${PWD}}