)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"fab3ccfcc66241a12eef47161358947b70b897f6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove deprecated network existence meter"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Deleting all the old existence meters for network and"},{"line_number":10,"context_line":"network services."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic14d70c1ebdd128c8e111c8df52fe3359cc8e7d9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a1a9d01_004c2775","line":10,"updated":"2015-10-01 14:21:10.000000000","message":"-1: with so much changes, I think DocImpact is needed, \n\nAn option has been changed\nmany meters have been removed","commit_id":"dba16cd462bc75711bb82131e716530834d66048"}],"ceilometer/exchange_control.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"fab3ccfcc66241a12eef47161358947b70b897f6","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    cfg.StrOpt(\u0027neutron_control_exchange\u0027,"},{"line_number":48,"context_line":"               default\u003d\u0027neutron\u0027,"},{"line_number":49,"context_line":"               help\u003d\"Exchange name for Neutron notifications.\","},{"line_number":50,"context_line":"               deprecated_name\u003d\u0027quantum_control_exchange\u0027),"},{"line_number":51,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_ccb3cbe8","line":50,"updated":"2015-10-01 14:21:10.000000000","message":"-1: we should add deprecated group as well","commit_id":"dba16cd462bc75711bb82131e716530834d66048"}],"ceilometer/network/services/lbaas.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"cf63ab8bf2a30110e64d010436053fc4243b0ed4","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            yield sample.Sample("},{"line_number":188,"context_line":"                name\u003d\u0027network.services.lb.health_monitor\u0027,"},{"line_number":189,"context_line":"                type\u003dsample.TYPE_GAUGE,"},{"line_number":190,"context_line":"                unit\u003d\u0027health_monitor\u0027,"},{"line_number":191,"context_line":"                volume\u003d1,"},{"line_number":192,"context_line":"                user_id\u003dNone,"},{"line_number":193,"context_line":"                project_id\u003dprobe[\u0027tenant_id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_fa7ef817","side":"PARENT","line":190,"updated":"2015-09-29 06:56:33.000000000","message":"Prad, could you please describe our plan about removing the pollsters with value\u003d1? My understanding is that they were created because people wanted to get the information about service health (like in this case: network.services.lb.health_monitor). And now we remove this pollster. So how this problem may be solved now? I thought that we will keep polling and will process result message from a service in event_pipeline","commit_id":"8ece143acd657c1cd23ad31747e3e750eac3d1eb"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"93435af16779316d778f17a4431b93e0622b8d28","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            yield sample.Sample("},{"line_number":188,"context_line":"                name\u003d\u0027network.services.lb.health_monitor\u0027,"},{"line_number":189,"context_line":"                type\u003dsample.TYPE_GAUGE,"},{"line_number":190,"context_line":"                unit\u003d\u0027health_monitor\u0027,"},{"line_number":191,"context_line":"                volume\u003d1,"},{"line_number":192,"context_line":"                user_id\u003dNone,"},{"line_number":193,"context_line":"                project_id\u003dprobe[\u0027tenant_id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_d2572a24","side":"PARENT","line":190,"in_reply_to":"9a1a9d01_ec74f836","updated":"2015-09-30 06:41:45.000000000","message":"Yep, right, but we still need to poll, don\u0027t we? From http://specs.openstack.org/openstack/ceilometer-specs/specs/liberty/deprecate-existence-meters.html :\n\"We will need to convert these pollsters to republish information as events\"\n\nSo my question is, how the republishing will be done? In a separate patch?","commit_id":"8ece143acd657c1cd23ad31747e3e750eac3d1eb"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"ac76260a4b8a879dbfba95d32f05b9a1108ed9b5","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            yield sample.Sample("},{"line_number":188,"context_line":"                name\u003d\u0027network.services.lb.health_monitor\u0027,"},{"line_number":189,"context_line":"                type\u003dsample.TYPE_GAUGE,"},{"line_number":190,"context_line":"                unit\u003d\u0027health_monitor\u0027,"},{"line_number":191,"context_line":"                volume\u003d1,"},{"line_number":192,"context_line":"                user_id\u003dNone,"},{"line_number":193,"context_line":"                project_id\u003dprobe[\u0027tenant_id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_ec74f836","side":"PARENT","line":190,"in_reply_to":"9a1a9d01_fa7ef817","updated":"2015-09-29 13:19:32.000000000","message":"we need a way to generate an healthcheck event from this instead of sample.","commit_id":"8ece143acd657c1cd23ad31747e3e750eac3d1eb"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"fab3ccfcc66241a12eef47161358947b70b897f6","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        return i_cache[pool_id]"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @property"},{"line_number":78,"context_line":"    def default_discovery(self):"},{"line_number":79,"context_line":"        return \u0027lb_pools\u0027"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @abc.abstractmethod"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_8c08732f","line":78,"updated":"2015-10-01 14:21:10.000000000","message":"it depends on a discovery which has been deleted","commit_id":"dba16cd462bc75711bb82131e716530834d66048"}],"ceilometer/tests/unit/network/services/test_lbaas.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"fab3ccfcc66241a12eef47161358947b70b897f6","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from ceilometer.network.services import lbaas"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class _BaseTestLBPollster(base.BaseTestCase):"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @mock.patch(\u0027ceilometer.pipeline.setup_pipeline\u0027, mock.MagicMock())"},{"line_number":29,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_ec100fae","line":26,"updated":"2015-10-01 14:21:10.000000000","message":"no need to be base class any more,merge them","commit_id":"dba16cd462bc75711bb82131e716530834d66048"}],"setup.cfg":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"fab3ccfcc66241a12eef47161358947b70b897f6","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    endpoint \u003d ceilometer.agent.discovery.endpoint:EndpointDiscovery"},{"line_number":63,"context_line":"    tenant \u003d ceilometer.agent.discovery.tenant:TenantDiscovery"},{"line_number":64,"context_line":"    local_node \u003d ceilometer.agent.discovery.localnode:LocalNodeDiscovery"},{"line_number":65,"context_line":"    lb_pools \u003d ceilometer.network.services.discovery:LBPoolsDiscovery"},{"line_number":66,"context_line":"    lb_vips \u003d ceilometer.network.services.discovery:LBVipsDiscovery"},{"line_number":67,"context_line":"    lb_members \u003d ceilometer.network.services.discovery:LBMembersDiscovery"},{"line_number":68,"context_line":"    lb_health_probes \u003d ceilometer.network.services.discovery:LBHealthMonitorsDiscovery"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"9a1a9d01_8c2153b7","side":"PARENT","line":65,"updated":"2015-10-01 14:21:10.000000000","message":"if this discovery is removed, how does the CI pass?","commit_id":"33fa515d6dad73766c8a6ee5e0284853c1529bba"}]}
