NullableAndForeignKeyCleanup Migration

src/Tgstation.Server.Host/Models/Migrations/20180918204520_MYNullableAndForeignKeyCleanup.Designer.cs

@@ -0,0 +1,652 @@
+// <auto-generated />
+using System;
+using Microsoft.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+namespace Tgstation.Server.Host.Models.Migrations
+{
+	[DbContext(typeof(MySqlDatabaseContext))]
+	[Migration("20180918204520_MYNullableAndForeignKeyCleanup")]
+	partial class MYNullableAndForeignKeyCleanup
+	{
+		/// <summary>
+		/// Builds the target model
+		/// </summary>
+		/// <param name="modelBuilder">The <see cref="ModelBuilder"/> to use</param>
+		protected override void BuildTargetModel(ModelBuilder modelBuilder)
+		{
+#pragma warning disable 612, 618
+			modelBuilder
+				.HasAnnotation("ProductVersion", "2.1.3-rtm-32065")
+				.HasAnnotation("Relational:MaxIdentifierLength", 64);
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatBot", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("ConnectionString")
+						.IsRequired();
+
+					b.Property<bool?>("Enabled");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<string>("Name")
+						.IsRequired();
+
+					b.Property<int?>("Provider");
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId");
+
+					b.HasIndex("Name")
+						.IsUnique();
+
+					b.ToTable("ChatBots");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatChannel", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<long>("ChatSettingsId");
+
+					b.Property<ulong?>("DiscordChannelId");
+
+					b.Property<string>("IrcChannel");
+
+					b.Property<bool?>("IsAdminChannel")
+						.IsRequired();
+
+					b.Property<bool?>("IsUpdatesChannel")
+						.IsRequired();
+
+					b.Property<bool?>("IsWatchdogChannel")
+						.IsRequired();
+
+					b.Property<string>("Tag");
+
+					b.HasKey("Id");
+
+					b.HasIndex("ChatSettingsId", "DiscordChannelId")
+						.IsUnique();
+
+					b.HasIndex("ChatSettingsId", "IrcChannel")
+						.IsUnique();
+
+					b.ToTable("ChatChannels");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.CompileJob", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("ByondVersion")
+						.IsRequired();
+
+					b.Property<Guid?>("DirectoryName")
+						.IsRequired();
+
+					b.Property<string>("DmeName")
+						.IsRequired();
+
+					b.Property<long>("JobId");
+
+					b.Property<int>("MinimumSecurityLevel");
+
+					b.Property<string>("Output")
+						.IsRequired();
+
+					b.Property<long>("RevisionInformationId");
+
+					b.HasKey("Id");
+
+					b.HasIndex("DirectoryName");
+
+					b.HasIndex("JobId")
+						.IsUnique();
+
+					b.HasIndex("RevisionInformationId");
+
+					b.ToTable("CompileJobs");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamDaemonSettings", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("AccessToken");
+
+					b.Property<bool?>("AllowWebClient")
+						.IsRequired();
+
+					b.Property<bool?>("AutoStart")
+						.IsRequired();
+
+					b.Property<long>("InstanceId");
+
+					b.Property<ushort?>("PrimaryPort")
+						.IsRequired();
+
+					b.Property<int?>("ProcessId");
+
+					b.Property<ushort?>("SecondaryPort")
+						.IsRequired();
+
+					b.Property<int>("SecurityLevel");
+
+					b.Property<bool?>("SoftRestart")
+						.IsRequired();
+
+					b.Property<bool?>("SoftShutdown")
+						.IsRequired();
+
+					b.Property<uint?>("StartupTimeout")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("DreamDaemonSettings");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamMakerSettings", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<ushort?>("ApiValidationPort")
+						.IsRequired();
+
+					b.Property<int>("ApiValidationSecurityLevel");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<string>("ProjectName");
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("DreamMakerSettings");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.Instance", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<uint?>("AutoUpdateInterval")
+						.IsRequired();
+
+					b.Property<int>("ConfigurationType");
+
+					b.Property<string>("Name")
+						.IsRequired();
+
+					b.Property<bool?>("Online")
+						.IsRequired();
+
+					b.Property<string>("Path")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("Path")
+						.IsUnique();
+
+					b.ToTable("Instances");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.InstanceUser", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<ulong>("ByondRights");
+
+					b.Property<ulong>("ChatBotRights");
+
+					b.Property<ulong>("ConfigurationRights");
+
+					b.Property<ulong>("DreamDaemonRights");
+
+					b.Property<ulong>("DreamMakerRights");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<ulong>("InstanceUserRights");
+
+					b.Property<ulong>("RepositoryRights");
+
+					b.Property<long?>("UserId")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId");
+
+					b.HasIndex("UserId", "InstanceId")
+						.IsUnique();
+
+					b.ToTable("InstanceUsers");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.Job", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<ulong?>("CancelRight");
+
+					b.Property<ulong?>("CancelRightsType");
+
+					b.Property<bool?>("Cancelled")
+						.IsRequired();
+
+					b.Property<long?>("CancelledById");
+
+					b.Property<string>("Description")
+						.IsRequired();
+
+					b.Property<string>("ExceptionDetails");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<DateTimeOffset?>("StartedAt")
+						.IsRequired();
+
+					b.Property<long>("StartedById");
+
+					b.Property<DateTimeOffset?>("StoppedAt");
+
+					b.HasKey("Id");
+
+					b.HasIndex("CancelledById");
+
+					b.HasIndex("InstanceId");
+
+					b.HasIndex("StartedById");
+
+					b.ToTable("Jobs");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ReattachInformation", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("AccessIdentifier")
+						.IsRequired();
+
+					b.Property<string>("ChatChannelsJson")
+						.IsRequired();
+
+					b.Property<string>("ChatCommandsJson")
+						.IsRequired();
+
+					b.Property<long?>("CompileJobId");
+
+					b.Property<bool>("IsPrimary");
+
+					b.Property<ushort>("Port");
+
+					b.Property<int>("ProcessId");
+
+					b.Property<int>("RebootState");
+
+					b.Property<string>("ServerCommandsJson")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("CompileJobId");
+
+					b.ToTable("ReattachInformations");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RepositorySettings", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("AccessToken");
+
+					b.Property<string>("AccessUser");
+
+					b.Property<bool?>("AutoUpdatesKeepTestMerges")
+						.IsRequired();
+
+					b.Property<bool?>("AutoUpdatesSynchronize")
+						.IsRequired();
+
+					b.Property<string>("CommitterEmail")
+						.IsRequired();
+
+					b.Property<string>("CommitterName")
+						.IsRequired();
+
+					b.Property<long>("InstanceId");
+
+					b.Property<bool?>("PushTestMergeCommits")
+						.IsRequired();
+
+					b.Property<bool?>("ShowTestMergeCommitters")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("RepositorySettings");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevInfoTestMerge", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<long>("RevisionInformationId");
+
+					b.Property<long>("TestMergeId");
+
+					b.HasKey("Id");
+
+					b.HasIndex("RevisionInformationId");
+
+					b.HasIndex("TestMergeId");
+
+					b.ToTable("RevInfoTestMerges");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevisionInformation", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("CommitSha")
+						.IsRequired()
+						.HasMaxLength(40);
+
+					b.Property<long>("InstanceId");
+
+					b.Property<string>("OriginCommitSha")
+						.IsRequired()
+						.HasMaxLength(40);
+
+					b.HasKey("Id");
+
+					b.HasIndex("CommitSha")
+						.IsUnique();
+
+					b.HasIndex("InstanceId");
+
+					b.ToTable("RevisionInformations");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.TestMerge", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<string>("Author")
+						.IsRequired();
+
+					b.Property<string>("BodyAtMerge")
+						.IsRequired();
+
+					b.Property<string>("Comment");
+
+					b.Property<DateTimeOffset>("MergedAt");
+
+					b.Property<long>("MergedById");
+
+					b.Property<int?>("Number")
+						.IsRequired();
+
+					b.Property<long?>("PrimaryRevisionInformationId")
+						.IsRequired();
+
+					b.Property<string>("PullRequestRevision")
+						.IsRequired();
+
+					b.Property<string>("TitleAtMerge")
+						.IsRequired();
+
+					b.Property<string>("Url")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("MergedById");
+
+					b.HasIndex("PrimaryRevisionInformationId")
+						.IsUnique();
+
+					b.ToTable("TestMerges");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.User", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<ulong>("AdministrationRights");
+
+					b.Property<string>("CanonicalName")
+						.IsRequired();
+
+					b.Property<DateTimeOffset?>("CreatedAt")
+						.IsRequired();
+
+					b.Property<long?>("CreatedById");
+
+					b.Property<bool?>("Enabled")
+						.IsRequired();
+
+					b.Property<ulong>("InstanceManagerRights");
+
+					b.Property<DateTimeOffset?>("LastPasswordUpdate");
+
+					b.Property<string>("Name")
+						.IsRequired();
+
+					b.Property<string>("PasswordHash");
+
+					b.Property<string>("SystemIdentifier");
+
+					b.HasKey("Id");
+
+					b.HasIndex("CanonicalName")
+						.IsUnique();
+
+					b.HasIndex("CreatedById");
+
+					b.ToTable("Users");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.WatchdogReattachInformation", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd();
+
+					b.Property<long?>("AlphaId");
+
+					b.Property<bool>("AlphaIsActive");
+
+					b.Property<long?>("BravoId");
+
+					b.Property<long>("InstanceId");
+
+					b.HasKey("Id");
+
+					b.HasIndex("AlphaId");
+
+					b.HasIndex("BravoId");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("WatchdogReattachInformations");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatBot", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("ChatSettings")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatChannel", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.ChatBot", "ChatSettings")
+						.WithMany("Channels")
+						.HasForeignKey("ChatSettingsId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.CompileJob", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Job", "Job")
+						.WithOne()
+						.HasForeignKey("Tgstation.Server.Host.Models.CompileJob", "JobId")
+						.OnDelete(DeleteBehavior.Restrict);
+
+					b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "RevisionInformation")
+						.WithMany("CompileJobs")
+						.HasForeignKey("RevisionInformationId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamDaemonSettings", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithOne("DreamDaemonSettings")
+						.HasForeignKey("Tgstation.Server.Host.Models.DreamDaemonSettings", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamMakerSettings", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithOne("DreamMakerSettings")
+						.HasForeignKey("Tgstation.Server.Host.Models.DreamMakerSettings", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.InstanceUser", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("InstanceUsers")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+
+					b.HasOne("Tgstation.Server.Host.Models.User")
+						.WithMany("InstanceUsers")
+						.HasForeignKey("UserId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.Job", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.User", "CancelledBy")
+						.WithMany()
+						.HasForeignKey("CancelledById");
+
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("Jobs")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+
+					b.HasOne("Tgstation.Server.Host.Models.User", "StartedBy")
+						.WithMany()
+						.HasForeignKey("StartedById")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ReattachInformation", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.CompileJob", "CompileJob")
+						.WithMany()
+						.HasForeignKey("CompileJobId");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RepositorySettings", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithOne("RepositorySettings")
+						.HasForeignKey("Tgstation.Server.Host.Models.RepositorySettings", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevInfoTestMerge", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "RevisionInformation")
+						.WithMany("ActiveTestMerges")
+						.HasForeignKey("RevisionInformationId")
+						.OnDelete(DeleteBehavior.Cascade);
+
+					b.HasOne("Tgstation.Server.Host.Models.TestMerge", "TestMerge")
+						.WithMany("RevisonInformations")
+						.HasForeignKey("TestMergeId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevisionInformation", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("RevisionInformations")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.TestMerge", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.User", "MergedBy")
+						.WithMany("TestMerges")
+						.HasForeignKey("MergedById")
+						.OnDelete(DeleteBehavior.Restrict);
+
+					b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "PrimaryRevisionInformation")
+						.WithOne("PrimaryTestMerge")
+						.HasForeignKey("Tgstation.Server.Host.Models.TestMerge", "PrimaryRevisionInformationId")
+						.OnDelete(DeleteBehavior.Restrict);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.User", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.User", "CreatedBy")
+						.WithMany("CreatedUsers")
+						.HasForeignKey("CreatedById");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.WatchdogReattachInformation", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.ReattachInformation", "Alpha")
+						.WithMany()
+						.HasForeignKey("AlphaId");
+
+					b.HasOne("Tgstation.Server.Host.Models.ReattachInformation", "Bravo")
+						.WithMany()
+						.HasForeignKey("BravoId");
+
+					b.HasOne("Tgstation.Server.Host.Models.Instance")
+						.WithOne("WatchdogReattachInformation")
+						.HasForeignKey("Tgstation.Server.Host.Models.WatchdogReattachInformation", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+#pragma warning restore 612, 618
+		}
+	}
+}

src/Tgstation.Server.Host/Models/Migrations/20180918204520_MYNullableAndForeignKeyCleanup.cs

@@ -0,0 +1,219 @@
+using System;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+namespace Tgstation.Server.Host.Models.Migrations
+{
+	/// <summary>
+	/// Cleans up of nullable columns and foreign keys MySQL/MariaDB
+	/// </summary>
+	public partial class MYNullableAndForeignKeyCleanup : Migration
+	{
+		/// <summary>
+		/// Applies the migration
+		/// </summary>
+		/// <param name="migrationBuilder">The <see cref="MigrationBuilder"/> to use</param>
+		protected override void Up(MigrationBuilder migrationBuilder)
+		{
+			migrationBuilder.DropForeignKey(
+				name: "FK_Instances_WatchdogReattachInformations_WatchdogReattachInfor~",
+				table: "Instances");
+
+			migrationBuilder.DropForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges");
+
+			migrationBuilder.DropForeignKey(
+				name: "FK_CompileJobs_Jobs_JobId",
+				table: "CompileJobs");
+
+			migrationBuilder.DropIndex(
+				name: "IX_Instances_WatchdogReattachInformationId",
+				table: "Instances");
+
+			migrationBuilder.DropIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs");
+
+			migrationBuilder.DropColumn(
+				name: "WatchdogReattachInformationId",
+				table: "Instances");
+
+			migrationBuilder.AddColumn<long>(
+				name: "InstanceId",
+				table: "WatchdogReattachInformations",
+				nullable: false,
+				defaultValue: 0L);
+
+			migrationBuilder.AlterColumn<long>(
+				name: "PrimaryRevisionInformationId",
+				table: "TestMerges",
+				nullable: false,
+				oldClrType: typeof(long),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<string>(
+				name: "Output",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(string),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<long>(
+				name: "JobId",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(long),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<string>(
+				name: "DmeName",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(string),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<Guid>(
+				name: "DirectoryName",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(Guid),
+				oldNullable: true);
+
+			migrationBuilder.CreateIndex(
+				name: "IX_WatchdogReattachInformations_InstanceId",
+				table: "WatchdogReattachInformations",
+				column: "InstanceId",
+				unique: true);
+
+			migrationBuilder.CreateIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs",
+				column: "JobId",
+				unique: true);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_CompileJobs_Jobs_JobId",
+				table: "CompileJobs",
+				column: "JobId",
+				principalTable: "Jobs",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Restrict);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges",
+				column: "PrimaryRevisionInformationId",
+				principalTable: "RevisionInformations",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Restrict);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_WatchdogReattachInformations_Instances_InstanceId",
+				table: "WatchdogReattachInformations",
+				column: "InstanceId",
+				principalTable: "Instances",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Cascade);
+		}
+
+		/// <summary>
+		/// Unapplies the migration
+		/// </summary>
+		/// <param name="migrationBuilder">The <see cref="MigrationBuilder"/> to use</param>
+		protected override void Down(MigrationBuilder migrationBuilder)
+		{
+			migrationBuilder.DropForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges");
+
+			migrationBuilder.DropForeignKey(
+				name: "FK_WatchdogReattachInformations_Instances_InstanceId",
+				table: "WatchdogReattachInformations");
+
+			migrationBuilder.DropForeignKey(
+				name: "FK_CompileJobs_Jobs_JobId",
+				table: "CompileJobs");
+
+			migrationBuilder.DropIndex(
+				name: "IX_WatchdogReattachInformations_InstanceId",
+				table: "WatchdogReattachInformations");
+
+			migrationBuilder.DropIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs");
+
+			migrationBuilder.DropColumn(
+				name: "InstanceId",
+				table: "WatchdogReattachInformations");
+
+			migrationBuilder.AlterColumn<long>(
+				name: "PrimaryRevisionInformationId",
+				table: "TestMerges",
+				nullable: true,
+				oldClrType: typeof(long));
+
+			migrationBuilder.AddColumn<long>(
+				name: "WatchdogReattachInformationId",
+				table: "Instances",
+				nullable: true);
+
+			migrationBuilder.AlterColumn<string>(
+				name: "Output",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(string));
+
+			migrationBuilder.AlterColumn<long>(
+				name: "JobId",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(long));
+
+			migrationBuilder.AlterColumn<string>(
+				name: "DmeName",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(string));
+
+			migrationBuilder.AlterColumn<Guid>(
+				name: "DirectoryName",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(Guid));
+
+			migrationBuilder.CreateIndex(
+				name: "IX_Instances_WatchdogReattachInformationId",
+				table: "Instances",
+				column: "WatchdogReattachInformationId");
+
+			migrationBuilder.CreateIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs",
+				column: "JobId");
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_CompileJobs_Jobs_JobId",
+				table: "CompileJobs",
+				column: "JobId",
+				principalTable: "Jobs",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Restrict);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_Instances_WatchdogReattachInformations_WatchdogReattachInfor~",
+				table: "Instances",
+				column: "WatchdogReattachInformationId",
+				principalTable: "WatchdogReattachInformations",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Restrict);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges",
+				column: "PrimaryRevisionInformationId",
+				principalTable: "RevisionInformations",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.SetNull);
+		}
+	}
+}

