Merge lp:~cprov/core-result-checker/bump-uservice into lp:core-result-checker

Proposed by Celso Providelo
Status: Merged
Approved by: Celso Providelo
Approved revision: 11
Merged at revision: 10
Proposed branch: lp:~cprov/core-result-checker/bump-uservice
Merge into: lp:core-result-checker
Diff against target: 21 lines (+2/-2)
2 files modified
core_result_checker/__init__.py (+1/-1)
requirements.txt (+1/-1)
To merge this branch: bzr merge lp:~cprov/core-result-checker/bump-uservice
Reviewer Review Type Date Requested Status
Celso Providelo (community) Approve
Review via email: mp+255164@code.launchpad.net

Commit message

Upgrading uservices-utils dependency to 1.0.2.1 to benefit of working logstash support.

Description of the change

Upgrading uservices-utils dependency to 1.0.2.1 to benefit of working logstash support.

Also ensure the service logs a watermarked message on start ('Started!') so we can monitor if it is dying unexpectedly.

Project pip-cache already include the new package version.

To post a comment you must log in.
Revision history for this message
Celso Providelo (cprov) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'core_result_checker/__init__.py'
--- core_result_checker/__init__.py 2015-04-01 21:32:31 +0000
+++ core_result_checker/__init__.py 2015-04-03 01:49:55 +0000
@@ -54,7 +54,7 @@
54 config['logstash'] if 'logstash' in config else None54 config['logstash'] if 'logstash' in config else None
55 )55 )
5656
57 logger.info('Started ...', extra=constants.LOGGING_EXTRA)57 logger.info('Started!', extra=constants.LOGGING_EXTRA)
5858
59 # for now, do nothing at all.59 # for now, do nothing at all.
60 select.select([], [], [])60 select.select([], [], [])
6161
=== modified file 'requirements.txt'
--- requirements.txt 2015-04-02 04:43:29 +0000
+++ requirements.txt 2015-04-03 01:49:55 +0000
@@ -1,3 +1,3 @@
1kombu==3.0.241kombu==3.0.24
2python-logstash==0.4.22python-logstash==0.4.2
3uservice-utils==1.0.13uservice-utils==1.0.2.1

Subscribers

People subscribed via source and target branches