New Resource: azurerm_arc_machine_automanage_configuration_assignment

[liuwuliuyun]

Community Note

• Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
• Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

Swagger Link: https://github.com/Azure/azure-rest-api-specs/blob/34cb052b55b126a7447df9bc2d32e4185529400c/specification/automanage/resource-manager/Microsoft.Automanage/stable/2022-05-04/automanage.json#L1358-L1535

Docs: https://learn.microsoft.com/en-us/azure/automanage/automanage-arc

PR Checklist

• I have followed the guidelines in our Contributing Documentation.
• I have checked to ensure there aren't other open Pull Requests for the same update/change.
• I have checked if my changes close any open issues. If so please include appropriate closing keywords below.
• I have updated/added Documentation as required written in a helpful and kind way to assist users that may be unfamiliar with the resource / data source.
• I have used a meaningful PR title to help maintainers and other users understand this change and help prevent duplicate work.
  For example: “resource_name_here - description of change e.g. adding property new_property_name_here”

Testing

• My submission includes Test coverage as described in the Contribution Guide and the tests pass. (if this is not possible for any reason, please include details of why you did or could not add test coverage)

Testing evidence:

Note

For a successful test of this resource, locate "Automanage API Access" (appId: d828acde-4b48-47f5-a6e8-52460104a052) within your Entra ID tenant. Make sure it's granted access to the scope that includes the arc server.

=== RUN   TestAccArcMachineConfigurationAssignment_basic
=== PAUSE TestAccArcMachineConfigurationAssignment_basic
=== CONT  TestAccArcMachineConfigurationAssignment_basic
--- PASS: TestAccArcMachineConfigurationAssignment_basic (475.28s)
PASS


Process finished with the exit code 0

=== RUN   TestAccArcMachineConfigurationAssignment_requireImport
=== PAUSE TestAccArcMachineConfigurationAssignment_requireImport
=== CONT  TestAccArcMachineConfigurationAssignment_requireImport
--- PASS: TestAccArcMachineConfigurationAssignment_requireImport (488.56s)
PASS


Process finished with the exit code 0

Change Log

Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.

• New Resource: azurerm_arc_machine_automanage_configuration_assignment [New Resource: azurerm_arc_machine_automanage_configuration_assignment #26657]

This is a (please select all that apply):

• Bug Fix
• New Feature (ie adding a service, resource, or data source)
• Enhancement
• Breaking Change

[liuwuliuyun]

The deprecation warning for os.Getenv arises because we're simulating an Azure Linux VM as an Azure Hybrid Compute Machine. For the connection via Terraform, a service principal is essential; otherwise, an interactive login is needed, which would halt the testing process.

[liuwuliuyun]

Based on my findings, the method that employs a service principal (which includes a client_secret) lacks a corresponding technique when using managed identity. Therefore, it's necessary to use os.Getenv to make test running.

[liuwuliuyun]

In order to pass the PR checks, I manually removed test cases and mark test as skip. I will use the compiled version of AzureRM provider of this branch to test and show the test results.

Testing Script

Testing Logs in CMD

[katbyte]

LGTM 🌟

[github-actions]

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.