This repository has been archived by the owner on Oct 24, 2023. It is now read-only.
fix: Expose error details in Windows CSE #3159
Merged
Merged
Conversation
This file contains 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
/azp run e2e |
No pipelines are associated with this pull request. |
/azp run e2e-pr |
No pipelines are associated with this pull request. |
/azp run pr-e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
marosset
approved these changes
Apr 29, 2020
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks for looking into this.
Even if we can can't get an error-code having the message returned by the ARM operation is a big improvement!
Codecov Report
@@ Coverage Diff @@
## master #3159 +/- ##
=======================================
Coverage 71.13% 71.13%
=======================================
Files 147 147
Lines 25670 25670
=======================================
Hits 18261 18261
Misses 6272 6272
Partials 1137 1137
Continue to review full report at Codecov.
|
AbelHu
added a commit
to AbelHu/aks-engine
that referenced
this pull request
May 15, 2020
4 tasks
andyliuliming
added a commit
that referenced
this pull request
May 15, 2020
* build: fetch k8s Windows .zip from kubernetesartifacts storage (#2655) * build: fetch k8s Windows .zip from kubernetesartifacts storage * docs: remove build-windows-k8s * build: remove unsupported v1.15.5 from Windows VHD * fix: generate windows .zip URL correctly for download * test: update UT expectations for windowszip * fix: Expose error details in Windows CSE (#3159) * feat: Updating Windows VHDs to include May patches (#3263) * chore: update zip name for mooncake mirror (#3237) * chore: SinglePlacementGroup=false if VMSS + SLB (#3054) * fix: the os disk size restriction is changed in azure. (#3043) * refactor: Create signable PS scripts (#3015) * feat: Refactor PS scripts to be static * fix: don't hardcode pause image for containerD on Windows (#3158) * chore: update Windows pause image (#3210) * fix: USER_ASSIGNED_IDENTITY_ID is empty in azure.json (#3254) Co-authored-by: Matt Boersma <Matt.Boersma@microsoft.com> Co-authored-by: Mark Rossetti <marosset@microsoft.com> Co-authored-by: tamilmani1989 <tamilmani1989@gmail.com> Co-authored-by: Jack Francis <jackfrancis@gmail.com> Co-authored-by: Liming Liu <andyliuliming@outlook.com> Co-authored-by: Kalya Subramanian <42158129+ksubrmnn@users.noreply.github.com>
4 tasks
AbelHu
added a commit
to AbelHu/aks-engine
that referenced
this pull request
May 16, 2020
xuto2
pushed a commit
that referenced
this pull request
May 16, 2020
* fix: Expose error details in Windows CSE (#3159) * feat: Updating Windows VHDs to include May patches (#3263) * fix: the os disk size restriction is changed in azure. (#3043) * refactor: Create signable PS scripts (#3015) * feat: Refactor PS scripts to be static * fix: use full userAssignedID reference in ARM template (#3116) * fix: don't hardcode pause image for containerD on Windows (#3158) * chore: update Windows pause image (#3210) * fix: USER_ASSIGNED_IDENTITY_ID is empty in azure.json (#3254) Co-authored-by: Mark Rossetti <marosset@microsoft.com> Co-authored-by: Liming Liu <andyliuliming@outlook.com> Co-authored-by: Kalya Subramanian <42158129+ksubrmnn@users.noreply.github.com> Co-authored-by: Jack Francis <jackfrancis@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Reason for Change:
Expose error details in Windows CSE for better troubleshooting
Issue Fixed:
Fix #3156
Requirements:
Notes: