Reporting changes#7646
Merged
Merged
Conversation
This reverts commit 9159815.
This reverts commit 47b0341.
…into reporting-changes # Conflicts: # tests/UnitTests.proj
AlitzelMendez
approved these changes
Aug 2, 2021
missymessa
approved these changes
Aug 2, 2021
missymessa
left a comment
missymessa
left a comment
Member
There was a problem hiding this comment.
LGTM
I think this is good :)
Contributor
|
This PR should not go in, it was based on a misunderstanding. Closing. |
Member
|
@markwilkie did you mean to close it? We can revert the iOS queue changes, but the rest of the changes are necessary for the test retries, I believe. |
Contributor
|
I don't think any reverts should happen, and we should not turn off the ios queues. |
Contributor
|
I see....this PR is MORE than just ios queues being turned off. (duh) Yea, let's be sure to NOT turn off the IOS queues but get the PR working. (reopening) |
This reverts commit 2fd0632.
Contributor
Author
|
I reverted the removal of that queue. :-) |
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reintroduce #7383
This switches arcade from using a custom reporter, to using a reporter more tightly integrated with helix. That reporter has the capacity to handle local test retrying and reporting to Azure DevOps for "flaky" tests.
To enable the retry, a file "eng/test-configuration.json" must be created following the documented schema.