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

Show info if the test build is canceled due to timeout #569

Closed
llxia opened this issue Oct 12, 2021 · 11 comments
Closed

Show info if the test build is canceled due to timeout #569

llxia opened this issue Oct 12, 2021 · 11 comments

Comments

@llxia
Copy link
Contributor

llxia commented Oct 12, 2021

In Jenkins, a test build result can be FAILURE (not ABORT) if it is canceled due to timeout.

20:14:00.415  LT  20:28:06.223 - Completed 81.4%. Number of tests started=694633 (+118121)
Cancelling nested steps due to timeout
20:14:01.902  Sending interrupt signal to process
20:14:03.647  STF 20:28:11.977 - **FAILED** Process LT  ended with exit code (143) and not the expected exit code/s (0)
20:14:03.647  STF 20:28:11.978 - Monitoring Report Summary:
20:14:03.647  STF 20:28:11.978 -   o Process LT  ended with exit code (143) and not the expected exit code/s (0)
20:14:03.647  STF 20:28:11.978 - Killing processes: LT 
20:14:03.648  STF 20:28:11.978 -   o Process LT  pid 1708 is not running
20:14:03.648  **FAILED** at step 1 (Run Mauve load test). Expected return value=0 Actual=1 at /home/jenkins/workspace/Test_openjdk8_j9_special.system_aarch64_linux/aqa-tests/TKG/../TKG/output_16334948943599/MauveSingleInvocLoad_special_5m_26/20211006-202329-MauveSingleInvocLoad/execute.pl line 94.
...
[Pipeline] }
[Pipeline] // timeout
[Pipeline] echo
Exception: org.jenkinsci.plugins.workflow.steps.FlowInterruptedException
[Pipeline] }
[Pipeline] // node
[Pipeline] }
[Pipeline] // stage
[Pipeline] End of Pipeline
Finished: FAILURE

When this happens, TRSS won't be able to get the test summary because the test build is incomplete. Currently, in Grid view, TRSS shows the following:

image

In the tooltip, we should show the info for the build that is canceled due to timeout.

@Mercon55
Copy link
Contributor

Hi, I would like to work on this

@llxia
Copy link
Contributor Author

llxia commented Oct 14, 2021

Sorry, this should be in test-tools repo. I will move the issue.

@llxia llxia transferred this issue from adoptium/aqa-tests Oct 14, 2021
@llxia
Copy link
Contributor Author

llxia commented Oct 14, 2021

@Mercon55 Thanks for your interest. However, there are still some details that need to be discussed about this issue. It is not ready to be worked on.
I will create a couple of good first issues next week, please feel free to pick them up.

@Mercon55
Copy link
Contributor

Mercon55 commented Oct 14, 2021 via email

llxia added a commit to llxia/aqa-tests that referenced this issue Oct 19, 2021
@llxia
Copy link
Contributor Author

llxia commented Oct 19, 2021

On second thought, we should not manipulate the build result in TRSS. Jenkins builds should report the correct results.

@Mercon55
Copy link
Contributor

Okay then, can I work on it then using Jenkins build?

@llxia
Copy link
Contributor Author

llxia commented Oct 20, 2021

Thank you. We already have a PR for it.

@Mercon55
Copy link
Contributor

Mercon55 commented Oct 20, 2021 via email

@llxia
Copy link
Contributor Author

llxia commented Oct 20, 2021

I just created #572 . @Mercon55 you can give it a try if you are interested.

@Mercon55
Copy link
Contributor

Mercon55 commented Oct 20, 2021 via email

llxia added a commit to llxia/aqa-tests that referenced this issue Oct 26, 2021
llxia added a commit to llxia/aqa-tests that referenced this issue Oct 26, 2021
llxia added a commit to llxia/aqa-tests that referenced this issue Oct 26, 2021
smlambert pushed a commit to adoptium/aqa-tests that referenced this issue Oct 26, 2021
@llxia
Copy link
Contributor Author

llxia commented Oct 27, 2021

The problem should be resolved via adoptium/aqa-tests#3035. Close issue.

@llxia llxia closed this as completed Oct 27, 2021
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

No branches or pull requests

2 participants