From 8776bd832c2e3ac935a9a714cb48ae1ef50350e2 Mon Sep 17 00:00:00 2001 From: Erik Darling Date: Fri, 4 Sep 2026 12:11:10 -0400 Subject: [PATCH] Scrub the last fleet identifier from main MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `main` is the repo's default branch and is swept separately from `dev` — the two have divergent history, so the scrubs merged to dev (#2900, #2903) never reached it. Four locations still named a real tenant server. Uses `omega`, the slug #2900 reconciled this server onto, so the name is consistent with dev and is already allowlisted in `FleetIdentifierScrubTests.SyntheticSlugs`. Same precedent as 26a4e149. Line endings preserved per file. Co-Authored-By: Claude Opus 5 --- Darling/Darling.Tests/DarlingPeerDisclosureTests.cs | 4 ++-- Darling/Darling.Tests/StatementSplitTimingTests.cs | 2 +- PerformanceMonitor.Collectors/CollectorContext.cs | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Darling/Darling.Tests/DarlingPeerDisclosureTests.cs b/Darling/Darling.Tests/DarlingPeerDisclosureTests.cs index e7dc31996..5c1c21818 100644 --- a/Darling/Darling.Tests/DarlingPeerDisclosureTests.cs +++ b/Darling/Darling.Tests/DarlingPeerDisclosureTests.cs @@ -382,7 +382,7 @@ private static JsonElement RenderedServerList(DarlingPeerDirectory.Snapshot peer { var rows = new List { - new(1, "prod-sql-use1-beta-01", "ayr", 16, new DateTime(2026, 8, 19, 12, 0, 0, DateTimeKind.Utc)), + new(1, "prod-sql-use1-beta-01", "omega", 16, new DateTime(2026, 8, 19, 12, 0, 0, DateTimeKind.Utc)), }; return JsonDocument @@ -410,7 +410,7 @@ public void ListServers_CarriesThePeerFleetsSummary() /* The existing payload is untouched — the disclosure is additive here too. */ var server = Assert.Single(root.GetProperty("servers").EnumerateArray()); Assert.Equal("prod-sql-use1-beta-01", server.GetProperty("server_name").GetString()); - Assert.Equal("ayr", server.GetProperty("display_name").GetString()); + Assert.Equal("omega", server.GetProperty("display_name").GetString()); } [Fact] diff --git a/Darling/Darling.Tests/StatementSplitTimingTests.cs b/Darling/Darling.Tests/StatementSplitTimingTests.cs index 812c10737..1b7b68507 100644 --- a/Darling/Darling.Tests/StatementSplitTimingTests.cs +++ b/Darling/Darling.Tests/StatementSplitTimingTests.cs @@ -76,7 +76,7 @@ public void EveryPhaseAccountedFor_ThePartsNeverExceedTheWhole() /// /// #2312: the separate plan-XML and text fetches run INSIDE the driver's sql: stopwatch but are - /// their own queries against the Query Store catalogs — on ayr-01 a 0-row closed-only cycle still cost + /// their own queries against the Query Store catalogs — on omega-01 a 0-row closed-only cycle still cost /// 298s and the blended number could not say where. They must come out of drain exactly like the /// watermark phase, or drain silently absorbs the one cost this investigation needs isolated. /// diff --git a/PerformanceMonitor.Collectors/CollectorContext.cs b/PerformanceMonitor.Collectors/CollectorContext.cs index 596517e42..ba821dc8e 100644 --- a/PerformanceMonitor.Collectors/CollectorContext.cs +++ b/PerformanceMonitor.Collectors/CollectorContext.cs @@ -250,7 +250,7 @@ public sealed class CollectorContext /// runs one (Darling; Lite has no separate fetch and leaves it zero). The fetch is INSIDE the driver's /// per-item sql: stopwatch but is neither open nor drain — it is its own query against /// sys.query_store_plan, and on a database with a huge Query Store catalog it can dominate the - /// whole item (ayr-01: a 0-row closed-only cycle still cost 298s, and the blended number could not say + /// whole item (omega-01: a 0-row closed-only cycle still cost 298s, and the blended number could not say /// where). Measured so drain stops absorbing it, exactly the #2164 argument one seam further down. /// public long PerItemPlanFetchMs { get; set; }