InvalidTemplate error on Azure VM creation—"zone" parameter is invalid about kitchen-azurerm HOT 6 CLOSED

This is causing significant issues for our testing pipelines because the feature is broken and everything is being pinned to zone 1 in whatever region you're using which makes it more difficult for azure to place and provision the VM.

>>>>>>     Failed to complete #create action: [#<Azure::Resources::Mgmt::V2020_06_01::Models::StatusMessage:0x00007fe03ce13858 @status="Failed", @error=#<Azure::Resources::Mgmt::V2020_06_01::Models::ErrorResponse:0x00007fe03ce13128 @code="ZonalAllocationFailed", @message="Allocation failed. We do not have sufficient capacity for the requested VM size in this zone. Read more about improving likelihood of allocation success at http://aka.ms/allocation-guidance">>] on default-windows-2019-core

I consider this a serious bug in two ways:

1. The actual feature doesn't work as intended. The template is completely disconnected from the config.
2. This reduces the probability of the VM creation process to succeed.

I'm moving forward with a straight up rollback of the feature and am looking for support from Progress Chef to help get this merged and released.

from kitchen-azurerm.

For what it's worth the implementation is missing some critical details. I think #228 was merged too soon. In Azure you can specify a zone or not, but it definitely shouldn't default to 1. It makes it harder for Azure to place the VM. This is especially true when certain SKUs are only available in certain zones.

The zone should default to nil and all of the added things should be excluded when nil.

from kitchen-azurerm.

@tas50 as I started to dig into this I realized it's going to be a much bigger effort than I originally anticipated (to establish solid unit tests). The actual "fix" I think is done, but I don't have much confidence. I would recommend yanking #228 and then re-releasing if you think this is impactful enough.

from kitchen-azurerm.

@tas50 funny enough on further inspection the way #228 was implemented the zone config param is completely disconnected from the template. In other words zone 1 was hard coded into the template.

from kitchen-azurerm.

@jasonwbarnett might as well roll forward if the config isn't doing anything at this point.

from kitchen-azurerm.

@jasonwbarnett might as well roll forward if the config isn't doing anything at this point.

@tas50 Well it's hard coding folks to zone 1. Not sure if you mean why not roll out #233 without tests or saying why not leave the zone feature as-is.

from kitchen-azurerm.