Fix all integration tests using replay clients to disable retries

Author: vcjana

aws/sdk/integration-tests/cloudwatchlogs/tests/event_stream.rs

@@ -53,6 +53,7 @@ async fn start_request(
         .region(Region::from_static(region))
         .http_client(replayer.clone())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .credentials_provider(Credentials::for_tests())
         .build();
     let client = Client::from_conf(config);

aws/sdk/integration-tests/ec2/tests/waiters.rs

@@ -20,6 +20,7 @@ async fn prerequisites() -> (Client, ReplayingClient, TickAdvanceTime) {
         ReplayingClient::from_file("tests/instance-status-ok-waiter-success.json").unwrap();
     let config = aws_sdk_ec2::Config::builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .http_client(client.clone())
         .time_source(time_source.clone())
         .sleep_impl(sleep_impl)

aws/sdk/integration-tests/kms/tests/integration.rs

@@ -8,6 +8,7 @@ use aws_sdk_kms::operation::RequestId;
 use aws_smithy_http_client::test_util::{ReplayEvent, StaticReplayClient};
 use aws_smithy_runtime_api::client::result::SdkError;
 use aws_smithy_types::body::SdkBody;
+use aws_smithy_types::retry::RetryConfig;
 use http_1x::Uri;
 use kms::config::{Config, Credentials, Region};
 
@@ -66,6 +67,7 @@ async fn generate_random() {
         .region(Region::new("us-east-1"))
         .credentials_provider(Credentials::for_tests_with_session_token())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = kms::Client::from_conf(conf);
     let resp = client
@@ -145,6 +147,7 @@ async fn generate_random_keystore_not_found() {
         .region(Region::new("us-east-1"))
         .credentials_provider(Credentials::for_tests_with_session_token())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = kms::Client::from_conf(conf);
 

aws/sdk/integration-tests/s3/tests/checksums.rs

@@ -99,6 +99,7 @@ async fn test_checksum_on_streaming_response(
         .region(Region::new("us-east-1"))
         .http_client(http_client.clone())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
 
@@ -188,6 +189,7 @@ async fn test_checksum_on_streaming_request<'a>(
         .region(Region::new("us-east-1"))
         .http_client(http_client.clone())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
 

aws/sdk/integration-tests/s3/tests/content-length-enforcement.rs

@@ -3,7 +3,11 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-use aws_sdk_s3::{config::Region, error::DisplayErrorContext, Client, Config};
+use aws_sdk_s3::{
+    config::{retry::RetryConfig, Region},
+    error::DisplayErrorContext,
+    Client, Config,
+};
 use aws_smithy_http_client::test_util::dvr::ReplayingClient;
 
 #[tokio::test]
@@ -15,6 +19,7 @@ async fn test_content_length_enforcement_is_not_applied_to_head_request() {
         .with_test_defaults()
         .http_client(http_client.clone())
         .region(Region::new("us-east-1"))
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
     let _resp = client
@@ -40,6 +45,7 @@ async fn test_content_length_enforcement_get_request_short() {
         .with_test_defaults()
         .http_client(http_client.clone())
         .region(Region::new("us-east-1"))
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
     // The file we're fetching is exactly 10,000 bytes long, but we've set the
@@ -77,6 +83,7 @@ async fn test_content_length_enforcement_get_request_long() {
         .with_test_defaults()
         .http_client(http_client.clone())
         .region(Region::new("us-east-1"))
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
     // The file we're fetching is exactly 10,000 bytes long, but we've set the

aws/sdk/integration-tests/s3/tests/expires_interceptor.rs

@@ -28,6 +28,7 @@ fn make_client(expires_val: &str) -> Client {
         .region(Region::new("us-east-1"))
         .http_client(http_client)
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
 
     Client::from_conf(config)

aws/sdk/integration-tests/s3/tests/express.rs

@@ -10,7 +10,10 @@ use aws_config::Region;
 use aws_runtime::user_agent::test_util::{
     assert_ua_contains_metric_values, assert_ua_does_not_contain_metric_values,
 };
-use aws_sdk_s3::config::endpoint::{EndpointFuture, Params, ResolveEndpoint};
+use aws_sdk_s3::config::{
+    endpoint::{EndpointFuture, Params, ResolveEndpoint},
+    retry::RetryConfig,
+};
 use aws_sdk_s3::config::{Builder, Credentials};
 use aws_sdk_s3::presigning::PresigningConfig;
 use aws_sdk_s3::primitives::SdkBody;
@@ -324,6 +327,7 @@ async fn disable_s3_express_session_auth_at_service_client_level() {
         .http_client(http_client)
         .region(Region::new("us-west-2"))
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .disable_s3_express_session_auth(true)
         .build();
     let client = Client::from_conf(conf);
@@ -352,6 +356,7 @@ async fn disable_s3_express_session_auth_at_operation_level() {
         .http_client(http_client)
         .region(Region::new("us-west-2"))
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(conf);
 
@@ -455,6 +460,7 @@ async fn s3_express_auth_flow_should_not_be_reached_with_no_auth_schemes() {
             url: "http://127.0.0.1".to_owned(),
         })
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .timeout_config(
             TimeoutConfig::builder()
                 .operation_attempt_timeout(Duration::from_secs(1))

aws/sdk/integration-tests/s3/tests/ignore-invalid-xml-body-root.rs

@@ -47,6 +47,7 @@ async fn ignore_invalid_xml_body_root() {
         .region(Region::new("us-east-1"))
         .http_client(http_client.clone())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
 

aws/sdk/integration-tests/s3/tests/no_auth.rs

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-use aws_sdk_s3::config::Region;
+use aws_sdk_s3::config::{retry::RetryConfig, Region};
 use aws_sdk_s3::{Client, Config};
 use aws_smithy_http_client::test_util::capture_request;
 use aws_smithy_http_client::test_util::dvr::ReplayingClient;
@@ -25,6 +25,7 @@ async fn list_objects() {
     let config = Config::from(&config)
         .to_builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = aws_sdk_s3::Client::from_conf(config);
 
@@ -57,6 +58,7 @@ async fn list_objects_v2() {
     let config = Config::from(&config)
         .to_builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
 
@@ -88,6 +90,7 @@ async fn head_object() {
     let config = Config::from(&config)
         .to_builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
 
@@ -119,6 +122,7 @@ async fn get_object() {
     let config = Config::from(&config)
         .to_builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
 
@@ -170,6 +174,7 @@ async fn auth_scheme_preference_specifying_legacy_no_auth_scheme_id_should_be_su
         .http_client(http_client)
         .region(Region::new("us-east-2"))
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .auth_scheme_preference([AuthSchemeId::from("no_auth")])
         .build();
     let client = Client::from_conf(conf);

aws/sdk/integration-tests/s3/tests/signing-it.rs

@@ -6,7 +6,7 @@
 #![cfg(feature = "test-util")]
 
 use aws_credential_types::provider::SharedCredentialsProvider;
-use aws_sdk_s3::config::{Credentials, Region};
+use aws_sdk_s3::config::{retry::RetryConfig, Credentials, Region};
 use aws_sdk_s3::primitives::ByteStream;
 use aws_sdk_s3::{Client, Config};
 use aws_smithy_http_client::test_util::{capture_request, ReplayEvent, StaticReplayClient};
@@ -30,6 +30,7 @@ async fn test_signer() {
         .region(Region::new("us-east-1"))
         .http_client(http_client.clone())
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .build();
     let client = Client::from_conf(config);
     let _ = client
@@ -47,6 +48,7 @@ async fn disable_payload_signing_works() {
     let (http_client, request) = capture_request(None);
     let conf = aws_sdk_s3::Config::builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .behavior_version_latest()
         .region(Region::new("us-east-1"))
         .http_client(http_client)
@@ -80,6 +82,7 @@ async fn disable_payload_signing_works_with_checksums() {
     let (http_client, request) = capture_request(None);
     let conf = aws_sdk_s3::Config::builder()
         .with_test_defaults()
+        .retry_config(RetryConfig::disabled())
         .behavior_version_latest()
         .region(Region::new("us-east-1"))
         .http_client(http_client)