Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 60e940f

Browse files
committed
chore: pass through boolean value
1 parent 40618ae commit 60e940f

File tree

4 files changed

+15
-10
lines changed

4 files changed

+15
-10
lines changed

coderd/provisionerdserver/provisionerdserver.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -775,7 +775,8 @@ func (s *server) acquireProtoJob(ctx context.Context, job database.ProvisionerJo
775775

776776
protoJob.Type = &proto.AcquiredJob_TemplateImport_{
777777
TemplateImport: &proto.AcquiredJob_TemplateImport{
778-
UserVariableValues: convertVariableValues(userVariableValues),
778+
UserVariableValues: convertVariableValues(userVariableValues),
779+
TerraformWorkspaceReuse: false, // TODO: Toggle based on experiment
779780
Metadata: &sdkproto.Metadata{
780781
CoderUrl: s.AccessURL.String(),
781782
// There is no owner for a template import, but we can assume

provisionerd/proto/provisionerd.proto

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ message AcquiredJob {
3131
message TemplateImport {
3232
provisioner.Metadata metadata = 1;
3333
repeated provisioner.VariableValue user_variable_values = 2;
34-
bool terraform_workspace_reuse = 3;
3534
}
3635
message TemplateDryRun {
3736
reserved 1;

provisionerd/proto/version.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,9 @@ import "github.com/coder/coder/v2/apiversion"
6060
//
6161
// API v1.12:
6262
// - Added new field `template_version_id` to `provisioner.Metadata`
63+
// - Added new field `terraform_workspace_reuse` to `provisioner.Job.TemplateImport`
64+
// - Added new field `terraform_workspace_reuse` to `provisioner.Job.WorkspaceBuild`
65+
// - Added fields `template_version_id`, `template_id`, and `terraform_workspace_reuse` to `provisioner.Config`
6366
const (
6467
CurrentMajor = 1
6568
CurrentMinor = 12

provisionerd/runner/runner.go

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -514,9 +514,10 @@ func (r *Runner) runTemplateImport(ctx context.Context) (*proto.CompletedJob, *p
514514
defer span.End()
515515

516516
failedJob := r.configure(&sdkproto.Config{
517-
TemplateSourceArchive: r.job.GetTemplateSourceArchive(),
518-
TemplateId: r.job.GetTemplateImport().Metadata.TemplateId,
519-
TemplateVersionId: r.job.GetTemplateImport().Metadata.TemplateVersionId,
517+
TemplateSourceArchive: r.job.GetTemplateSourceArchive(),
518+
TemplateId: r.job.GetTemplateImport().Metadata.TemplateId,
519+
TemplateVersionId: r.job.GetTemplateImport().Metadata.TemplateVersionId,
520+
TerraformWorkspaceReuse: r.job.GetTemplateImport().TerraformWorkspaceReuse,
520521
})
521522
if failedJob != nil {
522523
return nil, failedJob
@@ -1012,11 +1013,12 @@ func (r *Runner) runWorkspaceBuild(ctx context.Context) (*proto.CompletedJob, *p
10121013
}
10131014

10141015
failedJob := r.configure(&sdkproto.Config{
1015-
TemplateSourceArchive: r.job.GetTemplateSourceArchive(),
1016-
State: r.job.GetWorkspaceBuild().State,
1017-
ProvisionerLogLevel: r.job.GetWorkspaceBuild().LogLevel,
1018-
TemplateId: r.job.GetWorkspaceBuild().Metadata.TemplateId,
1019-
TemplateVersionId: r.job.GetWorkspaceBuild().Metadata.TemplateVersionId,
1016+
TemplateSourceArchive: r.job.GetTemplateSourceArchive(),
1017+
State: r.job.GetWorkspaceBuild().State,
1018+
ProvisionerLogLevel: r.job.GetWorkspaceBuild().LogLevel,
1019+
TemplateId: r.job.GetWorkspaceBuild().Metadata.TemplateId,
1020+
TemplateVersionId: r.job.GetWorkspaceBuild().Metadata.TemplateVersionId,
1021+
TerraformWorkspaceReuse: r.job.GetWorkspaceBuild().TerraformWorkspaceReuse,
10201022
})
10211023
if failedJob != nil {
10221024
return nil, failedJob

0 commit comments

Comments
 (0)