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

[Do Not Merge] Private DNS Zones + Explicit TypeSet #1057

Closed
wants to merge 9 commits into from

Conversation

rileykarson
Copy link
Member

Really weird behaviour in update that I suspect is a Terraform bug. Looking for a quick review of the downstream.


[all]

[terraform]

[terraform-beta]

[ansible]

[inspec]

@modular-magician
Copy link
Collaborator

I am (still) a robot that works on MagicModules PRs!

I just wanted to let you know that your changes (as of commit 2f85c1b) have been included in your existing downstream PRs.

@chrisst
Copy link
Contributor

chrisst commented Dec 21, 2018

@rileykarson Magician couldn't run due to the merge conflicts. When you solve them it should get kicked again and re-gen your downstreams.

@rileykarson
Copy link
Member Author

I only created this branch for a bug report to Terraform core, but out current plan is to merge #981 with the broken test, fixing it ourselves if 0.12 doesn't. Closing out.

@rileykarson rileykarson deleted the dns branch December 21, 2018 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants