fix: include provisioner tags when creating template version (#125)

ethanndickson · web-flow · commit eaa33dbec54d · 2024-11-01T01:22:24.000+11:00
This was unfortunately missed. There's not a great way to assert that
the provisioner tags are even being included in the request. They're
stored in the database for each template version, but we don't expose a
way to retrieve them and verify they were sent.

Treating as hotfix and merging.
diff --git a/internal/provider/template_resource.go b/internal/provider/template_resource.go
@@ -1087,13 +1087,18 @@ func newVersion(ctx context.Context, client *codersdk.Client, req newVersionRequ
 			Value: variable.Value.ValueString(),
 		})
 	}
+	provTags := make(map[string]string, len(req.Version.ProvisionerTags))
+	for _, provisionerTag := range req.Version.ProvisionerTags {
+		provTags[provisionerTag.Name.ValueString()] = provisionerTag.Value.ValueString()
+	}
 	tmplVerReq := codersdk.CreateTemplateVersionRequest{
 		Name:               req.Version.Name.ValueString(),
 		Message:            req.Version.Message.ValueString(),
 		StorageMethod:      codersdk.ProvisionerStorageMethodFile,
 		Provisioner:        codersdk.ProvisionerTypeTerraform,
 		FileID:             uploadResp.ID,
 		UserVariableValues: vars,
+		ProvisionerTags:    provTags,
 	}
 	if req.TemplateID != nil {
 		tmplVerReq.TemplateID = *req.TemplateID
