)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"8de38ff93206b233e67c1f2c03b12fd0c76c9b62","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dynamic pollster support paging systems that return only a path for the next page"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oThis was found when working with Glance API, where the next page attribute only"},{"line_number":10,"context_line":"contains the context for the next call, and not a full qualified web address."},{"line_number":11,"context_line":"Moreover, while implementing this one, I noticed the there was no paging for"},{"line_number":12,"context_line":"the non-openstack pollster. Therefore, I added it as well."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bf51134e_0ff799d6","line":9,"updated":"2020-07-22 06:42:38.000000000","message":"Typo \"oThis\"","commit_id":"cc39dd676acb5b726c5db00c52c4f00cec07e29d"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"dc5131723c82d39b77dbb8c06e3a883256e43ea3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dynamic pollster support paging systems that return only a path for the next page"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oThis was found when working with Glance API, where the next page attribute only"},{"line_number":10,"context_line":"contains the context for the next call, and not a full qualified web address."},{"line_number":11,"context_line":"Moreover, while implementing this one, I noticed the there was no paging for"},{"line_number":12,"context_line":"the non-openstack pollster. Therefore, I added it as well."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bf51134e_2acfe857","line":9,"in_reply_to":"bf51134e_0ff799d6","updated":"2020-07-22 11:33:26.000000000","message":"Done","commit_id":"cc39dd676acb5b726c5db00c52c4f00cec07e29d"}],"ceilometer/tests/unit/polling/test_dynamic_pollster.py":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"8de38ff93206b233e67c1f2c03b12fd0c76c9b62","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"\"\"\"Tests for OpenStack dynamic pollster"},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":"from six.moves.urllib import parse as url_parse"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import copy"},{"line_number":19,"context_line":"import logging"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_cf264169","line":16,"updated":"2020-07-22 06:42:38.000000000","message":"Please move this line above line 20 (to keep the list of imports sorted)\n\nWhile you\u0027re at it, please also move line 24 (requests) in line, so that openstack imports and outside imports are separated.","commit_id":"cc39dd676acb5b726c5db00c52c4f00cec07e29d"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"dc5131723c82d39b77dbb8c06e3a883256e43ea3","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"\"\"\"Tests for OpenStack dynamic pollster"},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":"from six.moves.urllib import parse as url_parse"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import copy"},{"line_number":19,"context_line":"import logging"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_4a717c44","line":16,"in_reply_to":"bf51134e_cf264169","updated":"2020-07-22 11:33:26.000000000","message":"I am not sure if I got everything you wanted. Can you check please?","commit_id":"cc39dd676acb5b726c5db00c52c4f00cec07e29d"}],"ceilometer/tests/unit/polling/test_non_openstack_dynamic_pollster.py":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"8de38ff93206b233e67c1f2c03b12fd0c76c9b62","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from ceilometer.polling.dynamic_pollster import PollsterSampleGatherer"},{"line_number":32,"context_line":"from ceilometer.polling.dynamic_pollster import SingleMetricPollsterDefinitions"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"from six.moves.urllib import parse as url_parse"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"REQUIRED_POLLSTER_FIELDS \u003d [\u0027name\u0027, \u0027sample_type\u0027, \u0027unit\u0027, \u0027value_attribute\u0027,"},{"line_number":37,"context_line":"                            \u0027url_path\u0027, \u0027module\u0027, \u0027authentication_object\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_4f1a3117","line":34,"updated":"2020-07-22 06:42:38.000000000","message":"Please move this to the top imports (to the section non-openstack imports)","commit_id":"cc39dd676acb5b726c5db00c52c4f00cec07e29d"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"dc5131723c82d39b77dbb8c06e3a883256e43ea3","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from ceilometer.polling.dynamic_pollster import PollsterSampleGatherer"},{"line_number":32,"context_line":"from ceilometer.polling.dynamic_pollster import SingleMetricPollsterDefinitions"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"from six.moves.urllib import parse as url_parse"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"REQUIRED_POLLSTER_FIELDS \u003d [\u0027name\u0027, \u0027sample_type\u0027, \u0027unit\u0027, \u0027value_attribute\u0027,"},{"line_number":37,"context_line":"                            \u0027url_path\u0027, \u0027module\u0027, \u0027authentication_object\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_cae8ecb4","line":34,"in_reply_to":"bf51134e_4f1a3117","updated":"2020-07-22 11:33:26.000000000","message":"Done","commit_id":"cc39dd676acb5b726c5db00c52c4f00cec07e29d"}]}