-
Notifications
You must be signed in to change notification settings - Fork 7
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
Reslang generates topic names that are not terraform compatible. #126
Comments
@davinchia when I run |
I was seeing errors during the plan as well. Here's the specific error: https://github.com/LiveRamp/api-specs/pull/318#issuecomment-700881252 |
Oh, maybe this is due to the older terraform version (terraform0.12.20 in the linked comment). I think I'd rather update atlantis.yaml with a higher terraform version to see if that fixes the problem, before changing reslang output (since that would nuke + recreate infrastructure). I'll report back! |
Perhaps. Didn't look too much into this - can help out if needed. Just lmk on slack! Thanks! |
@davinchia now that api-specs has terraform 0.12.29, would you mind checking if atlantis still runs into the same error on the auto-generated plan for topic names? If the problem persists, we'll either need to update to terraform 13, or change the Reslang code |
Reslang currently generated topic names with
.
in them.Although this is pubsub compatible, this is not terraform compatible and causes errors.
The text was updated successfully, but these errors were encountered: