fix(coderd/database): revert addition of v prefix to provisioner_daemons.api_version (#11403)

johnstcn · web-flow · commit 4355894b2ba4 · 2024-01-04T11:47:31.000Z
"Reverts" #11385 by adding an inverse migration.
diff --git a/coderd/database/dump.sql b/coderd/database/dump.sql
diff --git a/coderd/database/migrations/000184_provisionerd_api_version_rm_prefix.down.sql b/coderd/database/migrations/000184_provisionerd_api_version_rm_prefix.down.sql
@@ -0,0 +1,5 @@
+ALTER TABLE ONLY provisioner_daemons
+	ALTER COLUMN api_version SET DEFAULT 'v1.0'::text;
+UPDATE provisioner_daemons
+	SET api_version = 'v1.0'
+	WHERE api_version = '1.0';
diff --git a/coderd/database/migrations/000184_provisionerd_api_version_rm_prefix.up.sql b/coderd/database/migrations/000184_provisionerd_api_version_rm_prefix.up.sql
@@ -0,0 +1,5 @@
+ALTER TABLE ONLY provisioner_daemons
+	ALTER COLUMN api_version SET DEFAULT '1.0'::text;
+UPDATE provisioner_daemons
+	SET api_version = '1.0'
+	WHERE api_version = 'v1.0';
diff --git a/provisionersdk/serve.go b/provisionersdk/serve.go
@@ -3,6 +3,7 @@ package provisionersdk
 import (
 	"context"
 	"errors"
+	"fmt"
 	"io"
 	"net"
 	"os"
@@ -21,10 +22,16 @@ import (
 )
 
 const (
+	CurrentMajor = 1
+	CurrentMinor = 0
+)
+
+var (
+	SupportedMajors = []int{1}
 	// APIVersionCurrent is the current provisionerd API version.
 	// Breaking changes to the provisionerd API **MUST** increment
-	// the major version below.
-	APIVersionCurrent = "v1.0"
+	// CurrentMajor above.
+	APIVersionCurrent = fmt.Sprintf("%d.%d", CurrentMajor, CurrentMinor)
 )
 
 // ServeOptions are configurations to serve a provisioner.
