Skip to content

Conversation

@kotlarmilos
Copy link
Member

Fixes the failing gcdump test on the CI. The regression was potentially introduced in the #89726 by enabling the test on apple mobile platforms.

@ghost
Copy link

ghost commented Aug 4, 2023

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

Issue Details

Fixes the failing gcdump test on the CI. The regression was potentially introduced in the #89726 by enabling the test on apple mobile platforms.

Author: kotlarmilos
Assignees: kotlarmilos
Labels:

area-Infrastructure-mono, os-ios

Milestone: 8.0.0

@lateralusX
Copy link
Member

This occurs in runtime-extra-platforms, right?

@kotlarmilos
Copy link
Member Author

kotlarmilos commented Aug 4, 2023

Correct, the test is failing on apple mobile platforms.

https://dev.azure.com/dnceng-public/public/_build/results?buildId=363106&view=logs&j=da83f4c3-af55-5024-34f6-3ac8d11ac15f

Test log:

[03:12:26.0256760] Launched application 'net.dot.tracing_eventpipe' on 'DNCENGOSX117' with pid 14378
[03:12:26.1444880] 2023-08-04 03:12:26.185 tracing_eventpipe[14378:13727264]   0.0s: ==TEST STARTING==
[03:12:26.1548200] 2023-08-04 03:12:26.198 tracing_eventpipe[14378:13727274]   0.0s: Started sending sentinel events...
[03:12:26.1581380] 2023-08-04 03:12:26.201 tracing_eventpipe[14378:13727276]   0.0s: Connecting to EventPipe...
[03:12:26.2010550] 2023-08-04 03:12:26.244 tracing_eventpipe[14378:13727276]   0.1s: Creating EventPipeEventSource...
[03:12:26.2053600] 2023-08-04 03:12:26.248 tracing_eventpipe[14378:13727276]   0.1s: EventPipeEventSource created
[03:12:26.2137250] 2023-08-04 03:12:26.257 tracing_eventpipe[14378:13727276]   0.1s: Dynamic.All callback registered
[03:12:26.2137700] 2023-08-04 03:12:26.257 tracing_eventpipe[14378:13727276]   0.1s: Running optional trace validator
[03:12:26.2221300] 2023-08-04 03:12:26.264 tracing_eventpipe[14378:13727276]   0.1s: Finished running optional trace validator
[03:12:26.2221690] 2023-08-04 03:12:26.265 tracing_eventpipe[14378:13727276]   0.1s: Starting stream processing...
[03:12:26.3359410] 2023-08-04 03:12:26.379 tracing_eventpipe[14378:13727276]   0.2s: Saw sentinel event
[03:12:26.3359870] 2023-08-04 03:12:26.379 tracing_eventpipe[14378:13727274]   0.2s: Stopped sending sentinel events
[03:12:26.3371410] 2023-08-04 03:12:26.380 tracing_eventpipe[14378:13727264]   0.2s: Starting event generating action...
[03:12:36.3443740] 2023-08-04 03:12:36.387 tracing_eventpipe[14378:13727264]  10.2s: Stopping event generating action
[03:12:36.3456970] 2023-08-04 03:12:36.389 tracing_eventpipe[14378:13727274]  10.2s: Sending StopTracing command...
[03:12:36.4805540] 2023-08-04 03:12:36.524 tracing_eventpipe[14378:13727276]  10.4s: Saw new provider 'Microsoft-DotNETCore-EventPipe'
[03:12:36.4967410] 2023-08-04 03:12:36.540 tracing_eventpipe[14378:13727276]  10.4s: Saw new provider 'Microsoft-Windows-DotNETRuntimeRundown'
[03:12:36.5134950] 2023-08-04 03:12:36.557 tracing_eventpipe[14378:13727274]  10.4s: Finished StopTracing command
[03:12:36.5411160] 2023-08-04 03:12:36.584 tracing_eventpipe[14378:13727276]  10.4s: Stopping stream processing
[03:12:36.5412890] 2023-08-04 03:12:36.585 tracing_eventpipe[14378:13727276]  10.4s: Dropped 0 events
[03:12:36.5414080] 2023-08-04 03:12:36.585 tracing_eventpipe[14378:13727264]  10.4s: Reader task finished
[03:12:36.5415490] 2023-08-04 03:12:36.585 tracing_eventpipe[14378:13727264]  10.4s: Validating optional callback...
[03:12:36.5416170] 2023-08-04 03:12:36.585 tracing_eventpipe[14378:13727264] Test failed due to missing GC heap events.
[03:12:36.5417250] 2023-08-04 03:12:36.585 tracing_eventpipe[14378:13727264] _seenGCStart =            False
[03:12:36.5417550] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264] _seenGCStop =             False
[03:12:36.5417610] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264] _bulkTypeCount =          0
[03:12:36.5419840] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264] _bulkNodeCount =          0
[03:12:36.5420020] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264] _bulkEdgeCount =          0
[03:12:36.5420080] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264] _bulkRootEdgeCount =      0
[03:12:36.5420130] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264] _bulkRootStaticVarCount = 0
[03:12:36.5420220] 2023-08-04 03:12:36.586 tracing_eventpipe[14378:13727264]  10.4s: ==TEST FINISHED: FAILED!==
[03:12:36.5473960] Xamarin.Hosting: Process '14378' exited with exit code 255 or crashing signal .
[03:12:36.5478560] Application 'net.dot.tracing_eventpipe' terminated (with exit code '255' and/or crashing signal ').

@lateralusX
Copy link
Member

lateralusX commented Aug 4, 2023

@kotlarmilos, OK, at some point I would like to investigate why, I have run the underlying gcdump mechanisms on maccatalyst (amd64,arm64), iosSimulator (amd64,arm64), tvosSimulator (amd64, arm64) and Android and we can collect dumps, so not sure why that CI test would fail. Do you know how I can run the specific runtime test on tvosSimulator for example?

@kotlarmilos
Copy link
Member Author

Repro steps:

  1. Build the runtime
./build.sh -ci -arch x64 -os tvossimulator  -s mono+libs -c Release
  1. Build and AOT compile the tests
./src/tests/build.sh  ci os tvossimulator x64 Release -mono tree tracing/eventpipe /p:ArchiveTests=true /p:DevTeamProvisioning=- /p:RunAOTCompilation=true /p:MonoForceInterpreter=true /p:LibrariesConfiguration=Release
  1. Run the app bundle using Xcode using the following environment variables
MONO_APPLE_APP_ENTRY_POINT_LIB_NAME = 'testdir-gcdump/gcdump.dll'
MONO_APPLE_APP_ASSEMBLY_LOAD_PREFIX = 'testdir-gcdump'

Note: You may need to provide a correct path to the cross compiler in the AppleBuild.targets.

@lateralusX
Copy link
Member

@kotlarmilos Do you think you can file an issue and assign it to me that we have disabled the test on Apple Mobile platforms? Just to make sure we don't forget to fix it at some future point in time?

@kotlarmilos kotlarmilos merged commit 04e8c7f into dotnet:main Aug 4, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants