Skip to content

feat: automatically stop workspaces based on failure_ttl #7989

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Jun 22, 2023
Merged

feat: automatically stop workspaces based on failure_ttl #7989

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
06399d0
chore: move autobuild/executor into autobuild
sreya Jun 13, 2023
315d869
feat: add support for stopping failed workspaces
sreya Jun 13, 2023
190c4bc
fix fake
sreya Jun 20, 2023
a8fa3f0
pr comments
sreya Jun 21, 2023
4fdaf19
move enterprise tests
sreya Jun 21, 2023
be1e37b
make gen
sreya Jun 21, 2023
a418d30
quotes
sreya Jun 21, 2023
7b6e5d1
ignore canceled jobs
sreya Jun 21, 2023
194e9eb
fmt
sreya Jun 21, 2023
4e137a1
Merge branch 'main' into jon/failedttl
sreya Jun 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'main' into jon/failedttl
  • Loading branch information
@sreya
sreya committed Jun 21, 2023
commit 4e137a1e98f0421c9307e712bc6e8192c7de0bd4
22 changes: 22 additions & 0 deletions coderd/coderdtest/coderdtest.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -318,6 +318,28 @@ func NewOptions(t testing.TB, options *Options) (func(http.Handler), context.Can
logger := slogtest.Make(t, nil).Leveled(slog.LevelDebug)
options.Logger = &logger
}
region := &tailcfg.DERPRegion{
EmbeddedRelay: true,
RegionID: int(options.DeploymentValues.DERP.Server.RegionID.Value()),
RegionCode: options.DeploymentValues.DERP.Server.RegionCode.String(),
RegionName: options.DeploymentValues.DERP.Server.RegionName.String(),
Nodes: []*tailcfg.DERPNode{{
Name: fmt.Sprintf("%db", options.DeploymentValues.DERP.Server.RegionID),
RegionID: int(options.DeploymentValues.DERP.Server.RegionID.Value()),
IPv4: "127.0.0.1",
DERPPort: derpPort,
// STUN port is added as a separate node by tailnet.NewDERPMap() if
// direct connections are enabled.
STUNPort: -1,
InsecureForTests: true,
ForceHTTP: options.TLSCertificates == nil,
}},
}
if !options.DeploymentValues.DERP.Server.Enable.Value() {
region = nil
}
derpMap, err := tailnet.NewDERPMap(ctx, region, []string{stunAddr.String()}, "", "", options.DeploymentValues.DERP.Config.BlockDirect.Value())
require.NoError(t, err)

return func(h http.Handler) {
mutex.Lock()
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.