)]}'
{"tempest/api/compute/servers/test_list_servers_negative.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"8147e5ea03ad19ba93ae91c5c101691afbb453fb","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    @test.idempotent_id(\u002712c80a9f-2dec-480e-882b-98ba15757659\u0027)"},{"line_number":103,"context_line":"    def test_list_servers_by_limits(self):"},{"line_number":104,"context_line":"        # List servers by specifying limits"},{"line_number":105,"context_line":"        body \u003d self.client.list_servers(**{\u0027limit\u0027: 1})"},{"line_number":106,"context_line":"        self.assertEqual(1, len([x for x in body[\u0027servers\u0027] if \u0027id\u0027 in x]))"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba3cc151_3ab3bc0b","line":105,"updated":"2015-07-02 21:22:54.000000000","message":"I think you should use the same approach that was proposed by afazekas here https://review.openstack.org/#/c/197467 .","commit_id":"f500af7d9ace2e6518c797ef51dbbfecda0dc05e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"596112824e61882fdf514efa2eeee940d6e408b4","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    @test.idempotent_id(\u002712c80a9f-2dec-480e-882b-98ba15757659\u0027)"},{"line_number":103,"context_line":"    def test_list_servers_by_limits(self):"},{"line_number":104,"context_line":"        # List servers by specifying limits"},{"line_number":105,"context_line":"        body \u003d self.client.list_servers(**{\u0027limit\u0027: 1})"},{"line_number":106,"context_line":"        self.assertEqual(1, len([x for x in body[\u0027servers\u0027] if \u0027id\u0027 in x]))"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba3cc151_7802a5b5","line":105,"in_reply_to":"ba3cc151_3ab3bc0b","updated":"2015-07-03 03:42:31.000000000","message":"Yeah, I agree with you.\nThanks for pointing it out.","commit_id":"f500af7d9ace2e6518c797ef51dbbfecda0dc05e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0c8e5c2859b836e235641891d185f8ad74b6c41c","unresolved":false,"context_lines":[{"line_number":130,"context_line":"    @test.idempotent_id(\u002787d12517-e20a-4c9c-97b6-dd1628d6d6c9\u0027)"},{"line_number":131,"context_line":"    def test_list_servers_by_changes_since_invalid_date(self):"},{"line_number":132,"context_line":"        # Return an error when invalid date format is passed"},{"line_number":133,"context_line":"        params \u003d {\u0027changes-since\u0027: \u00272011/01/01\u0027}"},{"line_number":134,"context_line":"        self.assertRaises(lib_exc.BadRequest, self.client.list_servers,"},{"line_number":135,"context_line":"                          **params)"},{"line_number":136,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"ba3cc151_4ce33ca5","line":133,"updated":"2015-07-10 08:56:39.000000000","message":"Ken\u0027ichi -\u003e \u0027changes-since\u0027 :)\n\nThis is nice example for how some arg names are bad :). We should fix those as done in v3 API.","commit_id":"cbc26a8c51f63953c78c5b4c85d1bc8fb2ad5582"}]}