Skip to content

[Diagnostic] Validate SQL initialization fix on platform 30.0.56092.0 - #12621

Closed
Prangshuman Das (t-prda) wants to merge 31 commits into
mainfrom
features/653457-sql-platform-fix-uptake
Closed

[Diagnostic] Validate SQL initialization fix on platform 30.0.56092.0#12621
Prangshuman Das (t-prda) wants to merge 31 commits into
mainfrom
features/653457-sql-platform-fix-uptake

Conversation

@t-prda

@t-prda Prangshuman Das (t-prda) commented Oct 9, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

DO NOT MERGE - matched fixed-platform validation

AB#653457
AB#653393

Validate the SQL initialization fix using the now-published 30.0.56092.0 runtime, independently of the standalone web-service-key experiment.

  • Official build 2860912 and publisher 2861912 succeeded. The publisher's actual build resource is 2860912/version 30.0.56092.0, whose source includes the fix. The candidate is present in the platform artifact index.
  • Ten original control cells: five sequential W1/DE pairs, with at most two concurrent containers, four mounted tenants and three workers per container.
  • No added warmup, readiness probe, retry, replacement, or disabled test. Later pairs wait for completion of the preceding pair; a failed original is retained, not retried.
  • Preserve the original compiled application source/packages, application artifact 30.0.55683.0, helper, image, authentication and 253-method / 21-codeunit / 19-existing-skip cohort.
  • Reuse the preserved original-comparison W1/DE package snapshots. Each cell checks immutable transport metadata and the original archive/package fingerprints; it does not rebuild or substitute fresh-main application code.
  • Update both BCPlatform.Version and the runtime URL/provenance guards. Require an actual installed-NST FileVersion of 30.0.56092.0 before test execution.
  • Preserve raw results, event diagnostics, lifecycle/resource evidence and run-owned cleanup. Ordinary CI fan-out is blocked for this diagnostic branch.

The original 300-trial run and the existing expanded-API experiment remain unchanged. This is not the held one-versus-two tenant repetition campaign.

Validation status

252 local PowerShell tests passed; PowerShell/YAML parsing and focused analyzer checks passed. Independent review found no significant issues. Both original snapshot IDs remain available and retain the expected run/head provenance.

No fixed-runtime test result exists yet. Publication, source ancestry, local tests and a green workflow alone are not proof of the SQL fix. Passing cells must retain the full cohort, original outcomes, installed-runtime proof and owned cleanup.

@t-prda
Separate API test workflow infrastructure from AL uptake
7a0b32d 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Remove relocated auth source-pattern checks from workflow tests
564bc06 
The historical workflow patch relocated the original core checks. Remove that duplicate while retaining the behavioral workflow tests and downstream AL adoption metadata assertions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Keep clean-codeunit lanes opt-in until authentication uptake
6e1d8bb 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Merge current main baseline while preserving scoped API test exclusions
1d81466 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Do not classify ordinary API HTTP500 failures as platform races
156bca0 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Do not classify ordinary API HTTP500 failures as platform races
decc121 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Move API test credential cleanup to supported PipelineFinalize hook
a439380 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Simplify finalizer maintenance by retaining committed wrappers
dd6207b 
Remove the wrapper generator and its generator-specific tests. Keep the shared finalizer and behavioral tests that execute every project wrapper.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Merge current main with simplified supported API credential finalizer
331ae4b 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Merge main baseline containing upstream CLEAN27 warning cleanup
dcc0733 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Document tenant info object return contract
7003849 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Merge captured main to unblock stacked API validation
69df417 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Enable reviewed Expense APIs and consolidate unique Expense regressions
304e5b1 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Enable remaining general API authentication tests
5c21bd3 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Enable remaining Expense HTTP API tests with existing clean isolation
4316269 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Inherit complete Expense API re-enablement
f709ef3 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Merge captured main after workflow squash to restore Expense-only review
88881be 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Inherit captured main and clean Expense-first ancestry
7c87fa2 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Defer general API clean-isolation tests until authenticated uptake
3070e7a 
Temporarily sequence193 existing methods through DisabledTests selectors:135 APIV1 and58 APIV2. Removed in#11860; no product quarantine or selection logic.

AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Enable deferred general API methods with authenticated uptake
98df7b3 
Remove the193 temporary Expense-stage selectors. Preserve the complete general uptake source tree exactly.

AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Reuse existing API exclusion manifests for Expense-first sequencing
bd3bbc0 
Relocate the same193 temporary method exclusions without changing selection, authentication or tests. Preserve existing manifest order and remove separate stage files.

AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Remove temporary method exclusions within existing API manifests
c4953dc 
Inherit Expense-first manifest relocation and restore the exact general uptake source tree. Existing general exclusions remain owned by their downstream fixes.

AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session:3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Add isolated paired SQL database-name reset diagnostic
0c81129 
Related to AB#646383; diagnostic only, does not resolve the work item.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Match SQL pilot action shells to baseline PowerShell 7
ae6513a 
Preserve failed preparation as inconclusive; one replacement pair authorized. Related to AB#646383, diagnostic only.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Allow bounded control-only SQL reset diagnostic repetitions
d7dd4dd 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Run isolated W1 and DE API warmup diagnostic trials
fcc1776 
Preserve pinned PR2 packages and reused worker databases; run the original first-app warmup before clean execution, then one authenticated companies probe per restored worker. No retries or production source changes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Allow bounded companies readiness probe retries only
b234da1 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Preserve nested clean test results separately from warmup
8a7ebc7 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Add isolated method-exclusion and evidence-gated SQL retry experiments
fbd7ec4 
AB#646383. Diagnostic only: twenty manual W1/DE trials, maximum five parallel; immutable source and runtime pins.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Schedule balanced 300-trial SQL API comparison (AB#646383)
0227094 
Diagnostic only; do not merge. Compare control, identical original warmup, and correlated once-per-CU SQL retry. Keep all tests enabled, pinned source/packages, single 60-second probes and six-container global scheduling. Preserve source ZIP snapshots and first-attempt evidence; communication replacements remain explicit and separately accounted.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Isolate ten original control trials on published SQL platform fix (AB…
06da094 
…#653457)

Pin NST 30.0.56092.0 while preserving original PR2 compiled artifacts, control lifecycle and authentication. Restrict the authorized manual experiment to five sequential W1/DE pairs and two container slots.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@github-actions github-actions Bot added Build: Automation Workflows and other setup in .github folder Build: scripts & configs Build scripts and configuration files AL: Apps (W1) Add-on apps for W1 Team: Finance GitHub request for Finance area labels Oct 9, 2026
github-advanced-security[bot]
github-advanced-security Bot found potential problems Oct 9, 2026
View reviewed changes
Comment thread .github/workflows/SqlApiComparisonBatch.yaml
uses: microsoft/AL-Go/Actions/ReadSecrets@91b96c2b294be6f823277dafe6f03350abfb9d23
with:
shell: pwsh
gitHubSecrets: ${{ toJson(secrets) }}
@github-actions github-actions Bot added this to the Version 30.0 milestone Oct 9, 2026
@t-prda

Prangshuman Das (t-prda) commented Oct 13, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by the native PR stack #12697 (#12674 to #12684, all green). Closing.

This branch was successfully deployed

1 active deployment
triage — 06da0944 Deployed Oct 9, 2026 by t-prda via Classify team ownership #7337
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@github-advanced-security github-advanced-security[bot] github-advanced-security[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

AL: Apps (W1) Add-on apps for W1 Build: Automation Workflows and other setup in .github folder Build: scripts & configs Build scripts and configuration files Team: Finance GitHub request for Finance area

Projects

None yet

Milestone

Version 30.0

Development

Successfully merging this pull request may close these issues.

2 participants

@t-prda @github-advanced-security