Add support for 'auto_update'

[paulkirkwood]

All Submissions:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My code follows the code style of this project.
• I ran lint checks locally prior to submission.
• Have you checked to ensure there aren't other open PRs for the same update/change?

What about the current behavior has changed?

Added behaviour to allow the setting of the 'auto_update' attribute within a 'azuredevops_agent_pool' resource.

Issue Number: #688

Does this introduce a change to go.mod, go.sum or vendor/?

• Yes
• No

Does this introduce a breaking change?

• Yes
• No

[paulkirkwood]

@microsoft-github-policy-service agree company="Vodafone"

[xuzhang3]

@paulkirkwood A few change request others look to me.

[xuzhang3]

can we call the update after the create if the auto_update set to false? So that user can initially set it to true or false.

Example code:

func resourceAzureAgentPoolCreate(d *schema.ResourceData, m interface{}) error {
	clients := m.(*client.AggregatedClient)
	agentPool, err := expandAgentPool(d, true)
	if err != nil {
		return fmt.Errorf("Error converting terraform data model to AzDO agentPool reference: %+v", err)
	}

	createdAgentPool, err := createAzureAgentPool(clients, agentPool)
	if err != nil {
		return fmt.Errorf("Error creating agent pool in Azure DevOps: %+v", err)
	}

	if agentPool.AutoUpdate != nil && !*agentPool.AutoUpdate {
		agentPool.Id = createdAgentPool.Id
		createdAgentPool, err = azureAgentPoolUpdate(clients, agentPool)
		if err != nil {
			return fmt.Errorf("Error updating agent pool in Azure DevOps: %+v", err)
		}
	}

	flattenAzureAgentPool(d, createdAgentPool)

	return resourceAzureAgentPoolRead(d, m)
}

[xuzhang3]

=== RUN   TestAccAgentPool_CreateAndUpdate
=== PAUSE TestAccAgentPool_CreateAndUpdate
=== CONT  TestAccAgentPool_CreateAndUpdate
--- PASS: TestAccAgentPool_CreateAndUpdate (29.91s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        36.392s
=== RUN   TestAccAgentPool_DataSource
=== PAUSE TestAccAgentPool_DataSource
=== CONT  TestAccAgentPool_DataSource
--- PASS: TestAccAgentPool_DataSource (23.20s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        23.833s
=== RUN   TestAccAgentQueue_DataSource
=== PAUSE TestAccAgentQueue_DataSource
=== CONT  TestAccAgentQueue_DataSource
--- PASS: TestAccAgentQueue_DataSource (40.18s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        40.786s
=== RUN   TestAccAgentPools_DataSource
=== PAUSE TestAccAgentPools_DataSource
=== CONT  TestAccAgentPools_DataSource
--- PASS: TestAccAgentPools_DataSource (21.85s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        22.378s