Skip to content

Setup CI to use Flight Recorder #4705

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

Open
SeanTAllen opened this issue May 7, 2025 · 0 comments
Open

Setup CI to use Flight Recorder #4705

SeanTAllen opened this issue May 7, 2025 · 0 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@SeanTAllen
Copy link
Member

We recently added runtime tracing and flight recording to the Pony runtime. When discussing #4702, we came to a decision to add flight recorder to most of our CI tests.

For standard PR tests, we want to build with flight recorder for all tests. Additionally, we want to add a new "use" test that is for "vanilla" which would be building without any use definitions to make sure that works. The runtime tracing use test that currently exists will be removed as all our primary PR tests will be exercising runtime tracing. The goal of the vanilla use test(s) is to make sure we don't lose basic "it compiles" coverage. We have our use jobs run tests but that is mostly "extra". For use jobs the primary goal is to verify compilation is successful. We doubt that any use option will break tests but it is always possible.

For stress tests, we want to build with flight recorder for all the stress tests.

For all flight recorder setup we want to dump either stderr or stdout so that we can get additional information from the CI logs without having to setup grabbing file based assets and uploading somewhere.

For all tests that are running "in the debugger", we want to continue to do so.

@SeanTAllen SeanTAllen added help wanted Extra attention is needed good first issue Good for newcomers labels May 7, 2025
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label May 7, 2025
@jemc jemc removed the discuss during sync Should be discussed during an upcoming sync label May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants