Skip to content

Commit be56d25

Browse files
committed
refactor: remove warn when mandatory parameters are not defined
1 parent 57a04f9 commit be56d25

File tree

1 file changed

+33
-35
lines changed

1 file changed

+33
-35
lines changed

src/DIRAC/ConfigurationSystem/Client/Utilities.py

Lines changed: 33 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -465,52 +465,50 @@ def getElasticDBParameters(fullname):
465465
certs = result["Value"]
466466
parameters["CRT"] = certs
467467

468-
# Elasticsearch ca_certs
469-
result = gConfig.getOption(cs_path + "/ca_certs")
470-
if not result["OK"]:
471-
# No CA certificate found, try at the common place
472-
result = gConfig.getOption("/Systems/NoSQLDatabases/ca_certs")
468+
# If connection is through certificates get the mandatory parameters: ca_certs, client_key, client_cert
469+
if parameters["CRT"]:
470+
parameters["Password"] = None
471+
parameters["User"] = None
472+
473+
# Elasticsearch ca_certs
474+
result = gConfig.getOption(cs_path + "/ca_certs")
473475
if not result["OK"]:
474-
gLogger.debug("Failed to get the configuration parameter: ca_certs. Using None")
475-
ca_certs = None
476+
# No CA certificate found, try at the common place
477+
result = gConfig.getOption("/Systems/NoSQLDatabases/ca_certs")
478+
if not result["OK"]:
479+
return S_ERROR("Failed to get the configuration parameter: ca_certs.")
480+
else:
481+
ca_certs = result["Value"]
476482
else:
477483
ca_certs = result["Value"]
478-
else:
479-
ca_certs = result["Value"]
480-
parameters["ca_certs"] = ca_certs
484+
parameters["ca_certs"] = ca_certs
481485

482-
# Elasticsearch client_key
483-
result = gConfig.getOption(cs_path + "/client_key")
484-
if not result["OK"]:
485-
# No client private key found, try at the common place
486-
result = gConfig.getOption("/Systems/NoSQLDatabases/client_key")
486+
# Elasticsearch client_key
487+
result = gConfig.getOption(cs_path + "/client_key")
487488
if not result["OK"]:
488-
gLogger.debug("Failed to get the configuration parameter: client_key. Using None")
489-
client_key = None
489+
# No client private key found, try at the common place
490+
result = gConfig.getOption("/Systems/NoSQLDatabases/client_key")
491+
if not result["OK"]:
492+
return S_ERROR("Failed to get the configuration parameter: client_key.")
493+
else:
494+
client_key = result["Value"]
490495
else:
491496
client_key = result["Value"]
492-
else:
493-
client_key = result["Value"]
494-
parameters["client_key"] = client_key
497+
parameters["client_key"] = client_key
495498

496-
# Elasticsearch client_cert
497-
result = gConfig.getOption(cs_path + "/client_cert")
498-
if not result["OK"]:
499-
# No cient certificate found, try at the common place
500-
result = gConfig.getOption("/Systems/NoSQLDatabases/client_cert")
499+
# Elasticsearch client_cert
500+
result = gConfig.getOption(cs_path + "/client_cert")
501501
if not result["OK"]:
502-
gLogger.debug("Failed to get the configuration parameter: client_cert. Using None")
503-
client_cert = None
502+
# No cient certificate found, try at the common place
503+
result = gConfig.getOption("/Systems/NoSQLDatabases/client_cert")
504+
if not result["OK"]:
505+
return S_ERROR("Failed to get the configuration parameter: client_cert.")
506+
else:
507+
client_cert = result["Value"]
504508
else:
505509
client_cert = result["Value"]
506-
else:
507-
client_cert = result["Value"]
508-
parameters["client_cert"] = client_cert
509-
510-
# If connection is not through certificates get: Password, User
511-
if parameters["CRT"]:
512-
parameters["Password"] = None
513-
parameters["User"] = None
510+
parameters["client_cert"] = client_cert
511+
# If connection is not through certificates get the mandatory parameters: Password, User
514512
else:
515513
result = gConfig.getOption(cs_path + "/Password")
516514
if not result["OK"]:

0 commit comments

Comments
 (0)