Skip to content

Commit bba2288

Browse files
committed
Improve comment further
thanks @chess-knight! Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
1 parent 4ff10ad commit bba2288

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

Tests/iaas/scs_0123_mandatory_services/mandatory_services.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ def s3_conn(creds, conn):
3030
"""Return an s3 client conn"""
3131
cfg = conn.config.config
3232
# Take insecure/verify/cacert parameter from clouds.yaml and pass it to boto3.resource.
33-
# If verify is True in clouds.yaml, fall back to cacert or None. In the latter case,
34-
# the default boto3 behavior is applied (where REQUESTS_CA_BUNDLE can still be used,
35-
# and otherwise, boto3 defaults to verify=True).
33+
# If insecure is False/None and verify is True/None in clouds.yaml, fall back to cacert or None.
34+
# In the latter case (None), the default boto3 behavior is applied (where config file or env
35+
# variables can still be used, and otherwise, boto3 defaults to verify=True).
3636
# Note: cacert must be used to pass the certificate; don't use verify for that; cf.
3737
# https://docs.openstack.org/openstacksdk/latest/user/config/configuration.html#ssl-settings
3838
if cfg.get("insecure") or not cfg.get("verify", True):

0 commit comments

Comments
 (0)