coder · mtojek · Nov 28, 2023 · Nov 24, 2023 · Nov 24, 2023 · Nov 27, 2023
diff --git a/coderd/apidoc/docs.go b/coderd/apidoc/docs.go
diff --git a/coderd/apidoc/swagger.json b/coderd/apidoc/swagger.json
diff --git a/coderd/audit/diff.go b/coderd/audit/diff.go
@@ -18,7 +18,8 @@ type Auditable interface {
 		database.AuditableGroup |
 		database.License |
 		database.WorkspaceProxy |
-		database.AuditOAuthConvertState
+		database.AuditOAuthConvertState |
+		database.HealthSettings
 }
 
 // Map is a map of changed fields in an audited resource. It maps field names to

diff --git a/coderd/audit/request.go b/coderd/audit/request.go
@@ -93,6 +93,8 @@ func ResourceTarget[T Auditable](tgt T) string {
 		return typed.Name
 	case database.AuditOAuthConvertState:
 		return string(typed.ToLoginType)
+	case database.HealthSettings:
+		return "" // no target?
 	default:
 		panic(fmt.Sprintf("unknown resource %T", tgt))
 	}
@@ -123,6 +125,9 @@ func ResourceID[T Auditable](tgt T) uuid.UUID {
 	case database.AuditOAuthConvertState:
 		// The merge state is for the given user
 		return typed.UserID
+	case database.HealthSettings:
+		// Artificial ID for auditing purposes
+		return typed.ID
 	default:
 		panic(fmt.Sprintf("unknown resource %T", tgt))
 	}
@@ -152,6 +157,8 @@ func ResourceType[T Auditable](tgt T) database.ResourceType {
 		return database.ResourceTypeWorkspaceProxy
 	case database.AuditOAuthConvertState:
 		return database.ResourceTypeConvertLogin
+	case database.HealthSettings:
+		return database.ResourceTypeHealthSettings
 	default:
 		panic(fmt.Sprintf("unknown resource %T", typed))
 	}

diff --git a/coderd/coderd.go b/coderd/coderd.go
@@ -970,7 +970,13 @@ func New(options *Options) *API {
 
 			r.Get("/coordinator", api.debugCoordinator)
 			r.Get("/tailnet", api.debugTailnet)
-			r.Get("/health", api.debugDeploymentHealth)
+			r.Route("/health", func(r chi.Router) {
+				r.Get("/", api.debugDeploymentHealth)
+				r.Route("/settings", func(r chi.Router) {
+					r.Get("/", api.deploymentHealthSettings)
+					r.Put("/", api.putDeploymentHealthSettings)
+				})
+			})
 			r.Get("/ws", (&healthcheck.WebsocketEchoServer{}).ServeHTTP)
 			r.Route("/{user}", func(r chi.Router) {
 				r.Use(httpmw.ExtractUserParam(options.Database))

diff --git a/coderd/database/dump.sql b/coderd/database/dump.sql
diff --git a/coderd/database/migrations/000172_health_settings_audit.down.sql b/coderd/database/migrations/000172_health_settings_audit.down.sql
@@ -0,0 +1 @@
+-- Nothing to do
diff --git a/coderd/database/migrations/000172_health_settings_audit.up.sql b/coderd/database/migrations/000172_health_settings_audit.up.sql
@@ -0,0 +1,2 @@
+-- This has to be outside a transaction
+ALTER TYPE resource_type ADD VALUE IF NOT EXISTS 'health_settings';
diff --git a/coderd/database/models.go b/coderd/database/models.go
diff --git a/coderd/database/types.go b/coderd/database/types.go
@@ -23,6 +23,11 @@ type AuditOAuthConvertState struct {
 	UserID      uuid.UUID `db:"user_id" json:"user_id"`
 }
 
+type HealthSettings struct {
+	ID                    uuid.UUID `db:"id" json:"id"`
+	DismissedHealthchecks []string  `db:"dismissed_healthchecks" json:"dismissed_healthchecks"`
+}
+
 type Actions []rbac.Action
 
 func (a *Actions) Scan(src interface{}) error {
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		-- This has to be outside a transaction
		ALTER TYPE resource_type ADD VALUE IF NOT EXISTS 'health_settings';