chore: remove extra stop_modules from provisionerd proto (#21706)

Was a duplicate of start_modules

Closes https://github.com/coder/coder/issues/21206
This commit is contained in:
Steven Masley
2026-01-28 09:25:47 -06:00
committed by GitHub
parent d06b21df45
commit e13f2a9869
7 changed files with 172 additions and 187 deletions
@@ -1652,7 +1652,6 @@ func (s *server) completeTemplateImportJob(ctx context.Context, job database.Pro
// Process modules
for transition, modules := range map[database.WorkspaceTransition][]*sdkproto.Module{
database.WorkspaceTransitionStart: jobType.TemplateImport.StartModules,
database.WorkspaceTransitionStop: jobType.TemplateImport.StopModules,
} {
for _, module := range modules {
s.Logger.Info(ctx, "inserting template import job module",
@@ -2309,19 +2309,17 @@ func TestCompleteJob(t *testing.T) {
Version: "1.0.0",
Source: "github.com/example/example",
},
},
StopResources: []*sdkproto.Resource{{
Name: "something2",
Type: "aws_instance",
ModulePath: "module.test2",
}},
StopModules: []*sdkproto.Module{
{
Key: "test2",
Version: "2.0.0",
Source: "github.com/example2/example",
},
},
StopResources: []*sdkproto.Resource{{
Name: "something2",
Type: "aws_instance",
ModulePath: "module.test2",
}},
Plan: []byte("{}"),
},
},
@@ -2358,7 +2356,7 @@ func TestCompleteJob(t *testing.T) {
Key: "test2",
Version: "2.0.0",
Source: "github.com/example2/example",
Transition: database.WorkspaceTransitionStop,
Transition: database.WorkspaceTransitionStart,
}},
},
{