Merge lp:~hloeung/ubuntu-repository-cache/add-send-to-influx-socket-timeout into lp:ubuntu-repository-cache

Proposed by Haw Loeung
Status: Merged
Approved by: Haw Loeung
Approved revision: 319
Merged at revision: 319
Proposed branch: lp:~hloeung/ubuntu-repository-cache/add-send-to-influx-socket-timeout
Merge into: lp:ubuntu-repository-cache
Diff against target: 11 lines (+1/-0)
1 file modified
files/health_check.py (+1/-0)
To merge this branch: bzr merge lp:~hloeung/ubuntu-repository-cache/add-send-to-influx-socket-timeout
Reviewer Review Type Date Requested Status
Paul Collins lgtm Approve
Canonical IS Reviewers Pending
Review via email: mp+398931@code.launchpad.net

Commit message

Add socket timeout for sending metrics to influx

To post a comment you must log in.
Revision history for this message
Paul Collins (pjdc) :
review: Approve (lgtm)
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Haw Loeung (hloeung) :
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 319

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'files/health_check.py'
2--- files/health_check.py 2021-02-16 23:48:45 +0000
3+++ files/health_check.py 2021-03-02 01:41:05 +0000
4@@ -48,6 +48,7 @@
5 port = int(port)
6 try:
7 s = socket.socket()
8+ s.settimeout(2)
9 s.connect((host, port))
10 with s.makefile(mode='w') as dest:
11 dest.write(metrics)

Subscribers

People subscribed via source and target branches