[COMMIT scylla-cluster-tests master] fix(artifacts pipeline): disable sct runner

0 views
Skip to first unread message

Commit Bot

<bot@cloudius-systems.com>
unread,
Sep 28, 2021, 3:51:07 AM9/28/21
to scylladb-dev@googlegroups.com, Amos Kong
From: Amos Kong <kongj...@gmail.com>
Committer: Evgeniy Naydanov <e.naj...@gmail.com>
Branch: master

fix(artifacts pipeline): disable sct runner

The test duration of artifacts-test jobs are always very short, the
test isn't heavy, and the problems are easy to reproduce.

It's waste of resource to use SCT runner for artifacts-test jobs.
This patch disabled it.

Signed-off-by: Amos Kong <kongj...@gmail.com>
Reported-by: Bentsi Magidovich <ben...@scylladb.com>

---
diff --git a/vars/artifactsPipeline.groovy b/vars/artifactsPipeline.groovy
--- a/vars/artifactsPipeline.groovy
+++ b/vars/artifactsPipeline.groovy
@@ -115,17 +115,6 @@ def call(Map pipelineParams) {
checkout scm
}
}
- stage('Create SCT Runner') {
- catchError(stageResult: 'FAILURE') {
- wrap([$class: 'BuildUser']) {
- dir('scylla-cluster-tests') {
- timeout(time: 5, unit: 'MINUTES') {
- createSctRunner(params, pipelineParams.timeout.time, builder.region)
- }
- }
- }
- }
- }
stage("Run SCT Test (${instance_type})") {
def cloud_provider = getCloudProviderFromBackend(params.backend)
sctScript """
@@ -195,12 +184,7 @@ def call(Map pipelineParams) {
export SCT_INSTANCE_PROVISION="${params.provision_type}"

echo "start test ......."
- RUNNER_IP=\$(cat sct_runner_ip||echo "")
- if [[ -n "\${RUNNER_IP}" ]] ; then
- ./docker/env/hydra.sh --execute-on-runner \${RUNNER_IP} run-test artifacts_test --backend ${params.backend}
- else
- ./docker/env/hydra.sh run-test artifacts_test --backend ${params.backend} --logdir "`pwd`"
- fi
+ ./docker/env/hydra.sh run-test artifacts_test --backend ${params.backend} --logdir "`pwd`"
echo "end test ....."
"""
}
@@ -236,15 +220,6 @@ def call(Map pipelineParams) {
}
}
}
- stage('Clean SCT Runners') {
- catchError(stageResult: 'FAILURE') {
- wrap([$class: 'BuildUser']) {
- dir('scylla-cluster-tests') {
- cleanSctRunners(params, currentBuild)
- }
- }
- }
- }
}
}
}
Reply all
Reply to author
Forward
0 new messages