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_route53_record import error processing #11603

Merged
merged 1 commit into from
Feb 1, 2017

Commits on Feb 1, 2017

  1. provider/aws: aws_route53_record import error processing

    Fixes: #11549
    
    When a user passes the wrong argument to a route53_record import, they
    got a crash. This was because we expected the ID to parse correctly. The
    crash looked like this:
    
    ```
    % terraform import aws_route53_record.import1 mike.westredd.com
    aws_route53_record.import1: Importing from ID "mike.westredd.com"...
    aws_route53_record.import1: Import complete!
      Imported aws_route53_record (ID: mike.westredd.com)
    aws_route53_record.import1: Refreshing state... (ID: mike.westredd.com)
    Error importing: 1 error(s) occurred:
    
    * aws_route53_record.import1: unexpected EOF
    panic: runtime error: index out of range
    ```
    
    Rather than throwing a panic to the user, we should present them with a more useful message that tells them what the error is:
    
    ```
    % terraform import aws_route53_record.import mike.westredd.com
    aws_route53_record.import: Importing from ID "mike.westredd.com"...
    aws_route53_record.import: Import complete!
      Imported aws_route53_record (ID: mike.westredd.com)
    aws_route53_record.import: Refreshing state... (ID: mike.westredd.com)
    Error importing: 1 error(s) occurred:
    
    * aws_route53_record.import: Error Importing aws_route_53 record. Please make sure the record ID is in the form ZONEID_RECORDNAME_TYPE (i.e. Z4KAPRWWNC7JR_dev_A
    ```
    
    At least they can work out what the problem is in this case
    stack72 committed Feb 1, 2017
    Configuration menu
    Copy the full SHA
    fb7601d View commit details
    Browse the repository at this point in the history