Update-DocsMsPackages.ps1 Explicitly exits 0 #1977
Conversation
|
The following pipelines have been queued for testing: |
|
@scbedd -- Correct. The last execution of an external command sets
|
|
This is why I've been getting all those docindex failed emails...:D |
* Fix variable names for $remainingPackages * Route stderr to stdout to prevent errors in mvn from breaking the docs onboarding process * Stringify potential error objects * Out-String * Redirect artifact download output * Out-Null * Test exit 1 * Test powershell task behavior * Test powershell task behavior * Test powershell task behavior with an explicit exit 0 * Sparse checkout Java repo * Revert Update-DocsMsPackages.ps1. Those changes are coming in Azure/azure-sdk-tools#1977 * Revert changes to Language-Settings.ps1 to use intended variable changes only * Tweak sparse checkout * Use a single checkout: none * Use default checkout * Revert to original SkipDefaultCheckout: true * Update eng/scripts/Language-Settings.ps1 Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]>
|
/azp run azure-sdk-tools - sync - eng-common |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
The following pipelines have been queued for testing: |
|
/azp run azure-sdk-tools - sync - eng-common |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
The following pipelines have been queued for testing: |
…ate up to a failure of the pipeline
danieljurek
force-pushed
the
update-docsmspackages-explicit-exit-code
branch
from
6125b91 to
31371ed
Compare
|
The following pipelines have been queued for testing: |
|
Hello @azure-sdk! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
ghost
merged commit 
Exit 0 so DevOps doesn't fail the build when the last command called by the domain-specific function exited with a non-zero exit code.
Specifically this mitigates issues in Java where the docindex pipeline fails because an inner
mvncommand failed during validation. The whole script succeeds because we properly handle errors but PowerShell just reads$LASTEXITCODE