)]}'
{"octavia/tests/tempest/v1/clients/health_monitors_client.py":[{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"92e4a458180e69e58d4ba32c36781004156f5f20","unresolved":false,"context_lines":[{"line_number":22,"context_line":"class HealthMonitorsClientJSON(service_client.ServiceClient):"},{"line_number":23,"context_line":"    \"\"\"Tests Health Monitors API.\"\"\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    _HEALTH_MONITORS_URL \u003d (\"v1/loadbalancers/{lb_id}/listeners/\""},{"line_number":26,"context_line":"                            \"{listener_id}/pools/{pool_id}/healthmonitor\")"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    def get_health_monitor(self, lb_id, listener_id, pool_id, params\u003dNone):"}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_9b1eaa61","line":25,"range":{"start_line":25,"start_character":52,"end_line":25,"end_character":54},"updated":"2016-02-22 06:56:26.000000000","message":"I don\u0027t think we need tempest tests against the deprecated URL, do we?  (ie. we can strip /listeners/{listener_id} out of this.) Note that in order for the tempest test to work, you still need to make sure the pool associated with the listener in some way-- but we should be using the non-deprecated URLs for this sort of thing, eh.","commit_id":"744a4155fb43cf4770f4760f4dd4f98dfcd78ae2"}],"octavia/tests/tempest/v1/scenario/base.py":[{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"92e4a458180e69e58d4ba32c36781004156f5f20","unresolved":false,"context_lines":[{"line_number":292,"context_line":"            fall_threshold\u003d5, rise_threshold\u003d5,"},{"line_number":293,"context_line":"            lb_id\u003dself.load_balancer[\u0027id\u0027],"},{"line_number":294,"context_line":"            pool_id\u003dself.pool[\u0027id\u0027],"},{"line_number":295,"context_line":"            listener_id\u003dself.listener[\u0027id\u0027])"},{"line_number":296,"context_line":"        self.assertTrue(self.hm)"},{"line_number":297,"context_line":"        self.addCleanup(self._cleanup_health_monitor,"},{"line_number":298,"context_line":"                        load_balancer_id\u003dself.load_balancer[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_a0825d92","line":295,"range":{"start_line":295,"start_character":12,"end_line":295,"end_character":23},"updated":"2016-02-22 06:56:26.000000000","message":"Health monitor creation (or any other manipulation) should not care about the listener_id. The only reason you\u0027d need that is if you\u0027re testing against the deprecated URL for the health monitor (which I don\u0027t think is the intent here).","commit_id":"744a4155fb43cf4770f4760f4dd4f98dfcd78ae2"}]}