Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

resource/aws_cloudwatch_log_group: Automatically trim :* suffix from ARN in API response #14214

Merged
merged 2 commits into from
Jul 30, 2020

Commits on Jul 30, 2020

  1. resource/aws_cloudwatch_log_group: Automatically trim :* suffix from …

    …ARN in API response
    
    Reference: #13046
    Reference: #13509
    
    Previously:
    
    ```
        TestAccAWSDataSyncTask_CloudWatchLogGroupARN: testing.go:684: Step 0 error: errors during apply:
    
            Error: error creating DataSync Task: ValidationException: 1 validation error detected: Value 'arn:aws:logs:us-west-2:123456789012:log-group:tf-acc-test-4735468151095290255:*' at 'cloudWatchLogGroupArn' failed to satisfy constraint: Member must satisfy regular expression pattern: ^arn:(aws|aws-cn|aws-us-gov|aws-iso|aws-iso-b):logs:[a-z\-0-9]*:[0-9]{12}:log-group:([^:\*]*)$
    ```
    
    Output from acceptance testing (`aws_route53_query_log` failure related to similar issue #13510):
    
    ```
    --- PASS: TestAccAWSCloudWatchLogGroup_disappears (9.19s)
    --- PASS: TestAccAWSCloudWatchLogGroup_namePrefix (13.55s)
    --- PASS: TestAccAWSCloudWatchLogGroup_generatedName (13.99s)
    --- PASS: TestAccAWSCloudWatchLogGroup_basic (15.24s)
    --- PASS: TestAccAWSCloudWatchLogGroup_multiple (15.65s)
    --- PASS: TestAccAWSCloudWatchLogGroup_namePrefix_retention (21.29s)
    --- PASS: TestAccAWSCloudWatchLogGroup_retentionPolicy (24.99s)
    --- PASS: TestAccAWSCloudWatchLogGroup_kmsKey (29.00s)
    --- PASS: TestAccAWSCloudWatchLogGroup_tagging (35.60s)
    
    --- PASS: TestAccAWSAPIGatewayStage_accessLogSettings (225.36s)
    --- PASS: TestAccAWSAPIGatewayStage_accessLogSettings_kinesis (332.67s)
    
    --- PASS: TestAccAWSAPIGatewayV2Stage_AccessLogSettings (56.73s)
    
    --- PASS: TestAccAWSDataSyncTask_CloudWatchLogGroupARN (304.98s)
    
    --- PASS: TestAccAWSDirectoryServiceLogSubscription_basic (1764.25s)
    
    --- PASS: TestAccAWSElasticSearchDomain_LogPublishingOptions (688.17s)
    
    --- PASS: TestAccAWSFlowLog_LogDestinationType_CloudWatchLogs (26.43s)
    
    --- FAIL: TestAccAWSRoute53QueryLog_Basic (42.80s)
        TestAccAWSRoute53QueryLog_Basic: testing.go:684: Step 0 error: errors during apply:
    
            Error: Provider produced inconsistent final plan
    
            When expanding the plan for aws_cloudwatch_log_group.test to include new
            values learned so far during apply, provider "aws" produced an invalid new
            value for .name: was
            cty.StringVal("/aws/route53/testaccawsroute53querylog_basic-rsbvm.com"), but
            now cty.StringVal("/aws/route53/testaccawsroute53querylog_basic-rsbvm.com.").
    
            This is a bug in the provider, which should be reported in the provider's own
            issue tracker.
    
    --- PASS: TestAccAWSStorageGatewayGateway_CloudWatchLogs (220.06s)
    ```
    bflad committed Jul 30, 2020
    Configuration menu
    Copy the full SHA
    57048f5 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    649e609 View commit details
    Browse the repository at this point in the history