src/Tgstation.Server.Host/Models/Migrations/20180918205224_MSNullableAndForeignKeyCleanup.Designer.cs

@@ -0,0 +1,679 @@
+// <auto-generated />
+using System;
+using Microsoft.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Metadata;
+using Microsoft.EntityFrameworkCore.Migrations;
+using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+
+namespace Tgstation.Server.Host.Models.Migrations
+{
+	[DbContext(typeof(SqlServerDatabaseContext))]
+	[Migration("20180918205224_MSNullableAndForeignKeyCleanup")]
+	partial class MSNullableAndForeignKeyCleanup
+	{
+		/// <summary>
+		/// Builds the target model
+		/// </summary>
+		/// <param name="modelBuilder">The <see cref="ModelBuilder"/> to use</param>
+		protected override void BuildTargetModel(ModelBuilder modelBuilder)
+		{
+#pragma warning disable 612, 618
+			modelBuilder
+				.HasAnnotation("ProductVersion", "2.1.3-rtm-32065")
+				.HasAnnotation("Relational:MaxIdentifierLength", 128)
+				.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatBot", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("ConnectionString")
+						.IsRequired();
+
+					b.Property<bool?>("Enabled");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<string>("Name")
+						.IsRequired();
+
+					b.Property<int?>("Provider");
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId");
+
+					b.HasIndex("Name")
+						.IsUnique();
+
+					b.ToTable("ChatBots");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatChannel", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<long>("ChatSettingsId");
+
+					b.Property<decimal?>("DiscordChannelId")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<string>("IrcChannel");
+
+					b.Property<bool?>("IsAdminChannel")
+						.IsRequired();
+
+					b.Property<bool?>("IsUpdatesChannel")
+						.IsRequired();
+
+					b.Property<bool?>("IsWatchdogChannel")
+						.IsRequired();
+
+					b.Property<string>("Tag");
+
+					b.HasKey("Id");
+
+					b.HasIndex("ChatSettingsId", "DiscordChannelId")
+						.IsUnique()
+						.HasFilter("[DiscordChannelId] IS NOT NULL");
+
+					b.HasIndex("ChatSettingsId", "IrcChannel")
+						.IsUnique()
+						.HasFilter("[IrcChannel] IS NOT NULL");
+
+					b.ToTable("ChatChannels");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.CompileJob", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("ByondVersion")
+						.IsRequired();
+
+					b.Property<Guid?>("DirectoryName")
+						.IsRequired();
+
+					b.Property<string>("DmeName")
+						.IsRequired();
+
+					b.Property<long>("JobId");
+
+					b.Property<int>("MinimumSecurityLevel");
+
+					b.Property<string>("Output")
+						.IsRequired();
+
+					b.Property<long>("RevisionInformationId");
+
+					b.HasKey("Id");
+
+					b.HasIndex("DirectoryName");
+
+					b.HasIndex("JobId")
+						.IsUnique();
+
+					b.HasIndex("RevisionInformationId");
+
+					b.ToTable("CompileJobs");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamDaemonSettings", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("AccessToken");
+
+					b.Property<bool?>("AllowWebClient")
+						.IsRequired();
+
+					b.Property<bool?>("AutoStart")
+						.IsRequired();
+
+					b.Property<long>("InstanceId");
+
+					b.Property<int>("PrimaryPort");
+
+					b.Property<int?>("ProcessId");
+
+					b.Property<int>("SecondaryPort");
+
+					b.Property<int>("SecurityLevel");
+
+					b.Property<bool?>("SoftRestart")
+						.IsRequired();
+
+					b.Property<bool?>("SoftShutdown")
+						.IsRequired();
+
+					b.Property<long>("StartupTimeout");
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("DreamDaemonSettings");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamMakerSettings", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<int>("ApiValidationPort");
+
+					b.Property<int>("ApiValidationSecurityLevel");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<string>("ProjectName");
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("DreamMakerSettings");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.Instance", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<long>("AutoUpdateInterval");
+
+					b.Property<int>("ConfigurationType");
+
+					b.Property<string>("Name")
+						.IsRequired();
+
+					b.Property<bool?>("Online")
+						.IsRequired();
+
+					b.Property<string>("Path")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("Path")
+						.IsUnique();
+
+					b.ToTable("Instances");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.InstanceUser", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<decimal>("ByondRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<decimal>("ChatBotRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<decimal>("ConfigurationRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<decimal>("DreamDaemonRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<decimal>("DreamMakerRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<long>("InstanceId");
+
+					b.Property<decimal>("InstanceUserRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<decimal>("RepositoryRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<long?>("UserId")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId");
+
+					b.HasIndex("UserId", "InstanceId")
+						.IsUnique();
+
+					b.ToTable("InstanceUsers");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.Job", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<decimal?>("CancelRight")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<decimal?>("CancelRightsType")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<bool?>("Cancelled")
+						.IsRequired();
+
+					b.Property<long?>("CancelledById");
+
+					b.Property<string>("Description")
+						.IsRequired();
+
+					b.Property<string>("ExceptionDetails");
+
+					b.Property<long>("InstanceId");
+
+					b.Property<DateTimeOffset?>("StartedAt")
+						.IsRequired();
+
+					b.Property<long>("StartedById");
+
+					b.Property<DateTimeOffset?>("StoppedAt");
+
+					b.HasKey("Id");
+
+					b.HasIndex("CancelledById");
+
+					b.HasIndex("InstanceId");
+
+					b.HasIndex("StartedById");
+
+					b.ToTable("Jobs");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ReattachInformation", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("AccessIdentifier")
+						.IsRequired();
+
+					b.Property<string>("ChatChannelsJson")
+						.IsRequired();
+
+					b.Property<string>("ChatCommandsJson")
+						.IsRequired();
+
+					b.Property<long?>("CompileJobId");
+
+					b.Property<bool>("IsPrimary");
+
+					b.Property<int>("Port");
+
+					b.Property<int>("ProcessId");
+
+					b.Property<int>("RebootState");
+
+					b.Property<string>("ServerCommandsJson")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("CompileJobId");
+
+					b.ToTable("ReattachInformations");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RepositorySettings", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("AccessToken");
+
+					b.Property<string>("AccessUser");
+
+					b.Property<bool?>("AutoUpdatesKeepTestMerges")
+						.IsRequired();
+
+					b.Property<bool?>("AutoUpdatesSynchronize")
+						.IsRequired();
+
+					b.Property<string>("CommitterEmail")
+						.IsRequired();
+
+					b.Property<string>("CommitterName")
+						.IsRequired();
+
+					b.Property<long>("InstanceId");
+
+					b.Property<bool?>("PushTestMergeCommits")
+						.IsRequired();
+
+					b.Property<bool?>("ShowTestMergeCommitters")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("RepositorySettings");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevInfoTestMerge", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<long>("RevisionInformationId");
+
+					b.Property<long>("TestMergeId");
+
+					b.HasKey("Id");
+
+					b.HasIndex("RevisionInformationId");
+
+					b.HasIndex("TestMergeId");
+
+					b.ToTable("RevInfoTestMerges");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevisionInformation", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("CommitSha")
+						.IsRequired()
+						.HasMaxLength(40);
+
+					b.Property<long>("InstanceId");
+
+					b.Property<string>("OriginCommitSha")
+						.IsRequired()
+						.HasMaxLength(40);
+
+					b.HasKey("Id");
+
+					b.HasIndex("CommitSha")
+						.IsUnique();
+
+					b.HasIndex("InstanceId");
+
+					b.ToTable("RevisionInformations");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.TestMerge", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<string>("Author")
+						.IsRequired();
+
+					b.Property<string>("BodyAtMerge")
+						.IsRequired();
+
+					b.Property<string>("Comment");
+
+					b.Property<DateTimeOffset>("MergedAt");
+
+					b.Property<long>("MergedById");
+
+					b.Property<int?>("Number")
+						.IsRequired();
+
+					b.Property<long?>("PrimaryRevisionInformationId")
+						.IsRequired();
+
+					b.Property<string>("PullRequestRevision")
+						.IsRequired();
+
+					b.Property<string>("TitleAtMerge")
+						.IsRequired();
+
+					b.Property<string>("Url")
+						.IsRequired();
+
+					b.HasKey("Id");
+
+					b.HasIndex("MergedById");
+
+					b.HasIndex("PrimaryRevisionInformationId")
+						.IsUnique();
+
+					b.ToTable("TestMerges");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.User", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<decimal>("AdministrationRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<string>("CanonicalName")
+						.IsRequired();
+
+					b.Property<DateTimeOffset?>("CreatedAt")
+						.IsRequired();
+
+					b.Property<long?>("CreatedById");
+
+					b.Property<bool?>("Enabled")
+						.IsRequired();
+
+					b.Property<decimal>("InstanceManagerRights")
+						.HasConversion(new ValueConverter<decimal, decimal>(v => default(decimal), v => default(decimal), new ConverterMappingHints(precision: 20, scale: 0)));
+
+					b.Property<DateTimeOffset?>("LastPasswordUpdate");
+
+					b.Property<string>("Name")
+						.IsRequired();
+
+					b.Property<string>("PasswordHash");
+
+					b.Property<string>("SystemIdentifier");
+
+					b.HasKey("Id");
+
+					b.HasIndex("CanonicalName")
+						.IsUnique();
+
+					b.HasIndex("CreatedById");
+
+					b.ToTable("Users");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.WatchdogReattachInformation", b =>
+				{
+					b.Property<long>("Id")
+						.ValueGeneratedOnAdd()
+						.HasAnnotation("SqlServer:ValueGenerationStrategy", SqlServerValueGenerationStrategy.IdentityColumn);
+
+					b.Property<long?>("AlphaId");
+
+					b.Property<bool>("AlphaIsActive");
+
+					b.Property<long?>("BravoId");
+
+					b.Property<long>("InstanceId");
+
+					b.HasKey("Id");
+
+					b.HasIndex("AlphaId");
+
+					b.HasIndex("BravoId");
+
+					b.HasIndex("InstanceId")
+						.IsUnique();
+
+					b.ToTable("WatchdogReattachInformations");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatBot", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("ChatSettings")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ChatChannel", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.ChatBot", "ChatSettings")
+						.WithMany("Channels")
+						.HasForeignKey("ChatSettingsId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.CompileJob", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Job", "Job")
+						.WithOne()
+						.HasForeignKey("Tgstation.Server.Host.Models.CompileJob", "JobId")
+						.OnDelete(DeleteBehavior.Restrict);
+
+					b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "RevisionInformation")
+						.WithMany("CompileJobs")
+						.HasForeignKey("RevisionInformationId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamDaemonSettings", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithOne("DreamDaemonSettings")
+						.HasForeignKey("Tgstation.Server.Host.Models.DreamDaemonSettings", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.DreamMakerSettings", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithOne("DreamMakerSettings")
+						.HasForeignKey("Tgstation.Server.Host.Models.DreamMakerSettings", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.InstanceUser", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("InstanceUsers")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+
+					b.HasOne("Tgstation.Server.Host.Models.User")
+						.WithMany("InstanceUsers")
+						.HasForeignKey("UserId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.Job", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.User", "CancelledBy")
+						.WithMany()
+						.HasForeignKey("CancelledById");
+
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("Jobs")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+
+					b.HasOne("Tgstation.Server.Host.Models.User", "StartedBy")
+						.WithMany()
+						.HasForeignKey("StartedById")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.ReattachInformation", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.CompileJob", "CompileJob")
+						.WithMany()
+						.HasForeignKey("CompileJobId");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RepositorySettings", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithOne("RepositorySettings")
+						.HasForeignKey("Tgstation.Server.Host.Models.RepositorySettings", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevInfoTestMerge", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "RevisionInformation")
+						.WithMany("ActiveTestMerges")
+						.HasForeignKey("RevisionInformationId")
+						.OnDelete(DeleteBehavior.Cascade);
+
+					b.HasOne("Tgstation.Server.Host.Models.TestMerge", "TestMerge")
+						.WithMany("RevisonInformations")
+						.HasForeignKey("TestMergeId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.RevisionInformation", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
+						.WithMany("RevisionInformations")
+						.HasForeignKey("InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.TestMerge", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.User", "MergedBy")
+						.WithMany("TestMerges")
+						.HasForeignKey("MergedById")
+						.OnDelete(DeleteBehavior.Restrict);
+
+					b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "PrimaryRevisionInformation")
+						.WithOne("PrimaryTestMerge")
+						.HasForeignKey("Tgstation.Server.Host.Models.TestMerge", "PrimaryRevisionInformationId")
+						.OnDelete(DeleteBehavior.Restrict);
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.User", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.User", "CreatedBy")
+						.WithMany("CreatedUsers")
+						.HasForeignKey("CreatedById");
+				});
+
+			modelBuilder.Entity("Tgstation.Server.Host.Models.WatchdogReattachInformation", b =>
+				{
+					b.HasOne("Tgstation.Server.Host.Models.ReattachInformation", "Alpha")
+						.WithMany()
+						.HasForeignKey("AlphaId");
+
+					b.HasOne("Tgstation.Server.Host.Models.ReattachInformation", "Bravo")
+						.WithMany()
+						.HasForeignKey("BravoId");
+
+					b.HasOne("Tgstation.Server.Host.Models.Instance")
+						.WithOne("WatchdogReattachInformation")
+						.HasForeignKey("Tgstation.Server.Host.Models.WatchdogReattachInformation", "InstanceId")
+						.OnDelete(DeleteBehavior.Cascade);
+				});
+#pragma warning restore 612, 618
+		}
+	}
+}

src/Tgstation.Server.Host/Models/Migrations/20180918205224_MSNullableAndForeignKeyCleanup.cs

@@ -0,0 +1,216 @@
+using System;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+namespace Tgstation.Server.Host.Models.Migrations
+{
+	/// <summary>
+	/// Cleans up of nullable columns and foreign keys MSSQL
+	/// </summary>
+	public partial class MSNullableAndForeignKeyCleanup : Migration
+	{
+		/// <summary>
+		/// Applies the migration
+		/// </summary>
+		/// <param name="migrationBuilder">The <see cref="MigrationBuilder"/> to use</param>
+		protected override void Up(MigrationBuilder migrationBuilder)
+		{
+			migrationBuilder.DropForeignKey(
+				name: "FK_Instances_WatchdogReattachInformations_WatchdogReattachInformationId",
+				table: "Instances");
+
+			migrationBuilder.DropForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges");
+
+			migrationBuilder.DropIndex(
+				name: "IX_TestMerges_PrimaryRevisionInformationId",
+				table: "TestMerges");
+
+			migrationBuilder.DropIndex(
+				name: "IX_Instances_WatchdogReattachInformationId",
+				table: "Instances");
+
+			migrationBuilder.DropIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs");
+
+			migrationBuilder.DropColumn(
+				name: "WatchdogReattachInformationId",
+				table: "Instances");
+
+			migrationBuilder.AddColumn<long>(
+				name: "InstanceId",
+				table: "WatchdogReattachInformations",
+				nullable: false,
+				defaultValue: 0L);
+
+			migrationBuilder.AlterColumn<long>(
+				name: "PrimaryRevisionInformationId",
+				table: "TestMerges",
+				nullable: false,
+				oldClrType: typeof(long),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<string>(
+				name: "Output",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(string),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<long>(
+				name: "JobId",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(long),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<string>(
+				name: "DmeName",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(string),
+				oldNullable: true);
+
+			migrationBuilder.AlterColumn<Guid>(
+				name: "DirectoryName",
+				table: "CompileJobs",
+				nullable: false,
+				oldClrType: typeof(Guid),
+				oldNullable: true);
+
+			migrationBuilder.CreateIndex(
+				name: "IX_WatchdogReattachInformations_InstanceId",
+				table: "WatchdogReattachInformations",
+				column: "InstanceId",
+				unique: true);
+
+			migrationBuilder.CreateIndex(
+				name: "IX_TestMerges_PrimaryRevisionInformationId",
+				table: "TestMerges",
+				column: "PrimaryRevisionInformationId",
+				unique: true);
+
+			migrationBuilder.CreateIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs",
+				column: "JobId",
+				unique: true);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges",
+				column: "PrimaryRevisionInformationId",
+				principalTable: "RevisionInformations",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Restrict);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_WatchdogReattachInformations_Instances_InstanceId",
+				table: "WatchdogReattachInformations",
+				column: "InstanceId",
+				principalTable: "Instances",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Cascade);
+		}
+
+		/// <summary>
+		/// Unapplies the migration
+		/// </summary>
+		/// <param name="migrationBuilder">The <see cref="MigrationBuilder"/> to use</param>
+		protected override void Down(MigrationBuilder migrationBuilder)
+		{
+			migrationBuilder.DropForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges");
+
+			migrationBuilder.DropForeignKey(
+				name: "FK_WatchdogReattachInformations_Instances_InstanceId",
+				table: "WatchdogReattachInformations");
+
+			migrationBuilder.DropIndex(
+				name: "IX_WatchdogReattachInformations_InstanceId",
+				table: "WatchdogReattachInformations");
+
+			migrationBuilder.DropIndex(
+				name: "IX_TestMerges_PrimaryRevisionInformationId",
+				table: "TestMerges");
+
+			migrationBuilder.DropIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs");
+
+			migrationBuilder.DropColumn(
+				name: "InstanceId",
+				table: "WatchdogReattachInformations");
+
+			migrationBuilder.AlterColumn<long>(
+				name: "PrimaryRevisionInformationId",
+				table: "TestMerges",
+				nullable: true,
+				oldClrType: typeof(long));
+
+			migrationBuilder.AddColumn<long>(
+				name: "WatchdogReattachInformationId",
+				table: "Instances",
+				nullable: true);
+
+			migrationBuilder.AlterColumn<string>(
+				name: "Output",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(string));
+
+			migrationBuilder.AlterColumn<long>(
+				name: "JobId",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(long));
+
+			migrationBuilder.AlterColumn<string>(
+				name: "DmeName",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(string));
+
+			migrationBuilder.AlterColumn<Guid>(
+				name: "DirectoryName",
+				table: "CompileJobs",
+				nullable: true,
+				oldClrType: typeof(Guid));
+
+			migrationBuilder.CreateIndex(
+				name: "IX_TestMerges_PrimaryRevisionInformationId",
+				table: "TestMerges",
+				column: "PrimaryRevisionInformationId",
+				unique: true,
+				filter: "[PrimaryRevisionInformationId] IS NOT NULL");
+
+			migrationBuilder.CreateIndex(
+				name: "IX_Instances_WatchdogReattachInformationId",
+				table: "Instances",
+				column: "WatchdogReattachInformationId");
+
+			migrationBuilder.CreateIndex(
+				name: "IX_CompileJobs_JobId",
+				table: "CompileJobs",
+				column: "JobId");
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_Instances_WatchdogReattachInformations_WatchdogReattachInformationId",
+				table: "Instances",
+				column: "WatchdogReattachInformationId",
+				principalTable: "WatchdogReattachInformations",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.Restrict);
+
+			migrationBuilder.AddForeignKey(
+				name: "FK_TestMerges_RevisionInformations_PrimaryRevisionInformationId",
+				table: "TestMerges",
+				column: "PrimaryRevisionInformationId",
+				principalTable: "RevisionInformations",
+				principalColumn: "Id",
+				onDelete: ReferentialAction.SetNull);
+		}
+	}
+}

src/Tgstation.Server.Host/Models/Migrations/MySqlDatabaseContextModelSnapshot.cs

@@ -2,6 +2,8 @@
 using System;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+using Tgstation.Server.Host.Models;
 
 namespace Tgstation.Server.Host.Models.Migrations
 {
@@ -83,15 +85,18 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.Property<string>("ByondVersion")
                         .IsRequired();
 
-                    b.Property<Guid?>("DirectoryName");
+                    b.Property<Guid?>("DirectoryName")
+                        .IsRequired();
 
-                    b.Property<string>("DmeName");
+                    b.Property<string>("DmeName")
+                        .IsRequired();
 
-                    b.Property<long?>("JobId");
+                    b.Property<long>("JobId");
 
                     b.Property<int>("MinimumSecurityLevel");
 
-                    b.Property<string>("Output");
+                    b.Property<string>("Output")
+                        .IsRequired();
 
                     b.Property<long>("RevisionInformationId");
 
@@ -99,7 +104,8 @@ namespace Tgstation.Server.Host.Models.Migrations
 
                     b.HasIndex("DirectoryName");
 
-                    b.HasIndex("JobId");
+                    b.HasIndex("JobId")
+                        .IsUnique();
 
                     b.HasIndex("RevisionInformationId");
 
@@ -189,15 +195,11 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.Property<string>("Path")
                         .IsRequired();
 
-                    b.Property<long?>("WatchdogReattachInformationId");
-
                     b.HasKey("Id");
 
                     b.HasIndex("Path")
                         .IsUnique();
 
-                    b.HasIndex("WatchdogReattachInformationId");
-
                     b.ToTable("Instances");
                 });
 
@@ -408,7 +410,8 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.Property<int?>("Number")
                         .IsRequired();
 
-                    b.Property<long?>("PrimaryRevisionInformationId");
+                    b.Property<long?>("PrimaryRevisionInformationId")
+                        .IsRequired();
 
                     b.Property<string>("PullRequestRevision")
                         .IsRequired();
@@ -479,12 +482,17 @@ namespace Tgstation.Server.Host.Models.Migrations
 
                     b.Property<long?>("BravoId");
 
+                    b.Property<long>("InstanceId");
+
                     b.HasKey("Id");
 
                     b.HasIndex("AlphaId");
 
                     b.HasIndex("BravoId");
 
+                    b.HasIndex("InstanceId")
+                        .IsUnique();
+
                     b.ToTable("WatchdogReattachInformations");
                 });
 
@@ -507,8 +515,9 @@ namespace Tgstation.Server.Host.Models.Migrations
             modelBuilder.Entity("Tgstation.Server.Host.Models.CompileJob", b =>
                 {
                     b.HasOne("Tgstation.Server.Host.Models.Job", "Job")
-                        .WithMany()
-                        .HasForeignKey("JobId");
+                        .WithOne()
+                        .HasForeignKey("Tgstation.Server.Host.Models.CompileJob", "JobId")
+                        .OnDelete(DeleteBehavior.Restrict);
 
                     b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "RevisionInformation")
                         .WithMany("CompileJobs")
@@ -532,13 +541,6 @@ namespace Tgstation.Server.Host.Models.Migrations
                         .OnDelete(DeleteBehavior.Cascade);
                 });
 
-            modelBuilder.Entity("Tgstation.Server.Host.Models.Instance", b =>
-                {
-                    b.HasOne("Tgstation.Server.Host.Models.WatchdogReattachInformation", "WatchdogReattachInformation")
-                        .WithMany()
-                        .HasForeignKey("WatchdogReattachInformationId");
-                });
-
             modelBuilder.Entity("Tgstation.Server.Host.Models.InstanceUser", b =>
                 {
                     b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
@@ -615,7 +617,7 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "PrimaryRevisionInformation")
                         .WithOne("PrimaryTestMerge")
                         .HasForeignKey("Tgstation.Server.Host.Models.TestMerge", "PrimaryRevisionInformationId")
-                        .OnDelete(DeleteBehavior.SetNull);
+                        .OnDelete(DeleteBehavior.Restrict);
                 });
 
             modelBuilder.Entity("Tgstation.Server.Host.Models.User", b =>
@@ -634,6 +636,11 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.HasOne("Tgstation.Server.Host.Models.ReattachInformation", "Bravo")
                         .WithMany()
                         .HasForeignKey("BravoId");
+
+                    b.HasOne("Tgstation.Server.Host.Models.Instance")
+                        .WithOne("WatchdogReattachInformation")
+                        .HasForeignKey("Tgstation.Server.Host.Models.WatchdogReattachInformation", "InstanceId")
+                        .OnDelete(DeleteBehavior.Cascade);
                 });
 #pragma warning restore 612, 618
         }

