Commit 7da4afb1 authored by Robert Knight's avatar Robert Knight

Remove Jenkins build debugging code

Remove the verbose logging that was enabled for Chrome in Jenkins builds
a while ago. This was used to debug a problem with sporadic failures
that we haven't seen too much recently. If it becomes a significant
problem in future we'll probably take the opportunity to shift the whole
client build to GitHub Actions instead.
parent 4fa5788c
...@@ -17,30 +17,6 @@ process.env.CHROME_BIN = require('puppeteer').executablePath(); ...@@ -17,30 +17,6 @@ process.env.CHROME_BIN = require('puppeteer').executablePath();
if (process.env.RUNNING_IN_DOCKER) { if (process.env.RUNNING_IN_DOCKER) {
chromeFlags.push('--no-sandbox'); chromeFlags.push('--no-sandbox');
// Enable debug logging from Chrome to help track down a cause of frequent
// build failures in Jenkins. The log files are written to `chrome_debug.log`
// in the workspace for the current build.
chromeFlags.push('--enable-logging');
chromeFlags.push('--v=1');
process.env.CHROME_LOG_FILE = path.resolve(
__dirname + '/../',
'chrome_debug.log'
);
// Enable even more debug logging from Chrome to help track down build
// failures in Jenkins. This generates a large (~40MB+) JSON file detailing
// events that happen during browser startup.
//
// See https://www.chromium.org/developers/how-tos/trace-event-profiling-tool/recording-tracing-runs#TOC-Capturing-chrome-desktop-startup
const traceFile = path.resolve(__dirname + '/../', 'chrome_trace.json');
chromeFlags.push('--trace-startup');
chromeFlags.push('--trace-startup-duration=7');
chromeFlags.push(`--trace-startup-file=${traceFile}`);
// Log test details as they are executed so we can tell where test execution
// got to if a failure happens.
mochaOutputMode = 'full';
// Disable `/dev/shm` usage as this can cause Chrome to fail to load large // Disable `/dev/shm` usage as this can cause Chrome to fail to load large
// HTML pages, such as the one Karma creates with all the tests loaded. // HTML pages, such as the one Karma creates with all the tests loaded.
// //
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment