Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource #24292

harshavmb · 2023-12-20T10:47:02Z

Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource.

Fixes #24282

…orchestrated_virtual_machine_scale_set

stephybun

Thanks for this @harshavmb.

Could you please take a look at the comments left in-line? Once that's done we can take another look through.

internal/services/compute/orchestrated_virtual_machine_scale_set_resource.go

website/docs/r/orchestrated_virtual_machine_scale_set.html.markdown

harshavmb · 2024-01-24T11:05:49Z

Hi @stephybun ,

I made changes as asked in the review. Could you please have another look on this?

stephybun

Unfortunately we have several test failures @harshavmb

Test ended in panic.
------- Stdout: -------
=== RUN   TestAccOrchestratedVirtualMachineScaleSet_AutomaticVMGuestPatchingHotpatchingEnabledWindows
=== PAUSE TestAccOrchestratedVirtualMachineScaleSet_AutomaticVMGuestPatchingHotpatchingEnabledWindows
=== CONT  TestAccOrchestratedVirtualMachineScaleSet_AutomaticVMGuestPatchingHotpatchingEnabledWindows
------- Stderr: -------
panic: Invalid address to set: []string{"additional_unattend_content"}
goroutine 550 [running]:
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*ResourceData).Set(0xc002de7600, {0x9628719, 0x1b}, {0x82f9040, 0xc005e6e900})
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/resource_data.go:233 +0x2b5
github.com/hashicorp/terraform-provider-azurerm/internal/services/compute.resourceOrchestratedVirtualMachineScaleSetRead(0xc002de7600, {0x8667da0?, 0xc00393e000?})
  /opt/teamcity-agent/work/3337027aeff310bf/internal/services/compute/orchestrated_virtual_machine_scale_set_resource.go:1319 +0x1aad