src/Tgstation.Server.Host/Models/Migrations/SqlServerDatabaseContextModelSnapshot.cs

@@ -4,6 +4,7 @@ using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Infrastructure;
 using Microsoft.EntityFrameworkCore.Metadata;
 using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+using Tgstation.Server.Host.Models;
 
 namespace Tgstation.Server.Host.Models.Migrations
 {
@@ -92,15 +93,18 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.Property<string>("ByondVersion")
                         .IsRequired();
 
-                    b.Property<Guid?>("DirectoryName");
+                    b.Property<Guid?>("DirectoryName")
+                        .IsRequired();
 
-                    b.Property<string>("DmeName");
+                    b.Property<string>("DmeName")
+                        .IsRequired();
 
-                    b.Property<long?>("JobId");
+                    b.Property<long>("JobId");
 
                     b.Property<int>("MinimumSecurityLevel");
 
-                    b.Property<string>("Output");
+                    b.Property<string>("Output")
+                        .IsRequired();
 
                     b.Property<long>("RevisionInformationId");
 
@@ -108,7 +112,8 @@ namespace Tgstation.Server.Host.Models.Migrations
 
                     b.HasIndex("DirectoryName");
 
-                    b.HasIndex("JobId");
+                    b.HasIndex("JobId")
+                        .IsUnique();
 
                     b.HasIndex("RevisionInformationId");
 
@@ -196,15 +201,11 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.Property<string>("Path")
                         .IsRequired();
 
-                    b.Property<long?>("WatchdogReattachInformationId");
-
                     b.HasKey("Id");
 
                     b.HasIndex("Path")
                         .IsUnique();
 
-                    b.HasIndex("WatchdogReattachInformationId");
-
                     b.ToTable("Instances");
                 });
 
@@ -431,7 +432,8 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.Property<int?>("Number")
                         .IsRequired();
 
-                    b.Property<long?>("PrimaryRevisionInformationId");
+                    b.Property<long?>("PrimaryRevisionInformationId")
+                        .IsRequired();
 
                     b.Property<string>("PullRequestRevision")
                         .IsRequired();
@@ -447,8 +449,7 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.HasIndex("MergedById");
 
                     b.HasIndex("PrimaryRevisionInformationId")
-                        .IsUnique()
-                        .HasFilter("[PrimaryRevisionInformationId] IS NOT NULL");
+                        .IsUnique();
 
                     b.ToTable("TestMerges");
                 });
@@ -507,12 +508,17 @@ namespace Tgstation.Server.Host.Models.Migrations
 
                     b.Property<long?>("BravoId");
 
+                    b.Property<long>("InstanceId");
+
                     b.HasKey("Id");
 
                     b.HasIndex("AlphaId");
 
                     b.HasIndex("BravoId");
 
+                    b.HasIndex("InstanceId")
+                        .IsUnique();
+
                     b.ToTable("WatchdogReattachInformations");
                 });
 
@@ -535,8 +541,9 @@ namespace Tgstation.Server.Host.Models.Migrations
             modelBuilder.Entity("Tgstation.Server.Host.Models.CompileJob", b =>
                 {
                     b.HasOne("Tgstation.Server.Host.Models.Job", "Job")
-                        .WithMany()
-                        .HasForeignKey("JobId");
+                        .WithOne()
+                        .HasForeignKey("Tgstation.Server.Host.Models.CompileJob", "JobId")
+                        .OnDelete(DeleteBehavior.Restrict);
 
                     b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "RevisionInformation")
                         .WithMany("CompileJobs")
@@ -560,13 +567,6 @@ namespace Tgstation.Server.Host.Models.Migrations
                         .OnDelete(DeleteBehavior.Cascade);
                 });
 
