Просмотр исходного кода

Merge branch 'sk-upgrade-metrics'

Shane Kilkelly 9 лет назад
Родитель
Сommit
6a6fdc13a5

+ 0 - 1
services/docstore/app.coffee

@@ -9,7 +9,6 @@ Path       = require "path"
 
 Metrics.initialize("docstore")
 logger.initialize("docstore")
-Metrics.mongodb.monitor(Path.resolve(__dirname + "/node_modules/mongojs/node_modules/mongodb"), logger)
 Metrics.event_loop?.monitor(logger)
 
 app = express()

+ 15 - 1
services/docstore/app/coffee/MongoManager.coffee

@@ -1,4 +1,6 @@
 {db, ObjectId} = require "./mongojs"
+logger = require 'logger-sharelatex'
+metrics = require 'metrics-sharelatex'
 
 module.exports = MongoManager =
 
@@ -67,4 +69,16 @@ module.exports = MongoManager =
 			$set: version: version
 		}, {
 			upsert: true
-		}, callback
+		}, callback
+
+
+[
+	'findDoc',
+	'getProjectsDocs',
+	'getArchivedProjectDocs',
+	'upsertIntoDocCollection',
+	'markDocAsArchived',
+	'getDocVersion',
+	'setDocVersion'
+].map (method) ->
+	metrics.timeAsyncMethod(MongoManager, method, 'mongo.MongoManager', logger)

+ 1 - 1
services/docstore/package.json

@@ -10,7 +10,7 @@
   "dependencies": {
     "settings-sharelatex": "git+https://github.com/sharelatex/settings-sharelatex.git#v1.0.0",
     "logger-sharelatex": "git+https://github.com/sharelatex/logger-sharelatex.git#v1.4.0",
-    "metrics-sharelatex": "git+https://github.com/sharelatex/metrics-sharelatex.git#v1.1.0",
+    "metrics-sharelatex": "git+https://github.com/sharelatex/metrics-sharelatex.git#v1.7.1",
     "mongojs": "0.18.2",
     "express": "~4.1.1",
     "underscore": "~1.6.0",

+ 3 - 1
services/docstore/test/unit/coffee/MongoManagerTests.coffee

@@ -11,6 +11,8 @@ describe "MongoManager", ->
 			"./mongojs":
 				db: @db = { docs: {}, docOps: {} }
 				ObjectId: ObjectId
+			'metrics-sharelatex': {timeAsyncMethod: sinon.stub()}
+			'logger-sharelatex': {log: ()->}
 		@project_id = ObjectId().toString()
 		@doc_id = ObjectId().toString()
 		@callback = sinon.stub()
@@ -151,4 +153,4 @@ describe "MongoManager", ->
 				.should.equal true
 
 		it "should call the callback", ->
-			@callback.called.should.equal true
+			@callback.called.should.equal true