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

trex_astf_client: use proper to_json_str call #1054

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bzalkilani
Copy link

The profile object returned from ASTFProfile.load
can be of type ASTFProfileLight if the passed
"profile" is a path to a json profile file.

ASTFProfileLight has a different to_json_str
signature, and calling the method assuming
the returned profile object is of type
ASTFProfile will lead to failures in this
case.

The profile object returned from ASTFProfile.load
can be of type ASTFProfileLight if the passed
"profile" is a path to a json profile file.

ASTFProfileLight has a different to_json_str
signature, and calling the method assuming
the returned profile object is of type
ASTFProfile will lead to failures in this
case.

Signed-off-by: bzalkilani <[email protected]>
@trex-bot
Copy link

Checked SHA: 909fec4
Status: FAILURE
Link to job: http://81.218.86.50:8080/job/trex_build/802/

@bzalkilani
Copy link
Author

@hhaim can you please help me figure out why the CI fails so I can fix it?

@hhaim
Copy link
Contributor

hhaim commented Aug 24, 2023

@bzalkilani it seems as our issue.

@ybrustin
Copy link
Contributor

recheck

@trex-bot
Copy link

Checked SHA: 909fec4
Status: SUCCESS ✔️
Link to job: http://81.218.86.50:8080/job/trex_build/803/

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