)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"cf177470a35da41194584e27b9e3903367c6c142","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Parse Sample volume as float"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Explicitly parses sample volume as float to"},{"line_number":10,"context_line":"prevent incorrect sample from being published."},{"line_number":11,"context_line":"In case of an sample with wrong volume type or"},{"line_number":12,"context_line":"no volume. the sample is dropped and a warning"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_8f1c7614","line":9,"updated":"2015-11-10 07:05:57.000000000","message":"I think maybe the original commit message is enough to use, just add another closes-bug tag","commit_id":"9113e24d71bdaa25e092bc1b9cffe5c848ad9eb8"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"c90759be2e186eb0127ae4dcddaab2904ebd20b6","unresolved":false,"context_lines":[{"line_number":11,"context_line":"to avoid storing samples with None or not numerical volumes."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic4b87c05d16dba765f601ca346b198eda7c91bea"},{"line_number":14,"context_line":"Closes-Bug: #1508220"},{"line_number":15,"context_line":"(cherry-picked from commit 804de4ef2f26e1892dc2580fe16897b14741a13a)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da85f559_ab96bc3b","line":14,"updated":"2015-11-11 15:46:28.000000000","message":"It might be better to keep the original tag: Closes-Bug: #1484117 as well","commit_id":"021a67cb4813df69708ac1a43215acf5355c610a"}],"ceilometer/pipeline.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"cf177470a35da41194584e27b9e3903367c6c142","unresolved":false,"context_lines":[{"line_number":30,"context_line":"from ceilometer.event.storage import models"},{"line_number":31,"context_line":"from ceilometer.i18n import _"},{"line_number":32,"context_line":"from ceilometer.openstack.common import log"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"from ceilometer import publisher"},{"line_number":35,"context_line":"from ceilometer.publisher import utils as publisher_utils"},{"line_number":36,"context_line":"from ceilometer import sample as sample_util"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_2f1b0a1d","line":33,"updated":"2015-11-10 07:05:57.000000000","message":"this empty line is not needed","commit_id":"9113e24d71bdaa25e092bc1b9cffe5c848ad9eb8"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"50dffcbbb19174db241ba22ebeec84726e4a3a7b","unresolved":false,"context_lines":[{"line_number":495,"context_line":"    def _validate_volume(self, s):"},{"line_number":496,"context_line":"        volume \u003d s.volume"},{"line_number":497,"context_line":"        if volume is None:"},{"line_number":498,"context_line":"            LOG.warning(_("},{"line_number":499,"context_line":"                \u0027metering data %(counter_name)s for %(resource_id)s \u0027"},{"line_number":500,"context_line":"                \u0027@ %(timestamp)s has no volume (volume: None), the sample \u0027"},{"line_number":501,"context_line":"                \u0027will be dropped\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da85f559_f057b055","line":498,"updated":"2015-11-12 06:29:28.000000000","message":"actually, it can be _LW() instead of _(), the original fix has it and kilo branch supports _LW() as well, there is no reason to remove it IIUC.  But it is acceptable to me because this patch already get +2","commit_id":"2ec8c7de84bd9f6806b8129aa2c253055ad175db"}]}