fix(setup): smaller transactions (#5742)

cmd/setup/10.go

@@ -2,8 +2,11 @@ package setup
 
 import (
 	"context"
+	"database/sql"
 	_ "embed"
+	"time"
 
+	"github.com/cockroachdb/cockroach-go/v2/crdb"
 	"github.com/zitadel/logging"
 
 	"github.com/zitadel/zitadel/internal/database"
@@ -15,35 +18,32 @@ var (
 )
 
 type CorrectCreationDate struct {
-	dbClient *database.DB
+	dbClient  *database.DB
+	FailAfter time.Duration
 }
 
 func (mig *CorrectCreationDate) Execute(ctx context.Context) (err error) {
-	tx, err := mig.dbClient.Begin()
+	ctx, cancel := context.WithTimeout(ctx, mig.FailAfter)
-	if err != nil {
+	defer cancel()
-		return err
+
-	}
-	if mig.dbClient.Type() == "cockroach" {
-		if _, err := tx.Exec("SET experimental_enable_temp_tables=on"); err != nil {
-			return err
-		}
-	}
-	defer func() {
-		if err != nil {
-			logging.OnError(tx.Rollback()).Debug("rollback failed")
-			return
-		}
-		err = tx.Commit()
-	}()
 	for {
-		res, err := tx.ExecContext(ctx, correctCreationDate10)
+		var affected int64
-		if err != nil {
+		err = crdb.ExecuteTx(ctx, mig.dbClient.DB, nil, func(tx *sql.Tx) error {
-			return err
+			if mig.dbClient.Type() == "cockroach" {
-		}
+				if _, err := tx.Exec("SET experimental_enable_temp_tables=on"); err != nil {
-		affected, _ := res.RowsAffected()
+					return err
-		logging.WithFields("count", affected).Info("creation dates changed")
+				}
-		if affected == 0 {
+			}
+			res, err := tx.ExecContext(ctx, correctCreationDate10)
+			if err != nil {
+				return err
+			}
+			affected, _ = res.RowsAffected()
+			logging.WithFields("count", affected).Info("creation dates changed")
 			return nil
+		})
+		if affected == 0 || err != nil {
+			return err
 		}
 	}
 }

cmd/setup/10.sql

@@ -5,8 +5,6 @@ CREATE temporary TABLE IF NOT EXISTS wrong_events (
     , next_cd TIMESTAMPTZ
 );
 
-TRUNCATE wrong_events;
-
 INSERT INTO wrong_events (
     SELECT * FROM (
         SELECT

cmd/setup/config.go

@@ -56,16 +56,16 @@ func MustNewConfig(v *viper.Viper) *Config {
 }
 
 type Steps struct {
-	s1ProjectionTable         *ProjectionTable
+	s1ProjectionTable    *ProjectionTable
-	s2AssetsTable             *AssetTable
+	s2AssetsTable        *AssetTable
-	FirstInstance             *FirstInstance
+	FirstInstance        *FirstInstance
-	s4EventstoreIndexes       *EventstoreIndexesNew
+	s4EventstoreIndexes  *EventstoreIndexesNew
-	s5LastFailed              *LastFailed
+	s5LastFailed         *LastFailed
-	s6OwnerRemoveColumns      *OwnerRemoveColumns
+	s6OwnerRemoveColumns *OwnerRemoveColumns
-	s7LogstoreTables          *LogstoreTables
+	s7LogstoreTables     *LogstoreTables
-	s8AuthTokens              *AuthTokenIndexes
+	s8AuthTokens         *AuthTokenIndexes
-	s9EventstoreIndexes2      *EventstoreIndexesNew
+	s9EventstoreIndexes2 *EventstoreIndexesNew
-	s10EventstoreCreationDate *CorrectCreationDate
+	CorrectCreationDate  *CorrectCreationDate
 }
 
 type encryptionKeyConfig struct {

cmd/setup/setup.go

@@ -88,7 +88,7 @@ func Setup(config *Config, steps *Steps, masterKey string) {
 	steps.s7LogstoreTables = &LogstoreTables{dbClient: dbClient.DB, username: config.Database.Username(), dbType: config.Database.Type()}
 	steps.s8AuthTokens = &AuthTokenIndexes{dbClient: dbClient}
 	steps.s9EventstoreIndexes2 = New09(dbClient)
-	steps.s10EventstoreCreationDate = &CorrectCreationDate{dbClient: dbClient}
+	steps.CorrectCreationDate.dbClient = dbClient
 
 	err = projection.Create(ctx, dbClient, eventstoreClient, config.Projections, nil, nil)
 	logging.OnError(err).Fatal("unable to start projections")
@@ -124,7 +124,7 @@ func Setup(config *Config, steps *Steps, masterKey string) {
 	logging.OnError(err).Fatal("unable to migrate step 8")
 	err = migration.Migrate(ctx, eventstoreClient, steps.s9EventstoreIndexes2)
 	logging.OnError(err).Fatal("unable to migrate step 9")
-	err = migration.Migrate(ctx, eventstoreClient, steps.s10EventstoreCreationDate)
+	err = migration.Migrate(ctx, eventstoreClient, steps.CorrectCreationDate)
 	logging.OnError(err).Fatal("unable to migrate step 10")
 
 	for _, repeatableStep := range repeatableSteps {

cmd/setup/steps.yaml

@@ -30,3 +30,5 @@ FirstInstance:
       MachineKey:
         ExpirationDate:
         Type:
+CorrectCreationDate:
+  FailAfter: 5m