Merge pull request #496 from Financial-Times/remove-graphite-healthch…

…ecks chore: Remove Graphite healthchecks
Financial-Times · Sep 5, 2024 · 7c8a766 · 7c8a766
2 parents a82f970 + 676cbd2
commit 7c8a766
Show file tree

Hide file tree

Showing 9 changed files with 3 additions and 93 deletions.
diff --git a/health/archive-dl-error-spike.js b/health/archive-dl-error-spike.js
diff --git a/health/article-dl-error-spike.js b/health/article-dl-error-spike.js
diff --git a/health/article-dl-no-data.js b/health/article-dl-no-data.js
diff --git a/health/dl-error-spikes.js b/health/dl-error-spikes.js
diff --git a/health/error-spikes.js b/health/error-spikes.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "ft-next-syndication-api",
   "description": "Next Syndication API",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "private": true,
   "dependencies": {
     "@dotcom-reliability-kit/crash-handler": "^2.1.1",

diff --git a/server/app-dl.js b/server/app-dl.js
@@ -32,8 +32,7 @@ const app = module.exports = express({
 	withBackendAuthentication: false,
 	withFlags: true,
 	healthchecks: [
-		require('../health/dl-error-spikes'),
-	]// this healthcheck probably needs to be removed
+	]
 });
 
 const middleware = [

diff --git a/server/app.js b/server/app.js
@@ -33,10 +33,6 @@ const app = module.exports = express({
 		require('../health/db-backups'),
 		require('../health/db-sync-state'),
 		require('../health/sqs'),
-		require('../health/error-spikes'),
-		require('../health/article-dl-no-data'),
-		require('../health/article-dl-error-spike'),
-		require('../health/archive-dl-error-spike'),
 	],
 	errorRateHealthcheck: {
 		severity: 2