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

aws_elasticache_subnet_group normalizes name to lowercase. #3120

Merged
merged 1 commit into from Sep 4, 2015
Merged

aws_elasticache_subnet_group normalizes name to lowercase. #3120

merged 1 commit into from Sep 4, 2015

Commits on Sep 3, 2015

  1. aws_elasticache_subnet_group normalizes name to lowercase.

    The Elasticache API accepts a mixed-case subnet name on create, but
    normalizes it to lowercase before storing it. When retrieving a subnet,
    the name is treated as case-sensitive, so the lowercase version must be
    used.
    
    Given that case within subnet names is not significant, the new StateFunc
    on the name attribute causes the state to reflect the lowercase version
    that the API uses, and changes in case alone will not show as a diff.
    
    Given that we must look up subnet names in lower case, we set the
    instance id to be a lowercase version of the user's provided name. This
    then allows a later Refresh call to succeed even if the user provided
    a mixed-case name.
    
    Previously users could work around this by just avoiding putting uppercase
    letters in the name, but that is often inconvenient if e.g. the name is
    being constructed from variables defined elsewhere that may already have
    uppercase letters present.
    apparentlymart committed Sep 3, 2015
    Copy the full SHA
    4ae3a17 View commit details
    Browse the repository at this point in the history