ENG-11253: create partial unique index on template

tinkerbell · Dec 10, 2020 · 419d7ff · 419d7ff
1 parent 9c52cca
commit 419d7ff
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 7 deletions.
diff --git a/db/migration/202012091055-add-partial-unique-constraint-on-template.go b/db/migration/202012091055-add-partial-unique-constraint-on-template.go
@@ -0,0 +1,12 @@
+package migration
+
+import migrate "github.com/rubenv/sql-migrate"
+
+func Get202012091055() *migrate.Migration {
+	return &migrate.Migration{
+		Id: "202012091055-add-partial-unique-constraint-on-template",
+		Up: []string{`
+                CREATE UNIQUE INDEX uidx_template_name ON template (name) WHERE deleted_at IS NULL;
+                `},
+	}
+}
diff --git a/db/migration/migration.go b/db/migration/migration.go
@@ -9,6 +9,7 @@ func GetMigrations() *migrate.MemoryMigrationSource {
 			Get202010071530(),
 			Get202010221010(),
 			Get202012041103(),
+			Get202012091055(),
 		},
 	}
 }
diff --git a/db/template.go b/db/template.go
@@ -19,13 +19,6 @@ func (d TinkDB) CreateTemplate(ctx context.Context, name string, data string, id
 		return err
 	}
 
-	fields := map[string]string{
-		"name": name,
-	}
-	_, _, _, err = d.GetTemplate(ctx, fields, false)
-	if err != sql.ErrNoRows {
-		return errors.New("Template with name '" + name + "' already exist")
-	}
 	tx, err := d.instance.BeginTx(ctx, &sql.TxOptions{Isolation: sql.LevelSerializable})
 	if err != nil {
 		return errors.Wrap(err, "BEGIN transaction")