Craftadoc/clsi
2
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

add metrics for EPIPE errors

Browse Source 
Co-Authored-By: Jakob Ackermann <jakob.ackermann@overleaf.com>
This commit is contained in:
Tim Alby
2020-04-10 12:28:48 +02:00
parent e3da458b37
commit 9e82ab0890
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/js/DockerRunner.js
View File

@@ -26,6 +26,7 @@ const LockManager = require('./DockerLockManager')
const fs = require('fs') const fs = require('fs')
const Path = require('path') const Path = require('path')
const _ = require('underscore') const _ = require('underscore')
const metrics = require('metrics-sharelatex')
logger.info('using docker runner') logger.info('using docker runner')
@@ -410,6 +411,7 @@ module.exports = DockerRunner = {
const callbackWithRetry = error => { const callbackWithRetry = error => {
if (error.message.match(/EPIPE/)) { if (error.message.match(/EPIPE/)) {
metrics.inc('container-inspect-epipe-retry')
return inspectContainer(container, callback) return inspectContainer(container, callback)
} }
callback(error) callback(error)
@@ -420,6 +422,7 @@ module.exports = DockerRunner = {
if ((error != null ? error.statusCode : undefined) === 404) { if ((error != null ? error.statusCode : undefined) === 404) {
return createAndStartContainer() return createAndStartContainer()
} else if (error != null) { } else if (error != null) {
metrics.inc('container-inspect-epipe-error')
logger.err( logger.err(
{ container_name: name, error }, { container_name: name, error },
'unable to inspect container to start' 'unable to inspect container to start'