Elukey has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/367382 )

Change subject: Reduce the DNS queries for the statsd domain
......................................................................


Reduce the DNS queries for the statsd domain

The sendto() function issues a DNS query if the host[0] parameter
is not an IP address. We run several instances of logster once every
minute, and for each metric a DNS query for the statsd domain is
issued.

Bug: T171318
Change-Id: Ia3f1e4429168e3cb5cd7606772517644e5660d22
---
M logster/logster.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Elukey: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/logster/logster.py b/logster/logster.py
index ba7c748..ac1e113 100755
--- a/logster/logster.py
+++ b/logster/logster.py
@@ -272,6 +272,7 @@
 def submit_statsd(metrics, options):
     if (not options.dry_run):
         host = options.statsd_host.split(':')
+        host[0] = socket.gethostbyname(host[0])
 
     for metric in metrics:
         metric_name = metric.name

-- 
To view, visit https://gerrit.wikimedia.org/r/367382
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3f1e4429168e3cb5cd7606772517644e5660d22
Gerrit-PatchSet: 1
Gerrit-Project: operations/debs/logster
Gerrit-Branch: master
Gerrit-Owner: Elukey <ltosc...@wikimedia.org>
Gerrit-Reviewer: Elukey <ltosc...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to