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

It seems like its impossible to import an aws_acm_certificate because subject_alternative_names see a change #13388

Closed
red8888 opened this issue May 19, 2020 · 2 comments

Comments

@red8888
Copy link

red8888 commented May 19, 2020

I imported an aws_acm_certificate

It has these subject name: subject_alternative_names = ["*.zzzz.com", "zzzz.com"]

I run plan and saw this:

      ~ subject_alternative_names = [ # forces replacement
          + "*.zzzz.com",
            "zzzz.com",
        ]

I then reordered the param like this: subject_alternative_names = ["zzzz.com", "*.zzzz.com"]
And then I got this error:

      ~ subject_alternative_names = [ # forces replacement
            "zzzz.com",
          + "*.zzzz.com",
        ]

????

I need to ignore it now for it to work (ignore_changes = ["subject_alternative_names"])

Why is this happening? After I apply with subject_alternative_names ignored is there a way to manually add it to state for now until this bug (gotta be a bug) is fixed?

@github-actions github-actions bot added the needs-triage Waiting for first response or review from a maintainer. label May 19, 2020
@bflad
Copy link
Contributor

bflad commented May 19, 2020

Hi @red8888 👋 Thank you for reporting this and sorry you ran into trouble here. I believe for imported certificates you can omit configuring subject_alternative_names since it would be determined by the certificate itself. Otherwise, please note that we have #8531 for tracking subject_alternative_names ordering issues with the aws_acm_certificate resource and #13053 to discuss remediation efforts since it will potentially require breaking changes in the next major version of the Terraform AWS Provider. Please follow those two issues for further updates on this. 👍

@bflad bflad closed this as completed May 19, 2020
@ghost
Copy link

ghost commented Jun 18, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Jun 18, 2020
@breathingdust breathingdust removed the needs-triage Waiting for first response or review from a maintainer. label Sep 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants