Move Postdeploy call within InitUnit to before the unit DB insertio…

…n. This avoids the case where interrupts or errors in postdeploy would leave a record of a unit in the database even though the actual container had been cleaned up.

platform/host_api.go

@@ -953,6 +953,11 @@ func (bh *BraveHost) InitUnit(backend Backend, unitParams *shared.Bravefile) (er
 		}
 	}
 
+	err = bh.Postdeploy(ctx, unitParams)
+	if err != nil {
+		return err
+	}
+
 	// Add unit into database
 
 	var braveUnit db.BraveUnit
@@ -985,13 +990,8 @@ func (bh *BraveHost) InitUnit(backend Backend, unitParams *shared.Bravefile) (er
 	braveUnit.Data = data
 
 	_, err = db.InsertUnitDB(database, braveUnit)
-	if err = shared.CollectErrors(err, ctx.Err()); err != nil {
-		return errors.New("failed to insert unit to database: " + err.Error())
-	}
-
-	err = bh.Postdeploy(ctx, unitParams)
 	if err != nil {
-		return err
+		return errors.New("failed to insert unit to database: " + err.Error())
 	}
 
 	return nil