Adding Tag and Tag Assignment resources. Also Tag Data Source.

[stmcallister]

Adding pagerduty_tag and pagerduty_tag_assignment resources as well as pagerduty_tag data source. Here are the test results for those additions.

$ TF_ACC=1 go test -run "TestAccPagerDutyTagAssignment" ./pagerduty -v -timeout 120m 
=== RUN   TestAccPagerDutyTagAssignment_import
--- PASS: TestAccPagerDutyTagAssignment_import (8.74s)
=== RUN   TestAccPagerDutyTagAssignment_User
--- PASS: TestAccPagerDutyTagAssignment_User (8.58s)
=== RUN   TestAccPagerDutyTagAssignment_Team
--- PASS: TestAccPagerDutyTagAssignment_Team (6.43s)
=== RUN   TestAccPagerDutyTagAssignment_EP
--- PASS: TestAccPagerDutyTagAssignment_EP (9.90s)
PASS
ok      github.com/terraform-providers/terraform-provider-pagerduty/pagerduty   34.224s

$ TF_ACC=1 go test -run "TestAccDataSourcePagerDutyTag" ./pagerduty -v -timeout 120m
=== RUN   TestAccDataSourcePagerDutyTag_Basic
--- PASS: TestAccDataSourcePagerDutyTag_Basic (3.29s)
PASS
ok      github.com/terraform-providers/terraform-provider-pagerduty/pagerduty   3.861s

[heimweh]

Nice work @stmcallister 🎉 I left a few inline comments 🙂

[heimweh]

We could also return the result of Retry() here I think like we do in the Read() func 🙂

	return resource.Retry(10*time.Second, func() *resource.RetryError {
		if _, err := client.Tags.Assign(assignment.EntityType, assignment.EntityID, assignments); err != nil {
			if isErrCode(err, 400) || isErrCode(err, 429) {
				return resource.RetryableError(err)
			}

			return resource.NonRetryableError(err)
		} else {
			d.SetId("")
		}
		return nil
	})

[stmcallister]

@heimweh Isn't that what I'm doing on line 149?

[heimweh]

Oh yeah 🙈 but you could also just return the result directly I think, similar to this: https://github.com/PagerDuty/terraform-provider-pagerduty/pull/402/files#diff-8e5db2413fabc26d71cf3f5be3427fcade8bdd7d76ea78ba83f0f553886a24b0R103 🙂