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

Merge with master #1

Conversation

mbfrahry
Copy link

@mbfrahry mbfrahry commented Dec 9, 2019

No description provided.

TestAccAzureRMKubernetesCluster_enableNodePublicIP - ForceNew for a PublicIP
TestAccAzureRMKubernetesCluster_windowsProfile - username is forcenew
Also adding a new test to confirm removing a node pool
tombuildsstuff and others added 27 commits December 6, 2019 13:24
r/nat_gateway: handling a crash when the `sku` block is nil
This was a fun one to track down.

Turns out due to the way that parallelization works, `init()` gets called after
the `resource.ParallelTest` has been called - which means that `testAccProviders`
could be an empty slice, which caused an unhelpful crash in the Plugin SDK.
refactor: making correlation id private
refactor: removing usages of deprecated packages
provider: ensuring the providers slice isn't a nil pointer
…torage_accounts

Bug: List storage accounts returns only first page
Custom Timeouts: adding compile-time checks
@brennerm brennerm merged commit 2fecf7a into brennerm:add-app-configuration-resource Dec 9, 2019
@mbfrahry mbfrahry deleted the brennerm-add-app-configuration-resource2 branch December 10, 2019 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.