B1: EF Core migrations baseline validated against real dev DB
- Design-time AppDbContextFactory now builds its connection string via MySqlConnectionStringBuilder from PREDICTALYTICS_DB_* env vars instead of a hardcoded local default, so no secret needs to live in source/config to run migrations against any target database. - InitialBaseline migration applied end-to-end against a fresh dev MySQL DB and confirmed via `dotnet ef migrations list`.
This commit is contained in:
+4
-2
@@ -74,8 +74,10 @@ Beim Review ist aufgefallen, dass `TraderAnalyticsWorker.CalculatePnL` **jeden T
|
|||||||
- [x] Aktuellen Ist-Stand des Schemas exakt erfassen (inkl. aller manuellen `ALTER TABLE`-Patches aus `DependencyInjection.cs`) — abgeglichen, Entities/`OnModelCreating` deckten den Patch-Stand bereits vollständig ab
|
- [x] Aktuellen Ist-Stand des Schemas exakt erfassen (inkl. aller manuellen `ALTER TABLE`-Patches aus `DependencyInjection.cs`) — abgeglichen, Entities/`OnModelCreating` deckten den Patch-Stand bereits vollständig ab
|
||||||
- [x] Erste **Baseline-Migration** erzeugt (`20260701102311_InitialBaseline`, in `src/Predictalytics.Infrastructure/Migrations/`), entspricht exakt dem aktuellen Schema
|
- [x] Erste **Baseline-Migration** erzeugt (`20260701102311_InitialBaseline`, in `src/Predictalytics.Infrastructure/Migrations/`), entspricht exakt dem aktuellen Schema
|
||||||
- [x] `EnsureCreatedAsync` + handgeschriebene `ExecuteIfColumnMissing`/`ALTER`-Helfer aus `DependencyInjection.cs` entfernt, durch `db.Database.MigrateAsync()` ersetzt (Platform-Seed bleibt als `INSERT IGNORE`)
|
- [x] `EnsureCreatedAsync` + handgeschriebene `ExecuteIfColumnMissing`/`ALTER`-Helfer aus `DependencyInjection.cs` entfernt, durch `db.Database.MigrateAsync()` ersetzt (Platform-Seed bleibt als `INSERT IGNORE`)
|
||||||
- [ ] **Offen (manueller Schritt durch Nutzer):** Live-DB per SQL als "bereits migriert" markieren (`__EFMigrationsHistory`-Tabelle + Insert für `20260701102311_InitialBaseline`), da die Tabellen dort schon existieren und nicht per `CreateTable` neu angelegt werden dürfen. SQL wurde im Chat bereitgestellt, Ausführung liegt beim Nutzer (bewusst nicht automatisch gegen die produktive Remote-DB ausgeführt)
|
- [x] Migration end-to-end gegen eine echte, frische DEV-Datenbank getestet (`lqf7.your-database.de`/`bergisnu_db0`, vom Nutzer bereitgestellt) — `dotnet ef database update` lief fehlerfrei durch, `dotnet ef migrations list` bestätigt sie als angewendet
|
||||||
- [ ] Alle künftigen Schemaänderungen aus Phase 1 (z.B. `TraderPosition`, `MarketOutcomePriceSnapshot`, `TraderCopytradingProfile`) als reguläre Migrationen anlegen — **dieser Task sollte vor A1 abgeschlossen sein**, damit wir dort nicht wieder manuell patchen
|
- [x] `AppDbContextFactory` (Design-Time-Factory für `dotnet ef`) liest Zielverbindung jetzt aus Env-Vars (`PREDICTALYTICS_DB_SERVER/_NAME/_USER/_PASSWORD`, via `MySqlConnectionStringBuilder` statt roher String-Konkatenation, da Passwörter Sonderzeichen wie `;`/`{}` enthalten können) statt fest codiert — kein Secret mehr im Repo nötig, um Migrationen gegen eine beliebige DB zu fahren
|
||||||
|
- [x] ~~Live-Produktions-DB manuell als "bereits migriert" markieren~~ — vorerst zurückgestellt: Da wir laut Abschnitt D ohnehin einen kompletten Neustart der Produktions-DB planen (Altdaten sind jederzeit nachladbar, siehe C0), richten wir die neue Produktions-DB am Ende genauso ein wie die DEV-DB (leer anlegen + `dotnet ef database update`) — der fummelige Stamp-Schritt auf die bestehende Live-DB entfällt dann komplett. Nur falls wir uns doch gegen den Neustart entscheiden, müsste dieser Schritt nachgeholt werden
|
||||||
|
- [ ] Alle künftigen Schemaänderungen aus Phase 1 (z.B. `TraderPosition`, `MarketOutcomePriceSnapshot`, `TraderCopytradingProfile`) als reguläre Migrationen anlegen — direkt gegen die DEV-DB entwickeln/testen, da wir dort laut Nutzer frei experimentieren dürfen
|
||||||
|
|
||||||
### B2. Secrets-Management (Release-Vorbereitung, kein akuter Risikofall)
|
### B2. Secrets-Management (Release-Vorbereitung, kein akuter Risikofall)
|
||||||
> Software läuft aktuell nur lokal, kein Fremdzugriff — Passwort bleibt vorerst wie es ist, keine Rotation nötig. Dieser Punkt ist reine **Vorbereitung**, damit das Projekt bei Bedarf später auch ohne die aktuellen Secrets veröffentlicht/geteilt werden könnte, ohne den Code nochmal anfassen zu müssen. Dadurch niedrigere Priorität als vorher angenommen — kann später in der Reihenfolge stehen.
|
> Software läuft aktuell nur lokal, kein Fremdzugriff — Passwort bleibt vorerst wie es ist, keine Rotation nötig. Dieser Punkt ist reine **Vorbereitung**, damit das Projekt bei Bedarf später auch ohne die aktuellen Secrets veröffentlicht/geteilt werden könnte, ohne den Code nochmal anfassen zu müssen. Dadurch niedrigere Priorität als vorher angenommen — kann später in der Reihenfolge stehen.
|
||||||
|
|||||||
@@ -1,19 +1,30 @@
|
|||||||
using Microsoft.EntityFrameworkCore;
|
using Microsoft.EntityFrameworkCore;
|
||||||
using Microsoft.EntityFrameworkCore.Design;
|
using Microsoft.EntityFrameworkCore.Design;
|
||||||
using Microsoft.Extensions.Configuration;
|
using MySqlConnector;
|
||||||
using System.IO;
|
|
||||||
|
|
||||||
namespace Predictalytics.Infrastructure.Data;
|
namespace Predictalytics.Infrastructure.Data;
|
||||||
|
|
||||||
|
/// <summary>
|
||||||
|
/// Used only by "dotnet ef" design-time tooling (migrations add/update), never at runtime.
|
||||||
|
/// Target database is picked via env vars so no connection string/secret ever needs to
|
||||||
|
/// live in source or config: PREDICTALYTICS_DB_SERVER/_NAME/_USER/_PASSWORD.
|
||||||
|
/// Falls back to a local dev default if the env vars aren't set.
|
||||||
|
/// </summary>
|
||||||
public class AppDbContextFactory : IDesignTimeDbContextFactory<AppDbContext>
|
public class AppDbContextFactory : IDesignTimeDbContextFactory<AppDbContext>
|
||||||
{
|
{
|
||||||
public AppDbContext CreateDbContext(string[] args)
|
public AppDbContext CreateDbContext(string[] args)
|
||||||
{
|
{
|
||||||
var optionsBuilder = new DbContextOptionsBuilder<AppDbContext>();
|
var optionsBuilder = new DbContextOptionsBuilder<AppDbContext>();
|
||||||
// Fallback for local migrations
|
|
||||||
var connectionString = "Server=localhost;Database=Predictalytics;User=root;Password=;";
|
|
||||||
|
|
||||||
optionsBuilder.UseMySql(connectionString, new MySqlServerVersion(new Version(8, 0, 31)));
|
var builder = new MySqlConnectionStringBuilder
|
||||||
|
{
|
||||||
|
Server = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_SERVER") ?? "localhost",
|
||||||
|
Database = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_NAME") ?? "Predictalytics",
|
||||||
|
UserID = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_USER") ?? "root",
|
||||||
|
Password = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_PASSWORD") ?? "",
|
||||||
|
};
|
||||||
|
|
||||||
|
optionsBuilder.UseMySql(builder.ConnectionString, new MySqlServerVersion(new Version(8, 0, 31)));
|
||||||
|
|
||||||
return new AppDbContext(optionsBuilder.Options);
|
return new AppDbContext(optionsBuilder.Options);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user