-
Notifications
You must be signed in to change notification settings - Fork 1.1k
.NET: ci: Unblock Merge queue by disabling DurableTask TTL tests #3464
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR temporarily disables flaky Time-To-Live (TTL) integration tests in the DurableTask test suite to unblock the merge queue. The tests are experiencing intermittent failures with TaskCanceledException, and this change allows the team to address the flakiness without blocking ongoing development work. Issue #3463 has been created to track re-enabling these tests.
Changes:
- Changed the test category trait from "Integration" to "IntegrationDisabled" for the TimeToLiveTests class
- Updated the CI workflow to filter out tests with "IntegrationDisabled" category when running integration tests
- Minor whitespace cleanup in the workflow file
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| dotnet/tests/Microsoft.Agents.AI.DurableTask.IntegrationTests/TimeToLiveTests.cs | Changed test category from "Integration" to "IntegrationDisabled" to exclude TTL tests from CI runs |
| .github/workflows/dotnet-build-and-test.yml | Added test filter to exclude "IntegrationDisabled" tests and removed trailing whitespace |
The DurableTask Integration tests have some flakiness around the TTL tests. To unblock the queue we will disable them and create an issue #3463 to track fixing/re-enabling.
Contribution Checklist
Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.