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

[Feature] Include parallelIndex in the JSON report #33036

Open
arjunattam opened this issue Oct 10, 2024 · 1 comment · May be fixed by #34740
Open

[Feature] Include parallelIndex in the JSON report #33036

arjunattam opened this issue Oct 10, 2024 · 1 comment · May be fixed by #34740
Labels
open-to-a-pull-request The feature request looks good, we are open to reviewing a PR

Comments

@arjunattam
Copy link
Contributor

Hi team! 😁 This is a follow-up to #19570 and #19519

My goal is to use the JSON report output to build a visualization that shows worker utilization (similar to the image in #19519). I believe this can be achieved by adding the parallelIndex property to JSONReportTestResult. Is there a different way to achieve the goal?

export interface JSONReportTestResult {
  workerIndex: number;
+ parallelIndex: number;
  status: TestStatus | undefined;
  ...
}

If this change sounds good, I can submit a PR. Thanks!

@pavelfeldman pavelfeldman added the open-to-a-pull-request The feature request looks good, we are open to reviewing a PR label Oct 10, 2024
@deshmukh-neel
Copy link

I'm working on this issue with Christopher and Marcelo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
open-to-a-pull-request The feature request looks good, we are open to reviewing a PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants