-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
New Resource: azurerm_linux_virtual_machine_scale_set
#4518
Conversation
e364a92
to
5d1b633
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.
Aside from a couple minor comments LGTM 👍
5d1b633
to
58982d8
Compare
This resource supersedes the existing `azurerm_virtual_machine_scale_set` resource which will form part of the 2.0 release - as outlined in #2807. Since this won't be shipping immediately this is intentionally feature toggled and missing from the sidebar - but will be added in when this goes live.
58982d8
to
6a0e3c7
Compare
8cddd2c
to
d2af05f
Compare
Ignoring 5 test failures (which are Preview related, where we're still trying to get access, will be fixed by this PR - or are blocked on this issue) - the tests otherwise look good: Since this is feature-toggled off we're good to merge this for the moment, and can publish the information for how to enable this once all the VM/VMSS resources are available and the upstream API issue is fixed |
This has been released in version 1.36.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example: provider "azurerm" {
version = "~> 1.36.0"
}
# ... other configuration ... |
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! |
This PR adds support for one of the 2.0 resources:
azurerm_linux_virtual_machine_scale_set
which supersedes the existingazurerm_virtual_machine_scale_set
resource as outlined in #2807.The original
azurerm_virtual_machine_scale_set
resource was introduced in March 2016, when Azure Resource Manager was still in Beta.Whilst we've been adding features since then, unfortunately the way the Virtual Machine Scale Set API behaves today is very different from how it did previously - as such the
azurerm_virtual_machine_scale_set
resource didn't fully cover all use-cases.After spending a considerable amount of time on this - we ultimately came to the conclusion it'd be better to start afresh rather than trying to fix the existing
azurerm_virtual_machine_scale_set
resource. The Azure API behaves differently depending on which Operating System is used on the Virtual Machine / Virtual Machine Scale Set - for example for a Windows Virtual Machine (& Scale Set) aname
field can be up to 15 Characters long, whereas for a Linux Virtual Machine (& Scale Set) thename
field can be up to 63 characters long.There's some intentional design decisions as a part of this resource:
azurerm_virtual_machine_scale_set
resource.sku
(or select other fields, like thesource_image_reference
etc) is updated. This can be disabled by setting theterraform_should_roll_instances_when_required
field tofalse
... and a few things to note here:
azurerm_image
: add option to set the hyper_v_generation for the image #4453.As mentioned above - this is feature-toggled off by default - once the remaining Virtual Machine/Virtual Machine Scale Set resources become available we'll publicise this, but for the moment I've intentionally held off documenting how to opt into this functionality.
Upcoming PR's will add support for the other new Virtual Machine and Virtual Machine Scale Set resources documented in the 2.0 announcement.
Fixes #160
Fixes #292
Fixes #536
Fixes #809
Fixes #1616
Fixes #1620
Fixes #1669
Fixes #1739
Fixes #3020
Fixes #3773
Supersedes #2750