Add metrics for server cert generation and server start
Summary: Will provide some visibility on error rates. Reviewed By: passy Differential Revision: D13671492 fbshipit-source-id: 60971a7e8163c58dd4d6540e57e05b38c852744e
This commit is contained in:
committed by
Facebook Github Bot
parent
7120bf1201
commit
a254cef98d
@@ -71,6 +71,7 @@ export default class Server extends EventEmitter {
|
|||||||
this.insecureServer = this.startServer(INSECURE_PORT);
|
this.insecureServer = this.startServer(INSECURE_PORT);
|
||||||
return;
|
return;
|
||||||
});
|
});
|
||||||
|
recordSuccessMetric(this.initialisePromise, 'initializeServer');
|
||||||
return this.initialisePromise;
|
return this.initialisePromise;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -21,6 +21,7 @@ const tmp = require('tmp');
|
|||||||
const tmpFile = promisify(tmp.file);
|
const tmpFile = promisify(tmp.file);
|
||||||
const tmpDir = promisify(tmp.dir);
|
const tmpDir = promisify(tmp.dir);
|
||||||
import iosUtil from '../fb-stubs/iOSContainerUtility';
|
import iosUtil from '../fb-stubs/iOSContainerUtility';
|
||||||
|
import {recordSuccessMetric} from './metrics';
|
||||||
|
|
||||||
// Desktop file paths
|
// Desktop file paths
|
||||||
const os = require('os');
|
const os = require('os');
|
||||||
@@ -77,7 +78,10 @@ export default class CertificateProvider {
|
|||||||
constructor(server: Server, logger: LogManager) {
|
constructor(server: Server, logger: LogManager) {
|
||||||
this.logger = logger;
|
this.logger = logger;
|
||||||
this.adb = adb.createClient();
|
this.adb = adb.createClient();
|
||||||
this.certificateSetup = this.ensureServerCertExists();
|
this.certificateSetup = recordSuccessMetric(
|
||||||
|
this.ensureServerCertExists(),
|
||||||
|
'ensureServerCertExists',
|
||||||
|
);
|
||||||
this.server = server;
|
this.server = server;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user