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

provider/aws: aws_alb_target_group arn_suffix missing the targetgroup #9911

Merged
merged 1 commit into from
Nov 7, 2016

Commits on Nov 7, 2016

  1. provider/aws: aws_alb_target_group arn_suffix missing the targetgroup

    The work to add the arn_suffix in #9734 skipped adding the targetgroup/
    part of the arn
    
    This PR adds it
    
    //cc @firthh
    
    ```
    % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSALBTargetGroup_'
    ==> Checking that code complies with gofmt requirements...
    go generate $(go list ./... | grep -v /terraform/vendor/)
    2016/11/07 12:19:16 Generated command/internal_plugin_list.go
    TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSALBTargetGroup_ -timeout 120m
    === RUN   TestAccAWSALBTargetGroup_basic
    --- PASS: TestAccAWSALBTargetGroup_basic (47.23s)
    === RUN   TestAccAWSALBTargetGroup_changeNameForceNew
    --- PASS: TestAccAWSALBTargetGroup_changeNameForceNew (80.09s)
    === RUN   TestAccAWSALBTargetGroup_changeProtocolForceNew
    --- PASS: TestAccAWSALBTargetGroup_changeProtocolForceNew (87.45s)
    === RUN   TestAccAWSALBTargetGroup_changePortForceNew
    --- PASS: TestAccAWSALBTargetGroup_changePortForceNew (78.47s)
    === RUN   TestAccAWSALBTargetGroup_changeVpcForceNew
    --- PASS: TestAccAWSALBTargetGroup_changeVpcForceNew (73.53s)
    === RUN   TestAccAWSALBTargetGroup_tags
    --- PASS: TestAccAWSALBTargetGroup_tags (75.60s)
    === RUN   TestAccAWSALBTargetGroup_updateHealthCheck
    --- PASS: TestAccAWSALBTargetGroup_updateHealthCheck (76.40s)
    PASS
    ok  	github.com/hashicorp/terraform/builtin/providers/aws	518.777s
    ```
    stack72 committed Nov 7, 2016
    Configuration menu
    Copy the full SHA
    19248a5 View commit details
    Browse the repository at this point in the history