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

Upgrading to v11 of the Azure SDK for Go #367

Merged
merged 52 commits into from
Oct 13, 2017
Merged

Upgrading to v11 of the Azure SDK for Go #367

merged 52 commits into from
Oct 13, 2017

Conversation

tombuildsstuff
Copy link
Contributor

@tombuildsstuff tombuildsstuff commented Sep 29, 2017

Upgrading to v11 of the Azure SDK for Go

Changes and Deprecation Notices:

  • azurerm_automation_account - the SKU Free has been replaced with Basic.
  • azurerm_container_registry - Azure has updated the SKU from Basic to Classic, with new Basic, Standard and Premium SKU's introduced.
  • azurerm_container_registry - the storage_account block is now storage_account_id and is only required for Classic SKU's
  • azurerm_key_vault - certificate_permissions, key_permissions and secret_permissions have all had the All option removed by Azure. Each permission now needs to be specified manually.
  • azurerm_servicebus_namespace - The capacity field can only be set for Premium SKU's
  • azurerm_servicebus_queue - The enable_batched_operations and support_ordering fields have been deprecated by Azure.
  • azurerm_servicebus_subscription - The dead_lettering_on_filter_evaluation_exceptions has been removed by Azure.
  • azurerm_servicebus_topic - The enable_filtering_messages_before_publishing field has been removed by Azure.

@tombuildsstuff
Copy link
Contributor Author

Taking a look into this - there's enough breaking changes that this is going to take some time - as such I'm going to park this PR for the moment.

@tombuildsstuff tombuildsstuff force-pushed the sdk11 branch 3 times, most recently from 37c49e6 to dee2a00 Compare October 4, 2017 15:23
@tombuildsstuff tombuildsstuff changed the title [WIP] Upgrading to v11 of the Azure SDK for Go Upgrading to v11 of the Azure SDK for Go Oct 4, 2017
@tombuildsstuff
Copy link
Contributor Author

I'm just kicking off the full test suite now - but I need to do a ton of checking of the state migrations next

@tombuildsstuff tombuildsstuff force-pushed the sdk11 branch 2 times, most recently from 8db584c to d8aa8ce Compare October 5, 2017 08:51
@tombuildsstuff
Copy link
Contributor Author

It appears the Traffic Manager SDK is broken as the API version used isn't rolled out in Azure at the moment - opened an issue on the SDK repository: Azure/azure-sdk-for-go#794

@tombuildsstuff
Copy link
Contributor Author

Container Registry's broken too: Azure/azure-sdk-for-go#795

@tombuildsstuff tombuildsstuff changed the title Upgrading to v11 of the Azure SDK for Go [ON HOLD] Upgrading to v11 of the Azure SDK for Go Oct 5, 2017
@tombuildsstuff
Copy link
Contributor Author

tombuildsstuff commented Oct 10, 2017

This PR will allow the following to be implemented: #272 #339

@tombuildsstuff tombuildsstuff changed the title [ON HOLD] Upgrading to v11 of the Azure SDK for Go Upgrading to v11 of the Azure SDK for Go Oct 12, 2017
Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a question about a comment and then the following code snippet. Other than that, this looks really good. Nice job Tom

return err
}

if namespace.Sku.Name != servicebus.Premium {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I might be reading this wrong but I just want to confirm based on your comment that premium has max size * 16

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this has been around for a while - so it hasn't changed AFAIK

@tombuildsstuff tombuildsstuff merged commit a43724f into master Oct 13, 2017
@tombuildsstuff tombuildsstuff deleted the sdk11 branch October 13, 2017 19:06
tombuildsstuff added a commit that referenced this pull request Oct 13, 2017
@vancluever
Copy link
Contributor

🎉 great job @tombuildsstuff!!!

@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 👉 [email protected]. 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

Successfully merging this pull request may close these issues.

5 participants