Skip to content

Commit e19ee1f

Browse files
kboomGrzegorz Gurgul (from Dev Box)
and
Grzegorz Gurgul (from Dev Box)
authored
Add customer intelligence event for failed redirects for DownloadBuildArtifacts@v0 task (#19777)
This adds a new CI event logged when the redirect to content stitcher is enabled and it fails all attempts. --------- Co-authored-by: Grzegorz Gurgul (from Dev Box) <[email protected]>
1 parent 0346829 commit e19ee1f

File tree

14 files changed

+6435
-5618
lines changed

14 files changed

+6435
-5618
lines changed

Tasks/DownloadBuildArtifactsV0/_buildConfigs/Node20/Tests/package-lock.json

+11-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Tasks/DownloadBuildArtifactsV0/_buildConfigs/Node20/package-lock.json

+3,197-2,798
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Tasks/DownloadBuildArtifactsV0/main.ts

+1
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,7 @@ async function main(): Promise<void> {
362362
ErrorDisplayMode.ShowWarning
363363
).catch((reason) => {
364364
console.log(tl.loc("FollowingDownloadRedirectFailed", reason));
365+
publishEvent('download-redirect', { "redirectLimit": retryRedirectLimitDownload, "err": JSON.stringify(reason, Object.getOwnPropertyNames(reason)) });
365366
const handlerConfig: IContainerHandlerConfig = { ...config, endpointUrl, templatePath, handler, preferRedirect: false };
366367
const downloadHandler: DownloadHandlerContainer = new DownloadHandlerContainer(handlerConfig);
367368
const fallbackDownloadPromise: Promise<models.ArtifactDownloadTicket[]> = executeWithRetries(

Tasks/DownloadBuildArtifactsV0/task.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"author": "Microsoft Corporation",
1010
"version": {
1111
"Major": 0,
12-
"Minor": 238,
12+
"Minor": 239,
1313
"Patch": 0
1414
},
1515
"groups": [

Tasks/DownloadBuildArtifactsV0/task.loc.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"author": "Microsoft Corporation",
1010
"version": {
1111
"Major": 0,
12-
"Minor": 238,
12+
"Minor": 239,
1313
"Patch": 0
1414
},
1515
"groups": [
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
Default|0.238.0
2-
Node20_229_3|0.238.1
1+
Default|0.239.0
2+
Node20_229_3|0.239.1

_generated/DownloadBuildArtifactsV0/main.ts

+1
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,7 @@ async function main(): Promise<void> {
362362
ErrorDisplayMode.ShowWarning
363363
).catch((reason) => {
364364
console.log(tl.loc("FollowingDownloadRedirectFailed", reason));
365+
publishEvent('download-redirect', { "redirectLimit": retryRedirectLimitDownload, "err": JSON.stringify(reason, Object.getOwnPropertyNames(reason)) });
365366
const handlerConfig: IContainerHandlerConfig = { ...config, endpointUrl, templatePath, handler, preferRedirect: false };
366367
const downloadHandler: DownloadHandlerContainer = new DownloadHandlerContainer(handlerConfig);
367368
const fallbackDownloadPromise: Promise<models.ArtifactDownloadTicket[]> = executeWithRetries(

_generated/DownloadBuildArtifactsV0/task.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"author": "Microsoft Corporation",
1010
"version": {
1111
"Major": 0,
12-
"Minor": 238,
12+
"Minor": 239,
1313
"Patch": 0
1414
},
1515
"groups": [
@@ -321,7 +321,7 @@
321321
}
322322
],
323323
"_buildConfigMapping": {
324-
"Default": "0.238.0",
325-
"Node20_229_3": "0.238.1"
324+
"Default": "0.239.0",
325+
"Node20_229_3": "0.239.1"
326326
}
327327
}

_generated/DownloadBuildArtifactsV0/task.loc.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"author": "Microsoft Corporation",
1010
"version": {
1111
"Major": 0,
12-
"Minor": 238,
12+
"Minor": 239,
1313
"Patch": 0
1414
},
1515
"groups": [
@@ -321,7 +321,7 @@
321321
}
322322
],
323323
"_buildConfigMapping": {
324-
"Default": "0.238.0",
325-
"Node20_229_3": "0.238.1"
324+
"Default": "0.239.0",
325+
"Node20_229_3": "0.239.1"
326326
}
327327
}

_generated/DownloadBuildArtifactsV0_Node20/Tests/package-lock.json

+11-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

_generated/DownloadBuildArtifactsV0_Node20/main.ts

+1
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,7 @@ async function main(): Promise<void> {
362362
ErrorDisplayMode.ShowWarning
363363
).catch((reason) => {
364364
console.log(tl.loc("FollowingDownloadRedirectFailed", reason));
365+
publishEvent('download-redirect', { "redirectLimit": retryRedirectLimitDownload, "err": JSON.stringify(reason, Object.getOwnPropertyNames(reason)) });
365366
const handlerConfig: IContainerHandlerConfig = { ...config, endpointUrl, templatePath, handler, preferRedirect: false };
366367
const downloadHandler: DownloadHandlerContainer = new DownloadHandlerContainer(handlerConfig);
367368
const fallbackDownloadPromise: Promise<models.ArtifactDownloadTicket[]> = executeWithRetries(

0 commit comments

Comments
 (0)