Skip to content
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

Long running test: SendMailAsync_CanBeCanceled_CancellationToken #73447

Closed
karelz opened this issue Aug 5, 2022 · 8 comments
Closed

Long running test: SendMailAsync_CanBeCanceled_CancellationToken #73447

karelz opened this issue Aug 5, 2022 · 8 comments
Assignees
Labels
area-System.Net os-linux Linux OS (any supported distro) os-mac-os-x macOS aka OSX
Milestone

Comments

@karelz
Copy link
Member

karelz commented Aug 5, 2022

Long running test:

  • SendMailAsync_CanBeCanceled_CancellationToken - System.Net.Mail.Tests.SmtpClientTest.SendMailAsync_CanBeCanceled_CancellationToken

Regression on 7/18

Failures 6/6-8/5 (incl. PRs):

  • x64 is default architecture
  • CoreCLR is default runtime
Date Run Notes
9/12 2x PR Alpine.314.Arm64.Open
openSUSE.15.2.Amd64.Open
9/10 1x Rolling run Centos.7.Amd64.Open
9/10 1x PR Windows.10.Amd64.Server2022.ES.Open - Unrelated infra timeout?
9/9 2x (1x Rolling run) OSX.1200.Amd64.Open - Mono
Centos.7.Amd64.Open
9/9 1x PR Windows.10.Amd64.Server2022.ES.Open - Unrelated infra timeout?
9/8 1x Rolling run Centos.7.Amd64.Open
9/8 1x PR #74623 Windows.10.Amd64.Server2022.ES.Open - Unrelated infra timeout?
9/6 1x PR openSUSE.15.2.Amd64.Open
8/29 1x Rolling run Centos.8.Amd64.Open
8/5-8/11 6x PR Only 1 unique PR #73472 - likely specific to the PR, unrelated error
8/5 1x PR Centos 8
8/3 1x Rolling run Debian 10 - Mono
8/2 2x PR Debian 10 - Mono
OpenSUSE 15.2
8/2 1x PR Windows 8.1 - Unrelated infra problem
8/1 1x PR Centos 8
7/30 1x PR OSX 12.00 - Mono
7/29 1x PR Debian 10 - Mono
7/29 1x PR Windows 10 - Windows Nano 1809 - amd64 - Unrelated infra timeout?
7/28 1x Rolling run Debian 10 - amd64 - Mono
7/27 3x (1x Rolling run) 2x Centos 8
Debian 10 - Mono
7/25 1x Rolling run Centos 7
7/22 2x (1x Rolling run) 2x openSUSE 15.2
7/21 1x PR Debian 10 - Mono
7/20 1x PR Alpine 314 - Arm32
7/18 1x Rolling run Fedora 34
let timeouts = (friendlyNamePrefix : string, includePR : bool) {
cluster('engsrvprod.kusto.windows.net').database('engineeringdata').WorkItems
| where Status == "Timeout"
| where FriendlyName startswith friendlyNamePrefix
//| where Finished > datetime('2021-09-07')
| distinct JobId, WorkItemId, Name, FriendlyName, ConsoleUri
| join kind=inner (cluster('engsrvprod.kusto.windows.net').database('engineeringdata').Jobs
    | where ((Branch == 'refs/heads/main') or (Branch == 'refs/heads/master') or (includePR and (Source startswith "pr/")) or (Branch startswith 'refs/heads/release/6.0'))
    | where Type startswith "test/functional/cli/"
        and not(Properties contains "runtime-staging")
    | summarize arg_max(Finished, Properties, Type, Branch, Source, Started, QueueName) by JobId
| project-rename JobType = Type) on JobId
| extend PropertiesJson = parse_json(Properties)
| extend OS = replace_regex(tostring(PropertiesJson.operatingSystem), @'\((.*)\).*|([^\(].*)', @'\1\2')
| extend Runtime = iif(PropertiesJson.runtimeFlavor == "mono", "Mono", iif(PropertiesJson.DefinitionName contains "coreclr", "CoreCLR", ""))
| extend TargetBranch = extractjson("$.['System.PullRequest.TargetBranch']", Properties)
| extend Architecture = PropertiesJson.architecture
| extend Scenario = iif(isempty(PropertiesJson.scenario), "--", PropertiesJson.scenario)
| project-away PropertiesJson
| order by Finished desc
};
timeouts('System.Net.Mail.Functional.Tests', true);
@karelz karelz added area-System.Net disabled-test The test is disabled in source code against the issue os-linux Linux OS (any supported distro) os-mac-os-x macOS aka OSX labels Aug 5, 2022
@karelz karelz added this to the 7.0.0 milestone Aug 5, 2022
@ghost
Copy link

