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

Commit2a47980

Browse files
committed
fix: create new template version when tfvars change
1 parentee9ac22 commit2a47980

File tree

2 files changed

+313
-71
lines changed

2 files changed

+313
-71
lines changed

‎internal/provider/template_resource.go‎

Lines changed: 65 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import (
66
"encoding/json"
77
"fmt"
88
"io"
9+
"slices"
910
"strings"
1011

1112
"cdr.dev/slog"
@@ -1053,7 +1054,7 @@ func markActive(ctx context.Context, client *codersdk.Client, templateID uuid.UU
10531054
ID:versionID,
10541055
})
10551056
iferr!=nil {
1056-
returnfmt.Errorf("Failed to update active template version: %s",err)
1057+
returnfmt.Errorf("failed to update active template version: %s",err)
10571058
}
10581059
tflog.Info(ctx,"marked template version as active")
10591060
returnnil
@@ -1231,8 +1232,9 @@ type LastVersionsByHash = map[string][]PreviousTemplateVersion
12311232
varLastVersionsKey="last_versions"
12321233

12331234
typePreviousTemplateVersionstruct {
1234-
ID uuid.UUID`json:"id"`
1235-
Namestring`json:"name"`
1235+
ID uuid.UUID`json:"id"`
1236+
Namestring`json:"name"`
1237+
TFVarsmap[string]string
12361238
}
12371239

12381240
typeprivateStateinterface {
@@ -1244,18 +1246,24 @@ func (v Versions) setPrivateState(ctx context.Context, ps privateState) (diags d
12441246
lv:=make(LastVersionsByHash)
12451247
for_,version:=rangev {
12461248
vbh,ok:=lv[version.DirectoryHash.ValueString()]
1249+
tfVars:=make(map[string]string,len(version.TerraformVariables))
1250+
for_,tfVar:=rangeversion.TerraformVariables {
1251+
tfVars[tfVar.Name.ValueString()]=tfVar.Value.ValueString()
1252+
}
12471253
// Store the IDs and names of all versions with the same directory hash,
12481254
// in the order they appear
12491255
ifok {
12501256
lv[version.DirectoryHash.ValueString()]=append(vbh,PreviousTemplateVersion{
1251-
ID:version.ID.ValueUUID(),
1252-
Name:version.Name.ValueString(),
1257+
ID:version.ID.ValueUUID(),
1258+
Name:version.Name.ValueString(),
1259+
TFVars:tfVars,
12531260
})
12541261
}else {
12551262
lv[version.DirectoryHash.ValueString()]= []PreviousTemplateVersion{
12561263
{
1257-
ID:version.ID.ValueUUID(),
1258-
Name:version.Name.ValueString(),
1264+
ID:version.ID.ValueUUID(),
1265+
Name:version.Name.ValueString(),
1266+
TFVars:tfVars,
12591267
},
12601268
}
12611269
}
@@ -1269,6 +1277,13 @@ func (v Versions) setPrivateState(ctx context.Context, ps privateState) (diags d
12691277
}
12701278

12711279
func (planVersionsVersions)reconcileVersionIDs(lvLastVersionsByHash,configVersionsVersions) {
1280+
// We remove versions that we've matched from `lv`, so make a copy for
1281+
// resolving tfvar changes at the end.
1282+
fullLv:=make(LastVersionsByHash)
1283+
fork,v:=rangelv {
1284+
fullLv[k]=slices.Clone(v)
1285+
}
1286+
12721287
fori:=rangeplanVersions {
12731288
prevList,ok:=lv[planVersions[i].DirectoryHash.ValueString()]
12741289
// If not in state, mark as known after apply since we'll create a new version.
@@ -1308,4 +1323,47 @@ func (planVersions Versions) reconcileVersionIDs(lv LastVersionsByHash, configVe
13081323
lv[planVersions[i].DirectoryHash.ValueString()]=prevList[1:]
13091324
}
13101325
}
1326+
1327+
// If only the Terraform variables have changed,
1328+
// we need to create a new version with the new variables.
1329+
fori:=rangeplanVersions {
1330+
if!planVersions[i].ID.IsUnknown() {
1331+
prevs,ok:=fullLv[planVersions[i].DirectoryHash.ValueString()]
1332+
if!ok {
1333+
continue
1334+
}
1335+
iftfVariablesChanged(prevs,&planVersions[i]) {
1336+
planVersions[i].ID=NewUUIDUnknown()
1337+
// We could always set the name to unknown here, to generate a
1338+
// random one (this is what the Web UI currently does when
1339+
// only updating tfvars).
1340+
// However, I think it'd be weird if the provider just started
1341+
// ignoring the name you set in the config, we'll instead
1342+
// require that users update the name if they update the tfvars.
1343+
ifconfigVersions[i].Name.IsNull() {
1344+
planVersions[i].Name=types.StringUnknown()
1345+
}
1346+
}
1347+
}
1348+
}
1349+
}
1350+
1351+
functfVariablesChanged(prevs []PreviousTemplateVersion,planned*TemplateVersion)bool {
1352+
for_,prev:=rangeprevs {
1353+
ifprev.ID==planned.ID.ValueUUID() {
1354+
// If the previous version has no TFVars, then it was created using
1355+
// an older provider version.
1356+
ifprev.TFVars==nil {
1357+
returntrue
1358+
}
1359+
for_,tfVar:=rangeplanned.TerraformVariables {
1360+
ifprev.TFVars[tfVar.Name.ValueString()]!=tfVar.Value.ValueString() {
1361+
returntrue
1362+
}
1363+
}
1364+
returnfalse
1365+
}
1366+
}
1367+
returntrue
1368+
13111369
}

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp