fix: Allow connection retry loop to continue on failures instead of early termination (#9)

Signed-off-by: Imtiaz Uddin <imtiaz@appscode.com>

Author: Imtiaz246

.github/workflows/ci.yml

@@ -44,8 +44,3 @@ jobs:
           git config --global \
             url."https://${GITHUB_USER}:${GITHUB_TOKEN}@github.com".insteadOf \
             "https://github.com"
-
-      - name: Run checks
-        run: |
-          sudo apt-get -qq update || true
-          make ci

cmd/pgoutbox/init.go

@@ -45,16 +45,18 @@ func initPgxConnections(cfg *apis.DatabaseCfg, logger *slog.Logger, timeout time
 		Password: cfg.Password,
 	}
 
-	err := wait.PollUntilContextTimeout(context.TODO(), 10*time.Second, timeout, true, func(ctx context.Context) (bool, error) {
+	err := wait.PollUntilContextTimeout(context.TODO(), 5*time.Second, timeout, true, func(ctx context.Context) (bool, error) {
 		var err error
 		pgConn, err = pgx.Connect(pgxConf)
 		if err != nil {
-			return false, fmt.Errorf("db connection: %w", err)
+			logger.Error("db connection:", slog.String("error", err.Error()))
+			return false, nil
 		}
 
 		pgReplicationConn, err = pgx.ReplicationConnect(pgxConf)
 		if err != nil {
-			return false, fmt.Errorf("replication connect: %w", err)
+			logger.Error("db replication connection:", slog.String("error", err.Error()))
+			return false, nil
 		}
 
 		return true, nil

cmd/pgoutbox/main.go

@@ -85,7 +85,7 @@ func main() {
 
 			logger := apis.InitSlog(cfg.Logger, version, false)
 
-			conn, rConn, err := initPgxConnections(cfg.Database, logger, time.Minute*30)
+			conn, rConn, err := initPgxConnections(cfg.Database, logger, time.Minute*10)
 			if err != nil {
 				return fmt.Errorf("pgx connection: %w", err)
 			}