MirrorMirror
/
Signal-Server
mirror of https://github.com/signalapp/Signal-Server.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Correct a misunderstanding about the metrics host tag.

This commit is contained in:
Jon Chambers 2021-06-07 15:52:01 -04:00 committed by Jon Chambers
parent 6cbd57f19f
commit 88c4b2be97
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
service/src/main/java/org/whispersystems/textsecuregcm/WhisperServerService.java
View File

@ -275,7 +275,7 @@ public class WhisperServerService extends Application<WhisperServerConfiguration
}); });
{ {
final String host; final String hostname;
{ {
String localHostName = "unknown"; String localHostName = "unknown";
@ -285,7 +285,7 @@ public class WhisperServerService extends Application<WhisperServerConfiguration
} catch (final UnknownHostException ignored) { } catch (final UnknownHostException ignored) {
} }
host = localHostName; hostname = localHostName;
} }
final DatadogMeterRegistry datadogMeterRegistry = new DatadogMeterRegistry(new DatadogConfig() { final DatadogMeterRegistry datadogMeterRegistry = new DatadogMeterRegistry(new DatadogConfig() {
@ -306,13 +306,14 @@ public class WhisperServerService extends Application<WhisperServerConfiguration
@Override @Override
public String hostTag() { public String hostTag() {
return host; return "host";
} }
}, Clock.SYSTEM); }, Clock.SYSTEM);
datadogMeterRegistry.config().commonTags( datadogMeterRegistry.config().commonTags(
Tags.of( Tags.of(
"service", "chat", "service", "chat",
"host", hostname,
"version", WhisperServerVersion.getServerVersion(), "version", WhisperServerVersion.getServerVersion(),
"env", config.getDatadogConfiguration().getEnvironment())) "env", config.getDatadogConfiguration().getEnvironment()))
.meterFilter(MeterFilter.denyNameStartsWith(MetricsRequestEventListener.REQUEST_COUNTER_NAME)) .meterFilter(MeterFilter.denyNameStartsWith(MetricsRequestEventListener.REQUEST_COUNTER_NAME))