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.

Sign up for GitHub

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

Jump to bottom

Enhancement: Organize Release Report summary by Platform Parent/Child jobs #501

Closed
smlambert opened this issue Jul 18, 2021 · 0 comments · Fixed by #518
Closed

Enhancement: Organize Release Report summary by Platform Parent/Child jobs #501

smlambert opened this issue Jul 18, 2021 · 0 comments · Fixed by #518
Assignees
@llxia
Labels
enhancement New feature or request

Comments

@smlambert
Copy link
Contributor

Currently the Release summary report feature lists out all test job failing without any particular order. It would be very useful to order the report by platform, and in the case of running in PARALLEL=dynamic, having the test job parent be listed directly above its children.

Snippet of current ordering can be seen at: adoptium/aqa-tests#2754

It would be better to group results by platform, and then within that, by test group, where the test groups that are run in parallel, have their children jobs listed directly below the parent.

Example snippet, the arm_linux _0, _1, _2 jobs would be listed in order under the arm_linux parent, where now there is a solaris job mixed in, and the arm_linux _0 child is listed much later in the report that is produced:
Screen Shot 2021-07-18 at 12 06 36 PM

...

Screen Shot 2021-07-18 at 12 06 54 PM
@smlambert smlambert added the enhancement New feature or request label Jul 18, 2021
@smlambert smlambert mentioned this issue Jul 22, 2021
Closed
@llxia llxia self-assigned this Jul 28, 2021
@smlambert smlambert mentioned this issue Jul 28, 2021
Closed
llxia added a commit to llxia/aqa-test-tools that referenced this issue Aug 11, 2021
@llxia
Enhance Release Summary Report logic
fae3431 
- Use Promise.all() to send queries in parallel to improve the performance
- Remove unnessary queries
- Use params() to handle special characters in parameter values (i.e., test name)
- Add title and Grid view link
- Sort build names in the report, so Version/Impl/Platform and Parent/Child are grouped together
- Handle errors

Resolves: adoptium#501 and adoptium#506

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
@llxia llxia mentioned this issue Aug 11, 2021
Merged
smlambert pushed a commit that referenced this issue Aug 12, 2021
@llxia
Enhance Release Summary Report logic (#518)
a861475 
- Use Promise.all() to send queries in parallel to improve the performance
- Remove unnessary queries
- Use params() to handle special characters in parameter values (i.e., test name)
- Add title and Grid view link
- Sort build names in the report, so Version/Impl/Platform and Parent/Child are grouped together
- Handle errors

Resolves: #501 and #506

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@llxia llxia

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enhance Release Summary Report logic llxia/aqa-test-tools
2 participants
@smlambert @llxia