-            modelBuilder.Entity("Tgstation.Server.Host.Models.Instance", b =>
-                {
-                    b.HasOne("Tgstation.Server.Host.Models.WatchdogReattachInformation", "WatchdogReattachInformation")
-                        .WithMany()
-                        .HasForeignKey("WatchdogReattachInformationId");
-                });
-
             modelBuilder.Entity("Tgstation.Server.Host.Models.InstanceUser", b =>
                 {
                     b.HasOne("Tgstation.Server.Host.Models.Instance", "Instance")
@@ -643,7 +643,7 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.HasOne("Tgstation.Server.Host.Models.RevisionInformation", "PrimaryRevisionInformation")
                         .WithOne("PrimaryTestMerge")
                         .HasForeignKey("Tgstation.Server.Host.Models.TestMerge", "PrimaryRevisionInformationId")
-                        .OnDelete(DeleteBehavior.SetNull);
+                        .OnDelete(DeleteBehavior.Restrict);
                 });
 
             modelBuilder.Entity("Tgstation.Server.Host.Models.User", b =>
@@ -662,6 +662,11 @@ namespace Tgstation.Server.Host.Models.Migrations
                     b.HasOne("Tgstation.Server.Host.Models.ReattachInformation", "Bravo")
                         .WithMany()
                         .HasForeignKey("BravoId");
+
+                    b.HasOne("Tgstation.Server.Host.Models.Instance")
+                        .WithOne("WatchdogReattachInformation")
+                        .HasForeignKey("Tgstation.Server.Host.Models.WatchdogReattachInformation", "InstanceId")
+                        .OnDelete(DeleteBehavior.Cascade);
                 });
 #pragma warning restore 612, 618
         }