[Fix] Adds check to ensure that the Grafana-Terraform-Provider header is set (#1631)

* adds check to ensure the Grafana-Terraform-Provider header is set to true

* lint

* fix

* revert changes to resource_slo.go

* add error check within tests to ensure that the terraform header is set

* check SLO deletion

* modify pointers

* lint

* lint

Author: elainevuong

internal/resources/slo/resource_slo.go

@@ -305,7 +305,6 @@ func resourceSloRead(ctx context.Context, d *schema.ResourceData, client *slo.AP
 
 	req := client.DefaultAPI.V1SloIdGet(ctx, sloID)
 	slo, _, err := req.Execute()
-
 	if err != nil {
 		return apiError("Unable to read SLO - API", err)
 	}

internal/resources/slo/resource_slo_test.go

@@ -112,11 +112,14 @@ func testAccSloCheckExists(rn string, slo *slo.SloV00Slo) resource.TestCheckFunc
 		client := testutils.Provider.Meta().(*common.Client).SLOClient
 		req := client.DefaultAPI.V1SloIdGet(context.Background(), rs.Primary.ID)
 		gotSlo, _, err := req.Execute()
-
 		if err != nil {
 			return fmt.Errorf("error getting SLO: %s", err)
 		}
 
+		if *gotSlo.ReadOnly.Provenance != "terraform" {
+			return fmt.Errorf("provenance header missing - verify within the Grafana Terraform Provider that the 'Grafana-Terraform-Provider' request header is set to 'true'")
+		}
+
 		*slo = *gotSlo
 
 		return nil
@@ -155,6 +158,7 @@ func testAccSloCheckDestroy(slo *slo.SloV00Slo) resource.TestCheckFunc {
 		if resp.StatusCode == 404 {
 			return nil
 		}
+
 		if gotSlo.ReadOnly.Status.Type == "deleting" {
 			return nil
 		}