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

Support of directory traces other than CTF #287

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

bhufmann
Copy link
Collaborator

The trace detection algorithm checks recursively for CTF traces. If no CTF trace are found under the root directory, then use the root directory as trace directory. The trace server back-end will validate if this path points to a valid trace or not.

With this it's now possible to open traces that are directories other than CTF.

Note that only a single trace can be opened in the experiment and not set of traces, which is currently only supported with CTF traces.

Examples trace type: uftrace

Signed-off-by: Bernd Hufmann [email protected]

The trace detection algorithm checks recursively for CTF traces. If no
CTF trace are found under the root directory, then use the root
directory as trace directory. The trace server back-end will validate
if this path points to a valid trace or not.

With this it's now possible to open traces that are directories other
than CTF.

Note that only a single trace can be opened in the experiment and not
set of traces, which is currently only supported with CTF traces.

Examples trace type: uftrace

Signed-off-by: Bernd Hufmann <[email protected]>
Copy link
Collaborator

@MatthewKhouzam MatthewKhouzam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix, instead, can we have the function isCtf renamed to isKnownDirTrace

and line 202 would be if (child[0] === 'metadata' || child[0] === 'info') {

That would allow us to recurse on uftraces too. I think it would allow CTF and uftrace to play well together. Maybe both changes are needed.

@MatthewKhouzam
Copy link
Collaborator

On second though, info just gave me some false positives, the name is not very unique. I don't know, Bernd, what do you think?

@yskelg
Copy link

yskelg commented Nov 28, 2024

Thank you @bhufmann @MatthewKhouzam for the great works!

@namhyung
Copy link

Thanks for working on this. For checking uftrace data directory, you can use "default.opts" file.

@bhufmann
Copy link
Collaborator Author

bhufmann commented Nov 28, 2024

Thanks for working on this. For checking uftrace data directory, you can use "default.opts" file.

Thanks for the suggestion. This would be useful if we'd like to support experiments with multiple uftrace traces and/or a combination of CTF traces and uftraces. This is out-of-scope of this PR and should be handled separately.

@bhufmann
Copy link
Collaborator Author

On second though, info just gave me some false positives, the name is not very unique. I don't know, Bernd, what do you think?

files with name info are more common. I noticed that each git rebository has such a file underneath .git. Experiment support with multiple uftraces and/or combination of CTF traces and uftrace is out-of-scope of this PR and should be handled separately.

@bhufmann bhufmann merged commit 7c551f8 into eclipse-cdt-cloud:master Nov 28, 2024
6 checks passed
@bhufmann bhufmann deleted the directory_traces branch November 28, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants