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

Ibc merge failure logging #5277

Merged
merged 2 commits into from
Apr 17, 2020
Merged

Ibc merge failure logging #5277

merged 2 commits into from
Apr 17, 2020

Conversation

tmat
Copy link
Member

@tmat tmat commented Apr 16, 2020

Description

Port #4678 and #4714 to 3.x

Customer Impact

Allows us to inspect IbcMerge input files in artifacts in case of failure.

Regression

No

Risk

Small.

Workarounds

None.

tmat and others added 2 commits April 16, 2020 12:20
* Remove obsolete properties

* Copy IbcMerge input files to log/ngen dir on failure
Currently an IBCMerge error is thrown even though the tool doesn't even
run. Adding a condition to only do error handling if IBC ran. As I
already was there, I also conditioned the Message task.
@tmat
Copy link
Member Author

tmat commented Apr 16, 2020

@markwilkie

@tmat tmat merged commit 0226543 into dotnet:release/3.x Apr 17, 2020
@tmat tmat deleted the Ibc3x branch April 17, 2020 22:50
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

Successfully merging this pull request may close these issues.

3 participants