From 21ee3995aa32168dc59e78ec946c9f71b28b1cb1 Mon Sep 17 00:00:00 2001 From: Christophe Haen Date: Tue, 16 Jun 2020 14:33:29 +0200 Subject: [PATCH 1/3] REMOVE READLONLY VARIABLES --- tests/Jenkins/dirac_ci.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/Jenkins/dirac_ci.sh b/tests/Jenkins/dirac_ci.sh index 387848c9ce8..9e3472802fc 100644 --- a/tests/Jenkins/dirac_ci.sh +++ b/tests/Jenkins/dirac_ci.sh @@ -66,11 +66,11 @@ fi # Creating default structure mkdir -p "$WORKSPACE/TestCode" # Where the test code resides -readonly TESTCODE=${_} +TESTCODE=${_} mkdir -p "$WORKSPACE/ServerInstallDIR" # Where servers are installed -readonly SERVERINSTALLDIR=${_} +SERVERINSTALLDIR=${_} mkdir -p "$WORKSPACE/ClientInstallDIR" # Where clients are installed -readonly CLIENTINSTALLDIR=${_} +CLIENTINSTALLDIR=${_} # Location of the CFG file to be used (this can be replaced by the extensions) INSTALL_CFG_FILE="${TESTCODE}/DIRAC/tests/Jenkins/install.cfg" From 1900ac8e248296ac7a180137e9456a13c74aefba Mon Sep 17 00:00:00 2001 From: Christophe Haen Date: Tue, 16 Jun 2020 15:47:49 +0200 Subject: [PATCH 2/3] SMS: drop tables in the correct order --- StorageManagementSystem/DB/StorageManagementDB.sql | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/StorageManagementSystem/DB/StorageManagementDB.sql b/StorageManagementSystem/DB/StorageManagementDB.sql index e1cb451b30e..a446b922fce 100644 --- a/StorageManagementSystem/DB/StorageManagementDB.sql +++ b/StorageManagementSystem/DB/StorageManagementDB.sql @@ -9,7 +9,11 @@ use StorageManagementDB; +DROP TABLE IF EXISTS TaskReplicas; +DROP TABLE IF EXISTS StageRequests; +DROP TABLE IF EXISTS CacheReplicas; DROP TABLE IF EXISTS Tasks; + CREATE TABLE Tasks( TaskID INTEGER AUTO_INCREMENT, Status VARCHAR(32) DEFAULT 'New', @@ -23,7 +27,6 @@ CREATE TABLE Tasks( INDEX(TaskID,Status) )ENGINE=INNODB; -DROP TABLE IF EXISTS TaskReplicas; CREATE TABLE TaskReplicas( TaskID INTEGER(8) NOT NULL REFERENCES Tasks(TaskID), ReplicaID INTEGER(8) NOT NULL REFERENCES CacheReplicas(ReplicaID), @@ -33,7 +36,7 @@ CREATE TABLE TaskReplicas( CREATE TRIGGER taskreplicasAfterInsert AFTER INSERT ON TaskReplicas FOR EACH ROW UPDATE CacheReplicas SET CacheReplicas.Links=CacheReplicas.Links+1 WHERE CacheReplicas.ReplicaID=NEW.ReplicaID; CREATE TRIGGER taskreplicasAfterDelete AFTER DELETE ON TaskReplicas FOR EACH ROW UPDATE CacheReplicas SET CacheReplicas.Links=CacheReplicas.Links-1 WHERE CacheReplicas.ReplicaID=OLD.ReplicaID; -DROP TABLE IF EXISTS CacheReplicas; + CREATE TABLE CacheReplicas( ReplicaID INTEGER AUTO_INCREMENT, Type VARCHAR(32) NOT NULL, @@ -52,7 +55,7 @@ CREATE TABLE CacheReplicas( INDEX(ReplicaID,Status,SE) )ENGINE=INNODB; -DROP TABLE IF EXISTS StageRequests; + CREATE TABLE StageRequests( ReplicaID INTEGER(8) NOT NULL REFERENCES CacheReplicas(ReplicaID), StageStatus VARCHAR(32) DEFAULT 'StageSubmitted', From 8c094852784af843c7aee5f57b1562c8b8582db4 Mon Sep 17 00:00:00 2001 From: Christophe Haen Date: Tue, 16 Jun 2020 17:46:19 +0200 Subject: [PATCH 3/3] tests: fix dropDBs bad escape --- tests/Jenkins/utilities.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/Jenkins/utilities.sh b/tests/Jenkins/utilities.sh index 5268ad15e72..ee1a1b9292d 100644 --- a/tests/Jenkins/utilities.sh +++ b/tests/Jenkins/utilities.sh @@ -892,8 +892,9 @@ diracMVDFCDB(){ dropDBs(){ echo '==> [dropDBs]' - dbs=$(cut -d ' ' -f 2 < databases | cut -d '.' -f 1 | grep -v ^RequestDB | grep -v ^FileCatalogDB) - python "${TESTCODE}/DIRAC/tests/Jenkins/dirac-drop-db.py" "$dbs" "${DEBUG}" + # make dbs a real array to avoid future mistake with escaping + mapfile -t dbs < <(cut -d ' ' -f 2 < /tmp/databases.txt | cut -d '.' -f 1 | grep -v ^RequestDB | grep -v ^FileCatalogDB) + python "${TESTCODE}/DIRAC/tests/Jenkins/dirac-drop-db.py" "${dbs[@]}" "${DEBUG}" } #-------------------------------------------------------------------------------