Change close order

Emyrk · Emyrk · commit fe40e4567135 · 2023-08-17T16:12:30.000-05:00
diff --git a/enterprise/coderd/coderd.go b/enterprise/coderd/coderd.go
@@ -404,10 +404,13 @@ type API struct {
 }
 
 func (api *API) Close() error {
-	api.cancel()
+	// Replica manager should be closed first. This is because the replica
+	// manager updates the replica's table in the database when it closes.
+	// This tells other Coderds that it is now offline.
 	if api.replicaManager != nil {
 		_ = api.replicaManager.Close()
 	}
+	api.cancel()
 	if api.derpMesh != nil {
 		_ = api.derpMesh.Close()
 	}

Original file line number	Diff line number	Diff line change
`@@ -404,10 +404,13 @@ type API struct {`
`404`	`404`	`}`
`405`	`405`
`406`	`406`	`func (api *API) Close() error {`
`407`		`- api.cancel()`
	`407`	`+ // Replica manager should be closed first. This is because the replica`
	`408`	`+ // manager updates the replica's table in the database when it closes.`
	`409`	`+ // This tells other Coderds that it is now offline.`
`408`	`410`	`if api.replicaManager != nil {`
`409`	`411`	`_ = api.replicaManager.Close()`
`410`	`412`	`}`
	`413`	`+ api.cancel()`
`411`	`414`	`if api.derpMesh != nil {`
`412`	`415`	`_ = api.derpMesh.Close()`
`413`	`416`	`}`