Merge pull request #101 from Financial-Times/graphite-migration

Point Graphite checks at the new system
Financial-Times · Oct 2, 2018 · c475af5 · c475af5
2 parents f783fee + 5ab5ed8
commit c475af5
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/checks/graphiteSpike.check.js b/src/checks/graphiteSpike.check.js
@@ -23,7 +23,7 @@ class GraphiteSpikeCheck extends Check {
 		this.seriesFunction = options.seriesFunction || 'sumSeries';
 		this.summarizeFunction = options.summarizeFunction || 'sum';
 
-		this.ftGraphiteBaseUrl = 'https://graphite-api.ft.com/render/?';
+		this.ftGraphiteBaseUrl = 'https://graphitev2-api.ft.com/render/?';
 		this.ftGraphiteKey = process.env.FT_GRAPHITE_KEY;
 		if (!this.ftGraphiteKey) {
 			throw new Error('You must set FT_GRAPHITE_KEY environment variable');

diff --git a/src/checks/graphiteThreshold.check.js b/src/checks/graphiteThreshold.check.js
@@ -20,7 +20,7 @@ class GraphiteThresholdCheck extends Check {
 
 		this.samplePeriod = options.samplePeriod || '10min';
 
-		this.ftGraphiteBaseUrl = 'https://graphite-api.ft.com/render/?';
+		this.ftGraphiteBaseUrl = 'https://graphitev2-api.ft.com/render/?';
 		this.ftGraphiteKey = process.env.FT_GRAPHITE_KEY;
 		if (!this.ftGraphiteKey) {
 			throw new Error('You must set FT_GRAPHITE_KEY environment variable');

diff --git a/src/checks/graphiteWorking.check.js b/src/checks/graphiteWorking.check.js
@@ -30,7 +30,7 @@ class GraphiteWorkingCheck extends Check {
 		}
 
 		const fromTime = options.time || '-5minutes';
-		this.url = encodeURI(`https://graphite-api.ft.com/render/?target=${this.metric}&from=${fromTime}&format=json`);
+		this.url = encodeURI(`https://graphitev2-api.ft.com/render/?target=${this.metric}&from=${fromTime}&format=json`);
 
 		this.checkOutput = "This check has not yet run";
 	}

diff --git a/test/graphiteSpike.check.spec.js b/test/graphiteSpike.check.spec.js
@@ -44,7 +44,7 @@ describe('Graphite Spike Check', function(){
 				}));
 				check.start();
 				setTimeout(() => {
-					expect(mockFetch.firstCall.args[0]).to.match(/^https:\/\/graphite-api.ft.com\/render\/\?/);
+					expect(mockFetch.firstCall.args[0]).to.match(/^https:\/\/graphitev2-api.ft.com\/render\/\?/);
 					done();
 				});
 			});