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

Display phase name in stash dump of a test failure. #5223

Closed
wants to merge 1 commit into from

Conversation

dblock
Copy link
Member

@dblock dblock commented Nov 11, 2022

Signed-off-by: dblock [email protected]

Description

Looking at failures in #5219 it's not immediately obvious whether the failure is in setup/execution/teardown. You can deduce it from the above lines, but it doesn't cost much to display it inline with the dump.

Leaving this as draft, should trigger enough gradle failures to see it be useful/not.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dblock dblock force-pushed the phase-in-test-failure branch from 5d5f485 to c412a11 Compare November 15, 2022 02:12
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dblock dblock force-pushed the phase-in-test-failure branch from c412a11 to 641293b Compare November 15, 2022 18:49
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dblock
Copy link
Member Author

dblock commented Nov 15, 2022

I'm just waiting for those S3 YAML tests to fail :) too lazy to write a long loop

@dblock dblock force-pushed the phase-in-test-failure branch from 641293b to 07ba39a Compare November 16, 2022 12:57
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dblock dblock force-pushed the phase-in-test-failure branch from 07ba39a to 87e4605 Compare November 16, 2022 18:49
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dblock dblock force-pushed the phase-in-test-failure branch from 87e4605 to 626c053 Compare November 17, 2022 14:33
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@dblock dblock force-pushed the phase-in-test-failure branch from 626c053 to 11e97c2 Compare November 17, 2022 17:54
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: FAILURE ❌
  • FAILURES:

@dblock dblock force-pushed the phase-in-test-failure branch from 11e97c2 to 4915f57 Compare November 17, 2022 22:36
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #5223 (4915f57) into main (7aa615f) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main    #5223      +/-   ##
============================================
+ Coverage     71.02%   71.04%   +0.01%     
+ Complexity    58161    58156       -5     
============================================
  Files          4704     4704              
  Lines        277401   277401              
  Branches      40166    40166              
============================================
+ Hits         197025   197071      +46     
+ Misses        64204    64154      -50     
- Partials      16172    16176       +4     
Impacted Files Coverage Δ
...t/rest/yaml/OpenSearchClientYamlSuiteTestCase.java 10.05% <0.00%> (ø)
...pensearch/client/cluster/RemoteConnectionInfo.java 0.00% <0.00%> (-68.30%) ⬇️
...a/org/opensearch/client/cluster/SniffModeInfo.java 0.00% <0.00%> (-52.95%) ⬇️
...ava/org/opensearch/action/NoSuchNodeException.java 0.00% <0.00%> (-50.00%) ⬇️
.../opensearch/client/cluster/RemoteInfoResponse.java 61.53% <0.00%> (-38.47%) ⬇️
...org/opensearch/test/engine/MockInternalEngine.java 47.82% <0.00%> (-34.79%) ⬇️
...opensearch/snapshots/SnapshotMissingException.java 28.57% <0.00%> (-28.58%) ⬇️
...rc/main/java/org/opensearch/ingest/IngestInfo.java 51.72% <0.00%> (-27.59%) ⬇️
...earch/client/indices/GetIndexTemplatesRequest.java 50.00% <0.00%> (-23.08%) ⬇️
...adcast/BroadcastShardOperationFailedException.java 55.55% <0.00%> (-22.23%) ⬇️
... and 451 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@dblock
Copy link
Member Author

dblock commented Dec 9, 2022

@reta I believe you've looked at the flaky yaml/REST tests, this might be useful. I'll close it for now.

@dblock dblock closed this Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants