-
Notifications
You must be signed in to change notification settings - Fork 3
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
Import old Planetary resources into Terraform #126
base: main
Are you sure you want to change the base?
Conversation
bbd2e10
to
59469e2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey Gergely, this looks really clear and simple and I like the direction. I wasn't able to get terraform plan
to work for me locally, but I don't want to hold this up any more than I already have, so I'll message you about that offline.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw the comment that you don't have owner permissions on Cloudflare. I can get you that if needed. Is this code here as an example or would it actually create these records if I ran the terraform playbook?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would only create whatever doesnʼt exist.
Iʼm not sure about the owner access; although Terraform could manage the CF account altogether, iʼm not sure we want to go that deep in IaC. If it causes any trouble in the future, iʼll let you know.
adf5d01
to
456579c
Compare
dea55b5
to
54fc2e2
Compare
No description provided.