ghost commented Aug 5, 2022

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

Long running test:

  • SendMailAsync_CanBeCanceled_CancellationToken - 'System.Net.Mail.Tests.SmtpClientTest.SendMailAsync_CanBeCanceled_CancellationToken'

Regression on 7/18
Disabled on 8/4 in PR #73340

Failures 6/6-8/5 (incl. PRs):

Date Run Notes
8/1 1x PR Ubuntu 16.04 amd64
7/30 1x PR OSX 12.00 amd64
7/29 1x PR Ubuntu 18.04 amd64
7/29 1x PR Windows10 - Helix infra timeout
7/28 1x Official run Ubuntu 18.04 amd64
7/27 3x (1x Official run) 2x Ubuntu 16.04 amd64 + 1x Ubuntu 18.04 amd64
7/25 1x Official run Ubuntu 16.04 amd64
7/22 2x (1x Official run) 2x Ubuntu 16.04 amd64
7/21 1x PR Ubuntu 18.04 amd64
7/20 1x PR Ubuntu 18.04 arm
7/18 1x Official run Ubuntu 16.04 amd64
let timeouts = (includePR : bool) {
cluster('engsrvprod.kusto.windows.net').database('engineeringdata').WorkItems
| where Status == "Timeout"
| where FriendlyName startswith "System.Net.Mail.Functional.Tests"
//| where Finished > datetime('2021-09-07')
| distinct JobId, WorkItemId, Name, FriendlyName, ConsoleUri
| join kind=inner (cluster('engsrvprod.kusto.windows.net').database('engineeringdata').Jobs
    | where ((Branch == 'refs/heads/main') or (Branch == 'refs/heads/master') or (includePR and (Source startswith "pr/")) or (Branch startswith 'refs/heads/release/6.0'))
    | where Type startswith "test/functional/cli/"
        and not(Properties contains "runtime-staging")
    | summarize arg_max(Finished, Properties, Type, Branch, Source, Started, QueueName) by JobId
| project-rename JobType = Type) on JobId
| order by Finished desc
};
timeouts(true);
Author: karelz
Assignees: -
Labels:

area-System.Net, disabled-test, os-linux, os-mac-os-x

Milestone: 7.0.0

karelz added a commit that referenced this issue Aug 5, 2022
SendMailAsync_CanBeCanceled_CancellationToken is failing also on CoreCLR and on OSX - tracked by #73447
karelz added a commit that referenced this issue Aug 5, 2022
…73452)

SendMailAsync_CanBeCanceled_CancellationToken is failing also on CoreCLR and on OSX - tracked by #73447
@karelz
Copy link
Member Author

karelz commented Aug 7, 2022

Test disabled in PR #73452

@karelz
Copy link
Member Author

karelz commented Aug 25, 2022

Update: We are re-enabling the test in main via PR #74545 with changes that will hopefully give us more info on the root cause.

@wfurt wfurt removed the disabled-test The test is disabled in source code against the issue label Aug 29, 2022
@karelz
Copy link
Member Author

karelz commented Aug 30, 2022

@wfurt please check Kusto and if there is no new failure in last 5 days (since the test was re-enabled), let's close it as done (for 8.0). Thanks!

@karelz
Copy link
Member Author

karelz commented Sep 12, 2022

More hits updated in the top post

@wfurt
Copy link
Member

wfurt commented Nov 8, 2022

@rzikm can you please check if this was fixed by #76361 or if we still see failures? It is 18 days so it should be probably long enough period to close this if we don't see occurrences.

@rzikm
Copy link
Member

rzikm commented Nov 9, 2022

No hits (of either assert or long running test) in last 2 weeks, so I think it is safe to close this as fixed.

@rzikm rzikm closed this as completed Nov 9, 2022
@karelz karelz modified the milestones: 7.0.0, 8.0.0 Nov 29, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Dec 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Net os-linux Linux OS (any supported distro) os-mac-os-x macOS aka OSX
Projects
None yet
Development

No branches or pull requests

3 participants