diff --git a/jobs/blobstore_benchmark/templates/storage_cli_config_buildpacks.json.erb b/jobs/blobstore_benchmark/templates/storage_cli_config_buildpacks.json.erb index 5478b5a47a..babfd8996e 100644 --- a/jobs/blobstore_benchmark/templates/storage_cli_config_buildpacks.json.erb +++ b/jobs/blobstore_benchmark/templates/storage_cli_config_buildpacks.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,6 +88,7 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/blobstore_benchmark/templates/storage_cli_config_droplets.json.erb b/jobs/blobstore_benchmark/templates/storage_cli_config_droplets.json.erb index 69d70c40f8..c9cbafd38a 100644 --- a/jobs/blobstore_benchmark/templates/storage_cli_config_droplets.json.erb +++ b/jobs/blobstore_benchmark/templates/storage_cli_config_droplets.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,6 +88,7 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/blobstore_benchmark/templates/storage_cli_config_packages.json.erb b/jobs/blobstore_benchmark/templates/storage_cli_config_packages.json.erb index 4208fffd2f..08aecc5f16 100644 --- a/jobs/blobstore_benchmark/templates/storage_cli_config_packages.json.erb +++ b/jobs/blobstore_benchmark/templates/storage_cli_config_packages.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,6 +88,7 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/blobstore_benchmark/templates/storage_cli_config_resource_pool.json.erb b/jobs/blobstore_benchmark/templates/storage_cli_config_resource_pool.json.erb index 1fa2c8549c..e8daeb4bed 100644 --- a/jobs/blobstore_benchmark/templates/storage_cli_config_resource_pool.json.erb +++ b/jobs/blobstore_benchmark/templates/storage_cli_config_resource_pool.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,6 +88,7 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cc_deployment_updater/templates/storage_cli_config_buildpacks.json.erb b/jobs/cc_deployment_updater/templates/storage_cli_config_buildpacks.json.erb index afdf3a07a5..babfd8996e 100644 --- a/jobs/cc_deployment_updater/templates/storage_cli_config_buildpacks.json.erb +++ b/jobs/cc_deployment_updater/templates/storage_cli_config_buildpacks.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,6 +88,7 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end @@ -107,6 +111,6 @@ if provider == "dav" options["tls"]={"cert"=>{"ca"=>ca_cert}} end end - + -%> <%= JSON.pretty_generate(options) %> diff --git a/jobs/cc_deployment_updater/templates/storage_cli_config_droplets.json.erb b/jobs/cc_deployment_updater/templates/storage_cli_config_droplets.json.erb index d4ee28965a..c9cbafd38a 100644 --- a/jobs/cc_deployment_updater/templates/storage_cli_config_droplets.json.erb +++ b/jobs/cc_deployment_updater/templates/storage_cli_config_droplets.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,8 +88,9 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end - + if provider == "dav" options["provider"] = "dav" diff --git a/jobs/cc_deployment_updater/templates/storage_cli_config_packages.json.erb b/jobs/cc_deployment_updater/templates/storage_cli_config_packages.json.erb index 4208fffd2f..08aecc5f16 100644 --- a/jobs/cc_deployment_updater/templates/storage_cli_config_packages.json.erb +++ b/jobs/cc_deployment_updater/templates/storage_cli_config_packages.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,6 +88,7 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cc_deployment_updater/templates/storage_cli_config_resource_pool.json.erb b/jobs/cc_deployment_updater/templates/storage_cli_config_resource_pool.json.erb index f43306b978..e8daeb4bed 100644 --- a/jobs/cc_deployment_updater/templates/storage_cli_config_resource_pool.json.erb +++ b/jobs/cc_deployment_updater/templates/storage_cli_config_resource_pool.json.erb @@ -32,6 +32,7 @@ if provider == "azurebs" options["account_key"] = l.p("#{scope}.azure_storage_access_key") add_optional(options, "environment", l.p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", l.p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -42,6 +43,7 @@ if provider == "gcs" options["bucket_name"] = l.p("#{scope}.bucket_name") add_optional(options, "storage_class", l.p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", l.p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -77,6 +79,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", l.p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", l.p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", l.p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -85,8 +88,9 @@ if provider == "alioss" options["access_key_secret"] = l.p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = l.p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = l.p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", l.p("#{scope}.http_request_timeout", nil)) end - + if provider == "dav" options["provider"] = "dav" diff --git a/jobs/cloud_controller_clock/templates/storage_cli_config_buildpacks.json.erb b/jobs/cloud_controller_clock/templates/storage_cli_config_buildpacks.json.erb index 75cf93c566..61d009cf08 100644 --- a/jobs/cloud_controller_clock/templates/storage_cli_config_buildpacks.json.erb +++ b/jobs/cloud_controller_clock/templates/storage_cli_config_buildpacks.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_clock/templates/storage_cli_config_droplets.json.erb b/jobs/cloud_controller_clock/templates/storage_cli_config_droplets.json.erb index 7a9f52e63f..96ce2243a4 100644 --- a/jobs/cloud_controller_clock/templates/storage_cli_config_droplets.json.erb +++ b/jobs/cloud_controller_clock/templates/storage_cli_config_droplets.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_clock/templates/storage_cli_config_packages.json.erb b/jobs/cloud_controller_clock/templates/storage_cli_config_packages.json.erb index 8e5af19330..774fb89732 100644 --- a/jobs/cloud_controller_clock/templates/storage_cli_config_packages.json.erb +++ b/jobs/cloud_controller_clock/templates/storage_cli_config_packages.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_clock/templates/storage_cli_config_resource_pool.json.erb b/jobs/cloud_controller_clock/templates/storage_cli_config_resource_pool.json.erb index 3ee5a48aca..aeef150bf1 100644 --- a/jobs/cloud_controller_clock/templates/storage_cli_config_resource_pool.json.erb +++ b/jobs/cloud_controller_clock/templates/storage_cli_config_resource_pool.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_ng/templates/storage_cli_config_buildpacks.json.erb b/jobs/cloud_controller_ng/templates/storage_cli_config_buildpacks.json.erb index 75cf93c566..61d009cf08 100644 --- a/jobs/cloud_controller_ng/templates/storage_cli_config_buildpacks.json.erb +++ b/jobs/cloud_controller_ng/templates/storage_cli_config_buildpacks.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_ng/templates/storage_cli_config_droplets.json.erb b/jobs/cloud_controller_ng/templates/storage_cli_config_droplets.json.erb index 7a9f52e63f..96ce2243a4 100644 --- a/jobs/cloud_controller_ng/templates/storage_cli_config_droplets.json.erb +++ b/jobs/cloud_controller_ng/templates/storage_cli_config_droplets.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_ng/templates/storage_cli_config_packages.json.erb b/jobs/cloud_controller_ng/templates/storage_cli_config_packages.json.erb index 8e5af19330..774fb89732 100644 --- a/jobs/cloud_controller_ng/templates/storage_cli_config_packages.json.erb +++ b/jobs/cloud_controller_ng/templates/storage_cli_config_packages.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_ng/templates/storage_cli_config_resource_pool.json.erb b/jobs/cloud_controller_ng/templates/storage_cli_config_resource_pool.json.erb index 3ee5a48aca..aeef150bf1 100644 --- a/jobs/cloud_controller_ng/templates/storage_cli_config_resource_pool.json.erb +++ b/jobs/cloud_controller_ng/templates/storage_cli_config_resource_pool.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_worker/templates/storage_cli_config_buildpacks.json.erb b/jobs/cloud_controller_worker/templates/storage_cli_config_buildpacks.json.erb index 75cf93c566..61d009cf08 100644 --- a/jobs/cloud_controller_worker/templates/storage_cli_config_buildpacks.json.erb +++ b/jobs/cloud_controller_worker/templates/storage_cli_config_buildpacks.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_worker/templates/storage_cli_config_droplets.json.erb b/jobs/cloud_controller_worker/templates/storage_cli_config_droplets.json.erb index 7a9f52e63f..96ce2243a4 100644 --- a/jobs/cloud_controller_worker/templates/storage_cli_config_droplets.json.erb +++ b/jobs/cloud_controller_worker/templates/storage_cli_config_droplets.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,6 +86,7 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end diff --git a/jobs/cloud_controller_worker/templates/storage_cli_config_packages.json.erb b/jobs/cloud_controller_worker/templates/storage_cli_config_packages.json.erb index 467dcff293..774fb89732 100644 --- a/jobs/cloud_controller_worker/templates/storage_cli_config_packages.json.erb +++ b/jobs/cloud_controller_worker/templates/storage_cli_config_packages.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,8 +86,9 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end - + if provider == "dav" options["provider"] = "dav" diff --git a/jobs/cloud_controller_worker/templates/storage_cli_config_resource_pool.json.erb b/jobs/cloud_controller_worker/templates/storage_cli_config_resource_pool.json.erb index a7805201f3..aeef150bf1 100644 --- a/jobs/cloud_controller_worker/templates/storage_cli_config_resource_pool.json.erb +++ b/jobs/cloud_controller_worker/templates/storage_cli_config_resource_pool.json.erb @@ -30,6 +30,7 @@ if provider == "azurebs" options["account_key"] = p("#{scope}.azure_storage_access_key") add_optional(options, "environment", p("#{scope}.environment", "AzureCloud")) add_optional(options, "put_timeout_in_seconds", p("#{scope}.put_timeout_in_seconds", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) options = cli_cfg_with_default_timeout(options, 'storage_cli') end @@ -40,6 +41,7 @@ if provider == "gcs" options["bucket_name"] = p("#{scope}.bucket_name") add_optional(options, "storage_class", p("#{scope}.storage_class", nil)) add_optional(options, "encryption_key", p("#{scope}.encryption_key", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "s3" @@ -75,6 +77,7 @@ if provider == "s3" add_optional(options, "request_checksum_calculation_enabled", p("#{scope}.request_checksum_calculation_enabled", nil)) add_optional(options, "response_checksum_calculation_enabled", p("#{scope}.response_checksum_calculation_enabled", nil)) add_optional(options, "uploader_request_checksum_calculation_enabled", p("#{scope}.uploader_request_checksum_calculation_enabled", nil)) + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end if provider == "alioss" @@ -83,8 +86,9 @@ if provider == "alioss" options["access_key_secret"] = p("#{scope}.aliyun_accesskey_secret") options["endpoint"] = p("#{scope}.aliyun_oss_endpoint") options["bucket_name"] = p("#{scope}.aliyun_oss_bucket") + add_optional(options, "http_request_timeout", p("#{scope}.http_request_timeout", nil)) end - + if provider == "dav" options["provider"] = "dav" diff --git a/spec/cc_deployment_updater/storage_cli_config_jsons_spec.rb b/spec/cc_deployment_updater/storage_cli_config_jsons_spec.rb index ba2fc4900f..55f76648e7 100644 --- a/spec/cc_deployment_updater/storage_cli_config_jsons_spec.rb +++ b/spec/cc_deployment_updater/storage_cli_config_jsons_spec.rb @@ -105,6 +105,29 @@ def props_for_provider(provider) json = YAML.safe_load(template.render(props, consumes: links)) expect(json['put_timeout_in_seconds']).to eq('7') end + + it 'includes http_request_timeout when provided' do + set(link_props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json['http_request_timeout']).to eq('30s') + end + + it 'excludes http_request_timeout when not provided' do + set(link_props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).not_to have_key('http_request_timeout') + end end end end @@ -192,7 +215,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -222,7 +246,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' ) end end @@ -265,8 +290,8 @@ def props_for_provider(provider) 'bucket_name' => 'bucket', 'google_json_key_string' => '{}', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' - + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -276,7 +301,8 @@ def props_for_provider(provider) 'json_key' => '{}', 'credentials_source' => 'static', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' ) end end @@ -315,6 +341,19 @@ def props_for_provider(provider) 'bucket_name' => 'bucket' ) end + + it 'includes http_request_timeout when provided' do + set(link_props, keypath, { + 'provider' => 'alioss', + 'aliyun_accesskey_id' => 'key', + 'aliyun_accesskey_secret' => 'secret', + 'aliyun_oss_endpoint' => 'alioss.com', + 'aliyun_oss_bucket' => 'bucket', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).to include('http_request_timeout' => '30s') + end end end end diff --git a/spec/cloud_controller_clock/storage_cli_config_jsons_spec.rb b/spec/cloud_controller_clock/storage_cli_config_jsons_spec.rb index 2b9f75e4bf..50de9833fe 100644 --- a/spec/cloud_controller_clock/storage_cli_config_jsons_spec.rb +++ b/spec/cloud_controller_clock/storage_cli_config_jsons_spec.rb @@ -90,6 +90,29 @@ def props_for_provider(provider) json = YAML.safe_load(template.render(props, consumes: links)) expect(json['put_timeout_in_seconds']).to eq('7') end + + it 'includes http_request_timeout when provided' do + set(props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json['http_request_timeout']).to eq('30s') + end + + it 'excludes http_request_timeout when not provided' do + set(props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).not_to have_key('http_request_timeout') + end end end end @@ -169,7 +192,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -199,7 +223,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' ) end end @@ -234,8 +259,8 @@ def props_for_provider(provider) 'bucket_name' => 'bucket', 'google_json_key_string' => '{}', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' - + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -245,7 +270,8 @@ def props_for_provider(provider) 'json_key' => '{}', 'credentials_source' => 'static', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' ) end end @@ -276,6 +302,19 @@ def props_for_provider(provider) 'bucket_name' => 'bucket' ) end + + it 'includes http_request_timeout when provided' do + set(props, keypath, { + 'provider' => 'alioss', + 'aliyun_accesskey_id' => 'key', + 'aliyun_accesskey_secret' => 'secret', + 'aliyun_oss_endpoint' => 'alioss.com', + 'aliyun_oss_bucket' => 'bucket', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).to include('http_request_timeout' => '30s') + end end end end diff --git a/spec/cloud_controller_ng/storage_cli_config_jsons_spec.rb b/spec/cloud_controller_ng/storage_cli_config_jsons_spec.rb index d408f0ca7b..1e2ebcb481 100644 --- a/spec/cloud_controller_ng/storage_cli_config_jsons_spec.rb +++ b/spec/cloud_controller_ng/storage_cli_config_jsons_spec.rb @@ -90,6 +90,29 @@ def props_for_provider(provider) json = YAML.safe_load(template.render(props, consumes: links)) expect(json['put_timeout_in_seconds']).to eq('7') end + + it 'includes http_request_timeout when provided' do + set(props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json['http_request_timeout']).to eq('30s') + end + + it 'excludes http_request_timeout when not provided' do + set(props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).not_to have_key('http_request_timeout') + end end end end @@ -169,7 +192,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -199,7 +223,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' ) end end @@ -234,8 +259,8 @@ def props_for_provider(provider) 'bucket_name' => 'bucket', 'google_json_key_string' => '{}', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' - + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -245,7 +270,8 @@ def props_for_provider(provider) 'json_key' => '{}', 'credentials_source' => 'static', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' ) end end @@ -276,6 +302,19 @@ def props_for_provider(provider) 'bucket_name' => 'bucket' ) end + + it 'includes http_request_timeout when provided' do + set(props, keypath, { + 'provider' => 'alioss', + 'aliyun_accesskey_id' => 'key', + 'aliyun_accesskey_secret' => 'secret', + 'aliyun_oss_endpoint' => 'alioss.com', + 'aliyun_oss_bucket' => 'bucket', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).to include('http_request_timeout' => '30s') + end end end end diff --git a/spec/cloud_controller_worker/storage_cli_config_jsons_spec.rb b/spec/cloud_controller_worker/storage_cli_config_jsons_spec.rb index 23cf5a90c4..293158714e 100644 --- a/spec/cloud_controller_worker/storage_cli_config_jsons_spec.rb +++ b/spec/cloud_controller_worker/storage_cli_config_jsons_spec.rb @@ -90,6 +90,29 @@ def props_for_provider(provider) json = YAML.safe_load(template.render(props, consumes: links)) expect(json['put_timeout_in_seconds']).to eq('7') end + + it 'includes http_request_timeout when provided' do + set(props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json['http_request_timeout']).to eq('30s') + end + + it 'excludes http_request_timeout when not provided' do + set(props, keypath, { + 'provider' => 'azurebs', + 'azure_storage_account_name' => 'acc', + 'azure_storage_access_key' => 'key', + 'container_name' => 'cont' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).not_to have_key('http_request_timeout') + end end end end @@ -169,7 +192,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -199,7 +223,8 @@ def props_for_provider(provider) 'single_upload_threshold' => 2048, 'request_checksum_calculation_enabled' => false, 'response_checksum_calculation_enabled' => false, - 'uploader_request_checksum_calculation_enabled' => false + 'uploader_request_checksum_calculation_enabled' => false, + 'http_request_timeout' => '30s' ) end end @@ -234,8 +259,8 @@ def props_for_provider(provider) 'bucket_name' => 'bucket', 'google_json_key_string' => '{}', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' - + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' }) json = YAML.safe_load(template.render(props, consumes: links)) @@ -245,7 +270,8 @@ def props_for_provider(provider) 'json_key' => '{}', 'credentials_source' => 'static', 'storage_class' => 'STANDARD', - 'encryption_key' => 'key' + 'encryption_key' => 'key', + 'http_request_timeout' => '30s' ) end end @@ -276,6 +302,19 @@ def props_for_provider(provider) 'bucket_name' => 'bucket' ) end + + it 'includes http_request_timeout when provided' do + set(props, keypath, { + 'provider' => 'alioss', + 'aliyun_accesskey_id' => 'key', + 'aliyun_accesskey_secret' => 'secret', + 'aliyun_oss_endpoint' => 'alioss.com', + 'aliyun_oss_bucket' => 'bucket', + 'http_request_timeout' => '30s' + }) + json = YAML.safe_load(template.render(props, consumes: links)) + expect(json).to include('http_request_timeout' => '30s') + end end end end