From 9e82ab0890c5cc8c7fb95362c3f7edbcaad0cf29 Mon Sep 17 00:00:00 2001
From: Tim Alby <timothee.alby@gmail.com>
Date: Fri, 10 Apr 2020 12:28:48 +0200
Subject: [PATCH] add metrics for EPIPE errors

Co-Authored-By: Jakob Ackermann <jakob.ackermann@overleaf.com>
---
 app/js/DockerRunner.js | 3 +++
 1 file changed, 3 insertions(+)

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