From b58384c411c054d8f018f24e686fe4fd9a77a77e Mon Sep 17 00:00:00 2001 From: Federico Stagni Date: Mon, 2 Dec 2024 18:01:26 +0100 Subject: [PATCH] fix: even more setup removals --- .../Configuration/ConfigurationStructure/index.rst | 2 +- .../Systems/WorkloadManagement/Pilots/index.rst | 7 ------- .../AddingNewComponents/CheckYourInstallation/index.rst | 8 ++++---- .../interactingWithProductionSetups.rst | 1 - .../source/DeveloperGuide/Internals/Core/ClientServer.rst | 4 ++-- src/DIRAC/Core/scripts/dirac_configure.py | 2 -- src/DIRAC/MonitoringSystem/Service/MonitoringHandler.py | 4 ---- tests/CI/install_client.sh | 1 - .../TornadoServices/Test_TornadoAndDISETmixed.py | 2 -- 9 files changed, 7 insertions(+), 24 deletions(-) diff --git a/docs/source/AdministratorGuide/Configuration/ConfigurationStructure/index.rst b/docs/source/AdministratorGuide/Configuration/ConfigurationStructure/index.rst index b4bc997436f..8162acd6fbe 100644 --- a/docs/source/AdministratorGuide/Configuration/ConfigurationStructure/index.rst +++ b/docs/source/AdministratorGuide/Configuration/ConfigurationStructure/index.rst @@ -52,7 +52,7 @@ in the order of preference of the option resolution: For all the DIRAC commands there is option '-o' defined which takes one configuration option setting. For example:: - dirac-wms-job-submit job.jdl -o /DIRAC/Setup=Dirac-Production + dirac-wms-job-submit job.jdl *Command line argument specifying a CFG file* A config file can be passed to any dirac command with the ``--cfg`` flag:: diff --git a/docs/source/AdministratorGuide/Systems/WorkloadManagement/Pilots/index.rst b/docs/source/AdministratorGuide/Systems/WorkloadManagement/Pilots/index.rst index 7d1e80c3c09..0fa83304ae2 100644 --- a/docs/source/AdministratorGuide/Systems/WorkloadManagement/Pilots/index.rst +++ b/docs/source/AdministratorGuide/Systems/WorkloadManagement/Pilots/index.rst @@ -215,10 +215,6 @@ A simple example using the LHCbPilot extension follows:: DIRAC_SITE="${i#*=}" shift ;; - --lhcb-setup=*) - LHCBDIRAC_SETUP="${i#*=}" - shift - ;; --ce-name=*) CE_NAME="${i#*=}" shift @@ -237,8 +233,6 @@ A simple example using the LHCbPilot extension follows:: esac done - # Default if not given explicitly - LHCBDIRAC_SETUP=${LHCBDIRAC_SETUP:-LHCb-Production} # JOB_ID is used by when reporting LocalJobID by DIRAC watchdog #export JOB_ID="$VMTYPE:$VM_UUID" @@ -270,7 +264,6 @@ A simple example using the LHCbPilot extension follows:: #run the dirac-pilot script python dirac-pilot.py \ - --setup $LHCBDIRAC_SETUP \ --project LHCb \ --Name "$CE_NAME" \ --name "$1" \ diff --git a/docs/source/DeveloperGuide/AddingNewComponents/CheckYourInstallation/index.rst b/docs/source/DeveloperGuide/AddingNewComponents/CheckYourInstallation/index.rst index a3dc7c49205..050eba9014e 100644 --- a/docs/source/DeveloperGuide/AddingNewComponents/CheckYourInstallation/index.rst +++ b/docs/source/DeveloperGuide/AddingNewComponents/CheckYourInstallation/index.rst @@ -102,15 +102,15 @@ We will now play with a **dirac.cfg** file. For these exercises you can use the Try this:: >>> from DIRAC import gConfig - >>> gConfig.getValue('/DIRAC/Setup') - 'DeveloperSetup' + >>> gConfig.getValue('/DIRAC/DefaultGroup') + 'dirac_user' -Where does 'DeveloperSetup' come from? Open that dirac.cfg and search for it. Got it? it's in:: +Where does 'dirac_user' come from? Open that dirac.cfg and search for it. Got it? it's in:: DIRAC { ... - Setup = DeveloperSetup + DefaultGroup = dirac_user ... } diff --git a/docs/source/DeveloperGuide/DevelopmentEnvironment/DeveloperInstallation/interactingWithProductionSetups.rst b/docs/source/DeveloperGuide/DevelopmentEnvironment/DeveloperInstallation/interactingWithProductionSetups.rst index 9f2cb144b20..e474859578f 100644 --- a/docs/source/DeveloperGuide/DevelopmentEnvironment/DeveloperInstallation/interactingWithProductionSetups.rst +++ b/docs/source/DeveloperGuide/DevelopmentEnvironment/DeveloperInstallation/interactingWithProductionSetups.rst @@ -11,7 +11,6 @@ So, the only real thing that you need to have is: - a DIRAC developer installation - a (real) certificate, that is recognized by your server installation -- a dirac.cfg that include the (real) setup of the production environment that you want to connect to (in DIRAC/Setup section) - a dirac.cfg that include the (real) URL of the production Configuration server. The last 2 bullets can be achieved with the following command:: diff --git a/docs/source/DeveloperGuide/Internals/Core/ClientServer.rst b/docs/source/DeveloperGuide/Internals/Core/ClientServer.rst index 944289afe6c..a02da426b28 100644 --- a/docs/source/DeveloperGuide/Internals/Core/ClientServer.rst +++ b/docs/source/DeveloperGuide/Internals/Core/ClientServer.rst @@ -166,10 +166,10 @@ Complete path of packages are not on the diagram for readability: - requestHandler: :py:class:`DIRAC.Core.DISET.RequestHandler` -You can see that the client sends a proposalTuple, proposalTuple contain (service, setup, ClientVO) then (typeOfCall, method) and finaly extra-credentials. +You can see that the client sends a proposalTuple, proposalTuple contain (service, ClientVO) then (typeOfCall, method) and finaly extra-credentials. e.g.:: - (('Framework/serviceName', 'DeveloperSetup', 'unknown'), ('RPC', 'methodName'), '') + (('Framework/serviceName', 'unknown'), ('RPC', 'methodName'), '') diff --git a/src/DIRAC/Core/scripts/dirac_configure.py b/src/DIRAC/Core/scripts/dirac_configure.py index e381adf429b..ad2fba345d5 100755 --- a/src/DIRAC/Core/scripts/dirac_configure.py +++ b/src/DIRAC/Core/scripts/dirac_configure.py @@ -493,8 +493,6 @@ def runDiracConfigure(params): else: DIRAC.gLogger.notice(f"Will update {DIRAC.gConfig.diracConfigFilePath}") - if params.setup: - DIRAC.gLogger.verbose("/DIRAC/Setup =", params.setup) if params.vo: DIRAC.gLogger.verbose("/DIRAC/VirtualOrganization =", params.vo) if params.configurationServer: diff --git a/src/DIRAC/MonitoringSystem/Service/MonitoringHandler.py b/src/DIRAC/MonitoringSystem/Service/MonitoringHandler.py index 84b7cc4f488..18ee70a9f2a 100644 --- a/src/DIRAC/MonitoringSystem/Service/MonitoringHandler.py +++ b/src/DIRAC/MonitoringSystem/Service/MonitoringHandler.py @@ -71,8 +71,6 @@ def initializeHandler(cls, serviceInfo): return S_ERROR(f"Data location is not writable: {repr(err)}") gDataCache.setGraphsLocation(dataPath) - cls.diracSetup = CSGlobals.getSetup().lower() - return S_OK() types_listUniqueKeyValues = [str] @@ -277,7 +275,6 @@ def export_addRecords(self, indexname, monitoringType, data): :param list data: data to insert :returns: S_OK or S_ERROR """ - indexname = f"{self.diracSetup.lower()}_{indexname}" gLogger.debug("Bulk index:", indexname) mapping = self.__db.getMapping(monitoringType) gLogger.debug("Mapping:", mapping) @@ -292,7 +289,6 @@ def export_deleteIndex(self, indexName): :param str indexName: name of the index """ - indexName = f"{self.diracSetup.lower()}_{indexName}" gLogger.debug("delete index:", indexName) return self.__db.deleteIndex(indexName) diff --git a/tests/CI/install_client.sh b/tests/CI/install_client.sh index 85db87fb450..c2e72e586a3 100755 --- a/tests/CI/install_client.sh +++ b/tests/CI/install_client.sh @@ -51,7 +51,6 @@ echo -e "*** $(date -u) **** Got the DIRAC tests ****\n" source "${DIRAC_CI_SETUP_SCRIPT}" # shellcheck disable=SC2034 -DIRACSETUP=$(< "${INSTALL_CFG_FILE}" grep "Setup = " | cut -f5 -d " ") echo -e "*** $(date -u) **** Client INSTALLATION START ****\n" diff --git a/tests/Integration/TornadoServices/Test_TornadoAndDISETmixed.py b/tests/Integration/TornadoServices/Test_TornadoAndDISETmixed.py index 06faa9740e5..d1b94791ad1 100644 --- a/tests/Integration/TornadoServices/Test_TornadoAndDISETmixed.py +++ b/tests/Integration/TornadoServices/Test_TornadoAndDISETmixed.py @@ -56,7 +56,6 @@ ``` DIRAC { - Setup = DeveloperSetup Configuration { Servers = dips://localhost:9135/Configuration/Server # Case 1 @@ -95,7 +94,6 @@ ``` DIRAC { - Setup = DeveloperSetup Configuration { Servers = dips://localhost:9135/Configuration/Server