github.com/hashicorp/terraform-provider-azurerm/internal/services/compute.resourceOrchestratedVirtualMachineScaleSetCreate(0xc002de7600, {0x8667da0?, 0xc00393e000?})
  /opt/teamcity-agent/work/3337027aeff310bf/internal/services/compute/orchestrated_virtual_machine_scale_set_resource.go:695 +0x32c5
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).create(0xa0220f0?, {0xa0220f0?, 0xc001bc8c60?}, 0xd?, {0x8667da0?, 0xc00393e000?})
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/resource.go:766 +0x163
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc00461d0a0, {0xa0220f0, 0xc001bc8c60}, 0xc001fa88f0, 0xc002de6580, {0x8667da0, 0xc00393e000})
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/resource.go:909 +0xa7e
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc003ffbe48, {0xa0220f0?, 0xc001bc8bd0?}, 0xc001dbe000)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/grpc_provider.go:1060 +0xdbc
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ApplyResourceChange(0xc005ca1720, {0xa0220f0?, 0xc001b48d80?}, 0xc003a521c0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server/server.go:859 +0x56a
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ApplyResourceChange_Handler({0x9204680?, 0xc005ca1720}, {0xa0220f0, 0xc001b48d80}, 0xc003a52150, 0x0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:467 +0x169
google.golang.org/grpc.(*Server).processUnaryRPC(0xc002a40000, {0xa047940, 0xc003c8c1a0}, 0xc00372c120, 0xc0032d8660, 0xffaee78, 0x0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:1374 +0xde7
google.golang.org/grpc.(*Server).handleStream(0xc002a40000, {0xa047940, 0xc003c8c1a0}, 0xc00372c120, 0x0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:1751 +0x9e7
google.golang.org/grpc.(*Server).serveStreams.func1.1()
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:986 +0xbb
created by google.golang.org/grpc.(*Server).serveStreams.func1 in goroutine 334
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:997 +0x145

harshavmb · 2024-02-12T16:34:41Z

Hi @stephybun,

Unfortunately, I couldn't get this working! I am having problems with Autounattend.xml. I tried with several contents, all I get is below error which doesn't tell much. I doubt it's even supported by azure API as I see several comments here. I have had similar experience with flex orchestration APIs not having all feature support like uniform mode.

Error: waiting for creation of Orchestrated Virtual Machine Scale Set (Subscription: "b500a058-6396-45db-a15d-3f31913e84a5"
│ Resource Group Name: "factory-01"
│ Virtual Machine Scale Set Name: "acctestOVMSS-01"): Code="OSProvisioningInternalError" Message="OS Provisioning failed for VM 'acctestOVMSS-01_9cec32de' due to an internal error." Target="acctestOVMSS-01_9cec32de"

harshavmb · 2024-02-13T09:15:16Z

In the meantime, I got it working with below configuration..

additional_unattend_content {        
        setting = "FirstLogonCommands"
        content = "<FirstLogonCommands><SynchronousCommand><CommandLine>shutdown /r /t 0 /c \"initial reboot\"</CommandLine><Description>reboot</Description><Order>1</Order></SynchronousCommand></FirstLogonCommands>"        
}

harshavmb · 2024-02-13T09:16:23Z

Unfortunately we have several test failures @harshavmb

Test ended in panic.
------- Stdout: -------
=== RUN   TestAccOrchestratedVirtualMachineScaleSet_AutomaticVMGuestPatchingHotpatchingEnabledWindows
=== PAUSE TestAccOrchestratedVirtualMachineScaleSet_AutomaticVMGuestPatchingHotpatchingEnabledWindows
=== CONT  TestAccOrchestratedVirtualMachineScaleSet_AutomaticVMGuestPatchingHotpatchingEnabledWindows
------- Stderr: -------
panic: Invalid address to set: []string{"additional_unattend_content"}
goroutine 550 [running]:
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*ResourceData).Set(0xc002de7600, {0x9628719, 0x1b}, {0x82f9040, 0xc005e6e900})
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/resource_data.go:233 +0x2b5
github.com/hashicorp/terraform-provider-azurerm/internal/services/compute.resourceOrchestratedVirtualMachineScaleSetRead(0xc002de7600, {0x8667da0?, 0xc00393e000?})
  /opt/teamcity-agent/work/3337027aeff310bf/internal/services/compute/orchestrated_virtual_machine_scale_set_resource.go:1319 +0x1aad
github.com/hashicorp/terraform-provider-azurerm/internal/services/compute.resourceOrchestratedVirtualMachineScaleSetCreate(0xc002de7600, {0x8667da0?, 0xc00393e000?})
  /opt/teamcity-agent/work/3337027aeff310bf/internal/services/compute/orchestrated_virtual_machine_scale_set_resource.go:695 +0x32c5
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).create(0xa0220f0?, {0xa0220f0?, 0xc001bc8c60?}, 0xd?, {0x8667da0?, 0xc00393e000?})
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/resource.go:766 +0x163
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc00461d0a0, {0xa0220f0, 0xc001bc8c60}, 0xc001fa88f0, 0xc002de6580, {0x8667da0, 0xc00393e000})
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/resource.go:909 +0xa7e
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc003ffbe48, {0xa0220f0?, 0xc001bc8bd0?}, 0xc001dbe000)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema/grpc_provider.go:1060 +0xdbc
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ApplyResourceChange(0xc005ca1720, {0xa0220f0?, 0xc001b48d80?}, 0xc003a521c0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server/server.go:859 +0x56a
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ApplyResourceChange_Handler({0x9204680?, 0xc005ca1720}, {0xa0220f0, 0xc001b48d80}, 0xc003a52150, 0x0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:467 +0x169
google.golang.org/grpc.(*Server).processUnaryRPC(0xc002a40000, {0xa047940, 0xc003c8c1a0}, 0xc00372c120, 0xc0032d8660, 0xffaee78, 0x0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:1374 +0xde7
google.golang.org/grpc.(*Server).handleStream(0xc002a40000, {0xa047940, 0xc003c8c1a0}, 0xc00372c120, 0x0)
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:1751 +0x9e7
google.golang.org/grpc.(*Server).serveStreams.func1.1()
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:986 +0xbb
created by google.golang.org/grpc.(*Server).serveStreams.func1 in goroutine 334
  /opt/teamcity-agent/work/3337027aeff310bf/vendor/google.golang.org/grpc/server.go:997 +0x145

I fixed the test case & tests ran fine at my end. Could you please re-review?

katbyte

LGTM 🌋

<Actions> <action id="f410411e63aff4bb73a81c2aec1d373cf8a903e63b30dee2006b0030d8a94cc8"> <h3>Bump Terraform `azurerm` provider version</h3> <details id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24"> <summary>Update Terraform lock file</summary> <p>changes detected:
	"hashicorp/azurerm" updated from "3.91.0" to "3.92.0" in file ".terraform.lock.hcl"</p> <details> <summary>3.92.0</summary> <pre>Changelog retrieved from:
	https://github.com/hashicorp/terraform-provider-azurerm/releases/tag/v3.92.0
FEATURES:

* **New Data Source**: `azurerm_virtual_desktop_application_group` ([#24771](https://github.com/hashicorp/terraform-provider-azurerm/issues/24771))

ENHANCEMENTS:

* provider: support for the feature flag `postgresql_flexible_server.restart_server_on_configuration_value_change property` ([#23811](https://github.com/hashicorp/terraform-provider-azurerm/issues/23811))
* dependencies: updating to v0.20240214.1142753 of `github.com/hashicorp/go-azure-sdk` ([#24889](https://github.com/hashicorp/terraform-provider-azurerm/issues/24889))
* `automation`: updating to use the transport layer from `hashicorp/go-azure-sdk` rather than `Azure/go-autorest` ([#24858](https://github.com/hashicorp/terraform-provider-azurerm/issues/24858))
* `maintenance`: updating to use the transport layer from `hashicorp/go-azure-sdk` rather than `Azure/go-autorest` ([#24819](https://github.com/hashicorp/terraform-provider-azurerm/issues/24819))
* `containerapps`: updating to use the transport layer from `hashicorp/go-azure-sdk` rather than `Azure/go-autorest` ([#24862](https://github.com/hashicorp/terraform-provider-azurerm/issues/24862))
* `containerservices`: updating to use the transport layer from `hashicorp/go-azure-sdk` rather than `Azure/go-autorest` ([#24872](https://github.com/hashicorp/terraform-provider-azurerm/issues/24872))
* `timeseriesinsights`: updating to use the transport layer from `hashicorp/go-azure-sdk` rather than `Azure/go-autorest` ([#24889](https://github.com/hashicorp/terraform-provider-azurerm/issues/24889))
* `azurerm_container_app_environment`: support for the `infrastructure_resource_group_name` property ([#24361](https://github.com/hashicorp/terraform-provider-azurerm/issues/24361))
* `azurerm_cost_anomaly_alert` - support for the `subscription_id` property ([#24258](https://github.com/hashicorp/terraform-provider-azurerm/issues/24258))
* `azurerm_cosmosdb_account` - add default values for the `consistency_policy` code block ([#24830](https://github.com/hashicorp/terraform-provider-azurerm/issues/24830))
* `azurerm_dashboard_grafana` - support for the `smtp` block ([#24717](https://github.com/hashicorp/terraform-provider-azurerm/issues/24717))
* `azurerm_key_vault_certificates` - support for the `tags` property ([#24857](https://github.com/hashicorp/terraform-provider-azurerm/issues/24857))
* `azurerm_key_vault_secrets` - support for the `tags` property ([#24857](https://github.com/hashicorp/terraform-provider-azurerm/issues/24857))
* `azurerm_orchestrated_virtual_machine_scale_set` - support for the `additional_unattend_content` block ([#24292](https://github.com/hashicorp/terraform-provider-azurerm/issues/24292))
* `azurerm_virtual_desktop_host_pool` - support for the `vm_template` property ([#24369](https://github.com/hashicorp/terraform-provider-azurerm/issues/24369))

BUG FIXES:

* `azurerm_container_app_environment`: avoid unwanted changes when updating and using `log_analytics_workspace_id` ([#24303](https://github.com/hashicorp/terraform-provider-azurerm/issues/24303))
* `azurerm_cosmosdb_account` - fixed regression in the `backup` code block ([#24830](https://github.com/hashicorp/terraform-provider-azurerm/issues/24830))
* `azurerm_data_factory` - allow the `git_url` property to be blank/empty ([#24879](https://github.com/hashicorp/terraform-provider-azurerm/issues/24879))
* `azurerm_linux_web_app_slot` - the `worker_count` property now works correctly in the `site_config` block ([#24515](https://github.com/hashicorp/terraform-provider-azurerm/issues/24515))
* `azurerm_linux_web_app` - support `off` for the `file_system_level` property ([#24877](https://github.com/hashicorp/terraform-provider-azurerm/issues/24877))
* `azurerm_linux_web_app_slot` - support `off` for the `file_system_level` property ([#24877](https://github.com/hashicorp/terraform-provider-azurerm/issues/24877))
* `azurerm_private_endpoint` - fixing an issue where updating the Private Endpoint would remove any Application Security Group Association ([#24846](https://github.com/hashicorp/terraform-provider-azurerm/issues/24846))
* `azurerm_search_service` - fixed the update function to adjust for changed API behaviour ([#24837](https://github.com/hashicorp/terraform-provider-azurerm/issues/24837))
* `azurerm_search_service` - fixed the update function to adjust for changed API behaviour ([#24903](https://github.com/hashicorp/terraform-provider-azurerm/issues/24903))
* `azurerm_windows_web_app` - support `off` for the `file_system_level` property ([#24877](https://github.com/hashicorp/terraform-provider-azurerm/issues/24877))
* `azurerm_windows_web_app_slot` - support `off` for the `file_system_level` property ([#24877](https://github.com/hashicorp/terraform-provider-azurerm/issues/24877))


</pre> </details> </details> <a href="https://infra.ci.jenkins.io/job/updatecli/job/azure/job/main/3/">Jenkins pipeline link</a> </action> </Actions> --- <table> <tr> <td width="77"> <img src="https://www.updatecli.io/images/updatecli.png" alt="Updatecli logo" width="50" height="50"> </td> <td> <p> Created automatically by <a href="https://www.updatecli.io/">Updatecli</a> </p> <details><summary>Options:</summary> <br /> <p>Most of Updatecli configuration is done via <a href="https://www.updatecli.io/docs/prologue/quick-start/">its manifest(s)</a>.</p> <ul> <li>If you close this pull request, Updatecli will automatically reopen it, the next time it runs.</li> <li>If you close this pull request and delete the base branch, Updatecli will automatically recreate it, erasing all previous commits made.</li> </ul> <p> Feel free to report any issues at <a href="https://github.com/updatecli/updatecli/issues">github.com/updatecli/updatecli</a>.<br /> If you find this tool useful, do not hesitate to star <a href="https://github.com/updatecli/updatecli/stargazers">our GitHub repository</a> as a sign of appreciation, and/or to tell us directly on our <a href="https://matrix.to/#/#Updatecli_community:gitter.im">chat</a>! </p> </details> </td> </tr> </table> Co-authored-by: Jenkins Infra Bot (updatecli) <60776566+jenkins-infra-bot@users.noreply.github.com> Co-authored-by: Damien Duportal <damien.duportal@gmail.com>

…orchestra… (hashicorp#24292) * hashicorp#24282: Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set * hashicorp#24282: Changes as per review * hashicorp#24282: Fix errors and tests * hashicorp#24282: terrafmt

github-actions · 2024-04-22T02:09:39Z

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

hashicorp#24282: Support additional_unattend_content part of azurerm_…

165f4b3

…orchestrated_virtual_machine_scale_set

catriona-m added service/vmss Virtual Machine Scale Sets documentation enhancement labels Dec 20, 2023

rcskosir mentioned this pull request Jan 12, 2024

azurerm_orchestrated_virtual_machine_scale_set support for additional_unattend_content #24282

Closed

1 task

stephybun requested changes Jan 22, 2024

View reviewed changes

internal/services/compute/orchestrated_virtual_machine_scale_set_resource.go Outdated Show resolved Hide resolved

website/docs/r/orchestrated_virtual_machine_scale_set.html.markdown Outdated Show resolved Hide resolved

github-actions bot added the waiting-response label Jan 22, 2024

hashicorp#24282: Changes as per review

b9411d1

github-actions bot added size/S and removed waiting-response labels Jan 24, 2024

harshavmb requested a review from stephybun January 24, 2024 11:04

stephybun requested changes Jan 26, 2024

View reviewed changes

github-actions bot added the waiting-response label Jan 26, 2024

github-actions bot removed the waiting-response label Feb 12, 2024

hashicorp#24282: Fix errors and tests

44975fa

github-actions bot added size/L and removed size/S labels Feb 13, 2024

harshavmb requested a review from stephybun February 13, 2024 09:15

github-actions bot added the waiting-response label Feb 13, 2024

hashicorp#24282: terrafmt

d24771c

github-actions bot removed the waiting-response label Feb 13, 2024

katbyte approved these changes Feb 15, 2024

View reviewed changes

katbyte merged commit 38a7350 into hashicorp:main Feb 15, 2024
33 checks passed

katbyte changed the title ~~#24282: Support additional_unattend_content part of azurerm_orchestra…~~ Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource Feb 15, 2024

github-actions bot added this to the v3.92.0 milestone Feb 15, 2024

katbyte added a commit that referenced this pull request Feb 15, 2024

Update CHANGELOG.md #24292

450c334

harshavmb deleted the github-24282 branch February 20, 2024 13:44

rizkybiz pushed a commit to rizkybiz/terraform-provider-azurerm that referenced this pull request Feb 21, 2024

Update CHANGELOG.md hashicorp#24292

71acbe2

rizkybiz pushed a commit to rizkybiz/terraform-provider-azurerm that referenced this pull request Feb 29, 2024

Update CHANGELOG.md hashicorp#24292

87e4ba9

github-actions bot locked as resolved and limited conversation to collaborators Apr 22, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource #24292

Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource #24292

harshavmb commented Dec 20, 2023

stephybun left a comment

harshavmb commented Jan 24, 2024

stephybun left a comment

harshavmb commented Feb 12, 2024

harshavmb commented Feb 13, 2024

harshavmb commented Feb 13, 2024

katbyte left a comment

github-actions bot commented Apr 22, 2024

Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource #24292

Support additional_unattend_content part of azurerm_orchestrated_virtual_machine_scale_set resource #24292

Conversation

harshavmb commented Dec 20, 2023

stephybun left a comment

Choose a reason for hiding this comment

harshavmb commented Jan 24, 2024

stephybun left a comment

Choose a reason for hiding this comment

harshavmb commented Feb 12, 2024

harshavmb commented Feb 13, 2024

harshavmb commented Feb 13, 2024

katbyte left a comment

Choose a reason for hiding this comment

github-actions bot commented Apr 22, 2024