Skip to content

Allow PostAnalysis@2 task to continue on error for Windows_Packaging_CPU_x86_default#14332

Merged
rui-ren merged 2 commits into
mainfrom
adrianl/packaging-pipeline-postanalysis-workaround
Jan 19, 2023
Merged

Allow PostAnalysis@2 task to continue on error for Windows_Packaging_CPU_x86_default#14332
rui-ren merged 2 commits into
mainfrom
adrianl/packaging-pipeline-postanalysis-workaround

Conversation

@adrianlizarraga
Copy link
Copy Markdown
Contributor

@adrianlizarraga adrianlizarraga commented Jan 18, 2023

Description

Allows the PostAnalysis@2 task for windows CI jobs to continue even if an error is encountered.

Motivation and Context

This is a temporary workaround that enables the Windows_Packaging_CPU_x86_default job within the Zip-Nuget-Java-NodeJS packaging pipeline to finish. A recent update to dotnet 6 has broken the PostAnalysis task for this job.

This task was originally added by #13694

@snnn
Copy link
Copy Markdown
Contributor

snnn commented Jan 18, 2023

Have we contacted the authors of this ADO task?

@mszhanyi
Copy link
Copy Markdown
Contributor

mszhanyi commented Jan 18, 2023

Have we contacted the authors of this ADO task?

https://stackoverflow.microsoft.com/questions/338792.
But no response yet.

@adrianlizarraga adrianlizarraga marked this pull request as ready for review January 18, 2023 09:19
@adrianlizarraga adrianlizarraga requested review from a team, jchen351 and mszhanyi January 18, 2023 09:19
@adrianlizarraga
Copy link
Copy Markdown
Contributor Author

I'm not entirely sure this should be merged, but it's here in case we need to unblock.

@rui-ren rui-ren merged commit a491f33 into main Jan 19, 2023
@rui-ren rui-ren deleted the adrianl/packaging-pipeline-postanalysis-workaround branch January 19, 2023 03:54
mszhanyi added a commit that referenced this pull request Jan 20, 2023
mszhanyi added a commit that referenced this pull request Jan 21, 2023
#14375)

…ckaging_CPU_x86_default (#14332)"

This reverts commit a491f33.

### Description


### Motivation and Context
It looks an ADO issue.
Now, it's recovered.
It could be reenabled.
rui-ren pushed a commit that referenced this pull request Jan 27, 2023
#14375)

…ckaging_CPU_x86_default (#14332)"

This reverts commit a491f33.

### Description


### Motivation and Context
It looks an ADO issue.
Now, it's recovered.
It could be reenabled.
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.

4 participants