[release/13.2] Disable Kusto emulator tests on Linux for#15465
[release/13.2] Disable Kusto emulator tests on Linux for#15465sebastienros wants to merge 3 commits intomicrosoft:release/13.2from
Conversation
- Disabled: KustoEmulator_Starts - Disabled: KustoEmulator_WithDatabase_CanReadIngestedData - Disabled: KustoEmulator_WithDatabaseThatAlreadyExists_ErrorIsIgnored - Disabled: KustoEmulator_WithInvalidDatabase_LogsErrorAndContinues - Disabled: KustoEmulator_WithBindMount_IsUsedForPersistence - Issue: microsoft#11820 These tests are being disabled on Linux while the Kusto emulator image fails to start due to unsupported CPU instruction set requirements. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Preserved: KustoEmulator_Starts on AzDO - Preserved: KustoEmulator_WithDatabase_CanReadIngestedData on AzDO - Preserved: KustoEmulator_WithDatabaseThatAlreadyExists_ErrorIsIgnored on AzDO - Preserved: KustoEmulator_WithInvalidDatabase_LogsErrorAndContinues on AzDO - Preserved: KustoEmulator_WithBindMount_IsUsedForPersistence on AzDO - Issue: microsoft#11820 This keeps the existing Azure DevOps skip while also retaining the new Linux-targeted skip. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Linked: KustoEmulator_Starts Linux skip to microsoft#15463 - Linked: KustoEmulator_WithDatabase_CanReadIngestedData Linux skip to microsoft#15463 - Linked: KustoEmulator_WithDatabaseThatAlreadyExists_ErrorIsIgnored Linux skip to microsoft#15463 - Linked: KustoEmulator_WithInvalidDatabase_LogsErrorAndContinues Linux skip to microsoft#15463 - Linked: KustoEmulator_WithBindMount_IsUsedForPersistence Linux skip to microsoft#15463 This keeps the existing Azure DevOps disable on issue microsoft#11820 and moves the Linux-only disable to the dedicated Kusto emulator issue. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 15465Or
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 15465" |
There was a problem hiding this comment.
Pull request overview
Backports a test-quarantine adjustment to the release/13.2 branch to prevent Kusto emulator-backed tests from failing on Linux CI due to the current emulator image requiring unsupported CPU instructions.
Changes:
- Added a Linux-scoped
[ActiveIssue]skip (#15463) to Kusto emulator functional tests. - Preserved the existing Azure DevOps-scoped
[ActiveIssue]skip (#11820) on the same tests.
|
Sorry Seb, merged the other rename PR which caused conflicts here and I tried to push a fix to your fork but looks like I don't have permission to do so. Should be straight forward to merge. |
|
this pr is actually better as it updates the tests to match the new behavior of the kusto emulator, so if we are ok with the changes I'd rather to that instead of just diabling the tests. But disabling is also quick if we want to unblock release/13.2 flow |
Can we do both? Disabling the tests here to unblock 13.2 PRs, but then port that change to fix and re-enable? |
Description
This PR backports the Linux-targeted Kusto emulator test disable to
release/13.2while preserving the existing Azure DevOps skip.The current Kusto emulator image fails to start on Linux CI because it exits with an unsupported CPU instruction set error (
SSE4.2/AVX2), which causes the emulator-backed tests to fail before they can become healthy.Addresses #15463
Checklist
<remarks />and<code />elements on your triple slash comments?aspire.devissue:Validation
tests/Aspire.Hosting.Azure.Kusto.Tests/Aspire.Hosting.Azure.Kusto.Tests.csprojsuccessfully on the backport branch after updating theActiveIssueattributes#11820and added a Linux-targeted disable on#15463