fix

deansheather · deansheather · commit 4078fd112e02 · 2022-05-27T20:37:55.000Z
diff --git a/cli/create.go b/cli/create.go
@@ -173,6 +173,7 @@ func create() *cobra.Command {
 			// Run a plan with the given parameters to check correctness
 			after := time.Now()
 			planJob, err := client.CreateTemplateVersionPlan(cmd.Context(), templateVersion.ID, codersdk.CreateTemplateVersionPlanRequest{
+				WorkspaceName:   workspaceName,
 				ParameterValues: parameters,
 			})
 			if err != nil {
diff --git a/coderd/database/migrations/000099_provisioner_job_type_plan.down.sql b/coderd/database/migrations/000099_provisioner_job_type_plan.down.sql
diff --git a/coderd/database/migrations/000099_provisioner_job_type_plan.up.sql b/coderd/database/migrations/000099_provisioner_job_type_plan.up.sql
diff --git a/coderd/provisionerdaemons.go b/coderd/provisionerdaemons.go
@@ -113,6 +113,7 @@ type workspaceProvisionJob struct {
 // The input for a "template_version_plan" job.
 type templateVersionPlanJob struct {
 	TemplateVersionID uuid.UUID                 `json:"template_version_id"`
+	WorkspaceName     string                    `json:"workspace_name"`
 	ParameterValues   []database.ParameterValue `json:"parameter_values"`
 }
 
@@ -287,7 +288,8 @@ func (server *provisionerdServer) AcquireJob(ctx context.Context, _ *proto.Empty
 			TemplatePlan: &proto.AcquiredJob_TemplatePlan{
 				ParameterValues: protoParameters,
 				Metadata: &sdkproto.Provision_Metadata{
-					CoderUrl: server.AccessURL.String(),
+					CoderUrl:      server.AccessURL.String(),
+					WorkspaceName: input.WorkspaceName,
 				},
 			},
 		}
diff --git a/coderd/templateversions.go b/coderd/templateversions.go
@@ -213,6 +213,7 @@ func (api *API) createTemplateVersionPlan(rw http.ResponseWriter, r *http.Reques
 	// Marshal template version plan job with the parameters from the request.
 	input, err := json.Marshal(templateVersionPlanJob{
 		TemplateVersionID: templateVersion.ID,
+		WorkspaceName:     req.WorkspaceName,
 		ParameterValues:   parameterValues,
 	})
 	if err != nil {
diff --git a/coderd/templateversions_test.go b/coderd/templateversions_test.go
@@ -505,16 +505,11 @@ func TestTemplateVersionPlan(t *testing.T) {
 			defer close(logsDone)
 
 			logCount := 0
-			for {
-				select {
-				case _, ok := <-logs:
-					if !ok {
-						assert.GreaterOrEqual(t, logCount, 1, "unexpected log count")
-						return
-					}
-					logCount++
-				}
+			for range logs {
+				logCount++
 			}
+			assert.GreaterOrEqual(t, logCount, 1, "unexpected log count")
+			return
 		}()
 
 		// Wait for the job to complete
diff --git a/codersdk/templateversions.go b/codersdk/templateversions.go
@@ -119,6 +119,7 @@ func (c *Client) TemplateVersionLogsAfter(ctx context.Context, version uuid.UUID
 // CreateTemplateVersionPlanRequest defines the request parameters for
 // CreateTemplateVersionPlan.
 type CreateTemplateVersionPlanRequest struct {
+	WorkspaceName   string
 	ParameterValues []CreateParameterRequest
 }
 

Original file line number	Diff line number	Diff line change
`@@ -119,6 +119,7 @@ func (c *Client) TemplateVersionLogsAfter(ctx context.Context, version uuid.UUID`
`119`	`119`	`// CreateTemplateVersionPlanRequest defines the request parameters for`
`120`	`120`	`// CreateTemplateVersionPlan.`
`121`	`121`	`type CreateTemplateVersionPlanRequest struct {`
	`122`	`+ WorkspaceName string`
`122`	`123`	`ParameterValues []CreateParameterRequest`
`123`	`124`	`}`
`124`	`125`