Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

devops: fix azure function bugs (#4998)

Browse source 
This patch:
- starts processing dashboards serially to avoid hitting node.js azure
  function heap limit
- fixes typo in the new `dashboards_raw` processor
This commit is contained in:
Andrey Lushnikov 2021-01-13 14:15:38 +03:00 committed by GitHub
parent 8316dee44d
commit fcbfbe6fa1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
utils/flakiness-dashboard/processing/dashboard_raw.js
View file

@ -28,7 +28,6 @@ async function processDashboardRaw(context, report) {
SHA: ${report.metadata.commitSHA} SHA: ${report.metadata.commitSHA}
URL: ${report.metadata.runURL} URL: ${report.metadata.runURL}
timestamp: ${report.metadata.commitTimestamp} timestamp: ${report.metadata.commitTimestamp}
added specs: ${addedSpecs}
===== complete in ${Date.now() - timestamp}ms ===== ===== complete in ${Date.now() - timestamp}ms =====
`); `);
} }

11
utils/flakiness-dashboard/processing/index.js
View file

@ -27,10 +27,9 @@ module.exports = async function(context) {
const data = await gunzipAsync(context.bindings.newBlob); const data = await gunzipAsync(context.bindings.newBlob);
const report = JSON.parse(data.toString('utf8')); const report = JSON.parse(data.toString('utf8'));
// Upload report to both dashboards. // Process dashboards one-by-one to limit max heap utilization.
await Promise.all([ await processDashboardRaw(context, report);
processDashboardV1(context, report), await processDashboardV1(context, report);
processDashboardV2(context, report), // Disable V2 dashboard in favor of raw data.
processDashboardRaw(context, report), // await processDashboardV2(context, report);
]);
} }