Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit9b3c7d7

Browse files
fix: don't attempt to insert empty terraform plans into the database (cherry-pick#17426) (#17486)
Co-authored-by: ケイラ <mckayla@hey.com>
1 parentb760f1d commit9b3c7d7

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

‎coderd/provisionerdserver/provisionerdserver.go

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1415,13 +1415,15 @@ func (s *server) CompleteJob(ctx context.Context, completed *proto.CompletedJob)
14151415
returnnil,xerrors.Errorf("update template version external auth providers: %w",err)
14161416
}
14171417

1418-
err=s.Database.InsertTemplateVersionTerraformValuesByJobID(ctx, database.InsertTemplateVersionTerraformValuesByJobIDParams{
1419-
JobID:jobID,
1420-
CachedPlan:jobType.TemplateImport.Plan,
1421-
UpdatedAt:now,
1422-
})
1423-
iferr!=nil {
1424-
returnnil,xerrors.Errorf("insert template version terraform data: %w",err)
1418+
iflen(jobType.TemplateImport.Plan)>0 {
1419+
err:=s.Database.InsertTemplateVersionTerraformValuesByJobID(ctx, database.InsertTemplateVersionTerraformValuesByJobIDParams{
1420+
JobID:jobID,
1421+
CachedPlan:jobType.TemplateImport.Plan,
1422+
UpdatedAt:now,
1423+
})
1424+
iferr!=nil {
1425+
returnnil,xerrors.Errorf("insert template version terraform data: %w",err)
1426+
}
14251427
}
14261428

14271429
err=s.Database.UpdateProvisionerJobWithCompleteByID(ctx, database.UpdateProvisionerJobWithCompleteByIDParams{

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp