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

Upgrade to use API version 2016-04-01 #192

Closed
9 tasks done
tombuildsstuff opened this issue Jul 24, 2017 · 10 comments
Closed
9 tasks done

Upgrade to use API version 2016-04-01 #192

tombuildsstuff opened this issue Jul 24, 2017 · 10 comments
Assignees

Comments

@tombuildsstuff
Copy link
Contributor

tombuildsstuff commented Jul 24, 2017

Original Issue: coreos/tectonic-installer#1315 / Original PR: jen20/riviera#24

It appears that Azure's gradually removing support for the version of the DNS API that we're currently using via the Riviera SDK 2015-05-04-preview. Whilst we could switch over to using version 2016-04-01 it appears this requires some further refactoring where fields have changed.

The following PR's have been opened recently to switch these resources over to the Azure SDK for Go - which makes use of the updated API:

Given these PR's exist - it looks like it could be quicker to switch these over to the Azure SDK for Go

@justaugustus
Copy link
Contributor

@tombuildsstuff Thanks for your attention on this! Agreed that switching to Azure SDK for Go feels like the right route.

(Adding @alexsomesan, @s-urbaniak, @metral, @robszumski for visibility on the CoreOS side.)

@discordianfish
Copy link

discordianfish commented Jul 26, 2017

That's a huge issue. This effectively means everyone managing their azure setup with terraform are unable to do any (DNS) changes to their infrastructure right now. I don't even want to imagine what people going through right now who run prod stuff managed with terraform on azure and are effectively locked out of their infrastructure.
Has anyone contacted Azure to see if they could re-enable the APIs again?

@jdumars
Copy link

jdumars commented Jul 26, 2017

I'm looking into this now.

@tombuildsstuff
Copy link
Contributor Author

tombuildsstuff commented Jul 26, 2017

Hey @discordianfish

Has anyone contacted Azure to see if they could re-enable the APIs again?

That's a really good point - I've raised this internally at our end too and I'll post here when we've got an update (thanks @jdumars for requesting too)

That's a huge issue. This effectively means everyone managing their azure setup with terraform are unable to do any (DNS) changes to their infrastructure right now. I don't even want to imagine what people going through right now who run prod stuff managed with terraform on azure and are effectively locked out of their infrastructure.

Apologies for the inconvenience here - I completely agree this is a major issue and we’re looking into it right now. Unfortunately the removal of the API version appears to be rolling out gradually and we were unaware this version was due to be removed. Since we were notified of the issue we’ve been working through upgrading to the new API version - and that should be completed in the next couple of hours. We’ll also be double-checking the versions of the API’s we’re using as a result of this for any further deprecations.

Apologies once again for the inconvenience - as I mentioned above we should have a fix for this completed later today.

Thanks!

@jdumars
Copy link

jdumars commented Jul 26, 2017

Waiting to confirm on this, but I believe the issue has been fixed in Terraform master, and will be closed in the next update.

tombuildsstuff added a commit that referenced this issue Jul 26, 2017
@tombuildsstuff
Copy link
Contributor Author

tombuildsstuff commented Jul 26, 2017

Hey @justaugustus @discordianfish @jdumars

Waiting to confirm on this, but I believe the issue has been fixed in Terraform master, and will be closed in the next update.

Correct - I was just running the entire test suite before releasing :)

I've just released version 0.1.4 of the Terraform AzureRM Provider which switches over to using version 2016-04-01 of the Azure DNS API's - which is available in Terraform 0.10-rc1 (and can be used in Terraform 0.9 by updating the ~/.terraformrc file to reference the downloaded provider).

Would it be possible for you to take a look? :)

Thanks!

cc @alexsomesan, @s-urbaniak, @metral, @robszumski

@justaugustus
Copy link
Contributor

Thank you so much for chasing this down, @tombuildsstuff!
CoreOS team will give this a try and report back when we have an opportunity.

@alexsomesan
Copy link
Member

@justaugustus I will test this out tomorrow

@tombuildsstuff
Copy link
Contributor Author

Given the fix for this issue has been released - I'm going to close this issue for the moment. However please feel free to re-open it if there's still an issue here :)

@ghost
Copy link

ghost commented Apr 1, 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. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants