handle unique violation in wsbuilder

mafredri · mafredri · commit 7a703f0bcfa1 · 2025-01-08T13:00:31.000Z
diff --git a/coderd/wsbuilder/wsbuilder.go b/coderd/wsbuilder/wsbuilder.go
@@ -381,6 +381,10 @@ func (b *Builder) buildTx(authFunc func(action policy.Action, object rbac.Object
 			code := http.StatusInternalServerError
 			if rbac.IsUnauthorizedError(err) {
 				code = http.StatusForbidden
+			} else if database.IsUniqueViolation(err) {
+				// Concurrent builds may result in duplicate
+				// workspace_builds_workspace_id_build_number_key.
+				code = http.StatusConflict
 			}
 			return BuildError{code, "insert workspace build", err}
 		}

Original file line number	Diff line number	Diff line change
`@@ -381,6 +381,10 @@ func (b *Builder) buildTx(authFunc func(action policy.Action, object rbac.Object`
`381`	`381`	`code := http.StatusInternalServerError`
`382`	`382`	`if rbac.IsUnauthorizedError(err) {`
`383`	`383`	`code = http.StatusForbidden`
	`384`	`+ } else if database.IsUniqueViolation(err) {`
	`385`	`+ // Concurrent builds may result in duplicate`
	`386`	`+ // workspace_builds_workspace_id_build_number_key.`
	`387`	`+ code = http.StatusConflict`
`384`	`388`	`}`
`385`	`389`	`return BuildError{code, "insert workspace build", err}`
`386`	`390`	`}`