)]}'
{"tempest/api/network/admin/test_dhcp_agent_scheduler.py":[{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"df25758d4ae4126868a4909c9022775c5e845622","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":37,"context_line":"    def test_list_dhcp_agent_hosting_network(self):"},{"line_number":38,"context_line":"        resp, body \u003d self.admin_client.list_dhcp_agent_hosting_network("},{"line_number":39,"context_line":"            self.network[\u0027id\u0027])"},{"line_number":40,"context_line":"        self.assertEqual(resp[\u0027status\u0027], \u0027200\u0027)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_4bd89eef","side":"PARENT","line":39,"updated":"2014-08-14 18:59:28.000000000","message":"Indentation needs to change too.","commit_id":"3226ca1450f53ab7d6c94f2886968bc62ed851b7"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"df25758d4ae4126868a4909c9022775c5e845622","unresolved":false,"context_lines":[{"line_number":52,"context_line":"    def _check_network_in_dhcp_agent(self, network_id, agent):"},{"line_number":53,"context_line":"        network_ids \u003d []"},{"line_number":54,"context_line":"        resp, body \u003d self.admin_client.list_networks_hosted_by_one_dhcp_agent("},{"line_number":55,"context_line":"            agent[\u0027id\u0027])"},{"line_number":56,"context_line":"        self.assertEqual(resp[\u0027status\u0027], \u0027200\u0027)"},{"line_number":57,"context_line":"        networks \u003d body[\u0027networks\u0027]"},{"line_number":58,"context_line":"        for network in networks:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_8bf9d64f","side":"PARENT","line":55,"updated":"2014-08-14 18:59:28.000000000","message":"Ditto","commit_id":"3226ca1450f53ab7d6c94f2886968bc62ed851b7"}],"tempest/common/rest_client.py":[{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"7c10c8af9beec4059936def083542b3c668b2396","unresolved":false,"context_lines":[{"line_number":198,"context_line":"                      \"codes which explicitly defined in the RFC 2616. {0}\""},{"line_number":199,"context_line":"                      \" is not a defined Success Code!\").format(expected_code)"},{"line_number":200,"context_line":"        if isinstance(read_code, basestring):"},{"line_number":201,"context_line":"            read_code \u003d int(read_code)"},{"line_number":202,"context_line":"        if isinstance(expected_code, basestring):"},{"line_number":203,"context_line":"            expected_code \u003d int(expected_code)"},{"line_number":204,"context_line":"        if isinstance(expected_code, list):"}],"source_content_type":"text/x-python","patch_set":7,"id":"1abeadc6_d99d9867","line":201,"updated":"2014-08-24 10:23:52.000000000","message":"I agree with David\u0027s comments, this isn\u0027t necessary.","commit_id":"cc0bc4cddb0b6c718930d1f03a5b9639f96cf855"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"06eab0d7c12ae0da5aee3dbc818b434e390dbd71","unresolved":false,"context_lines":[{"line_number":197,"context_line":"        assert_msg \u003d (\"This function only allowed to use for HTTP status\""},{"line_number":198,"context_line":"                      \"codes which explicitly defined in the RFC 2616. {0}\""},{"line_number":199,"context_line":"                      \" is not a defined Success Code!\").format(expected_code)"},{"line_number":200,"context_line":"        if isinstance(read_code, basestring):"},{"line_number":201,"context_line":"            read_code \u003d int(read_code)"},{"line_number":202,"context_line":"        if isinstance(expected_code, basestring):"},{"line_number":203,"context_line":"            expected_code \u003d int(expected_code)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1abeadc6_f1ad4dea","line":200,"updated":"2014-08-21 20:33:56.000000000","message":"There is no need to do this. Please just pass numbers for the expected code and resp.status for the read code as all the other clients do. Is there a reason you did not do that?","commit_id":"9900e21dace784c561fa8a75aef5ae3144f1cd46"}]}