)]}'
{"tempest/api/compute/admin/test_agents.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"17eb8daf37f7297509bd9866ee8a457558cbf667","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        # Create and update an agent."},{"line_number":66,"context_line":"        body \u003d self.client.create_agent(**self.params_agent)[\u0027agent\u0027]"},{"line_number":67,"context_line":"        self.addCleanup(self.client.delete_agent, body[\u0027agent_id\u0027])"},{"line_number":68,"context_line":"        agent_id \u003d body[\u0027agent_id\u0027]"},{"line_number":69,"context_line":"        params \u003d self._param_helper("},{"line_number":70,"context_line":"            version\u003d\u00278.0\u0027, url\u003d\u0027xxx://xxxx/xxx/xxx2\u0027,"},{"line_number":71,"context_line":"            md5hash\u003d\u0027add6bb58e139be103324d04d82d8f547\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_4012d2b5","line":68,"updated":"2016-09-28 21:20:38.000000000","message":"Is there a reason you used a temp variable here and not in any of the tests below?","commit_id":"c1daa121a6368d60afe8f2d64e48c6f18ba283ea"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"889296a198a19c4d2ffd900b0f6fb507f1180be5","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        # Create and update an agent."},{"line_number":66,"context_line":"        body \u003d self.client.create_agent(**self.params_agent)[\u0027agent\u0027]"},{"line_number":67,"context_line":"        self.addCleanup(self.client.delete_agent, body[\u0027agent_id\u0027])"},{"line_number":68,"context_line":"        agent_id \u003d body[\u0027agent_id\u0027]"},{"line_number":69,"context_line":"        params \u003d self._param_helper("},{"line_number":70,"context_line":"            version\u003d\u00278.0\u0027, url\u003d\u0027xxx://xxxx/xxx/xxx2\u0027,"},{"line_number":71,"context_line":"            md5hash\u003d\u0027add6bb58e139be103324d04d82d8f547\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_bdce5458","line":68,"in_reply_to":"3a98d1f4_4012d2b5","updated":"2016-09-29 03:01:48.000000000","message":"No , there is no reason to use here temp variable. \nThank you .\nBenny","commit_id":"c1daa121a6368d60afe8f2d64e48c6f18ba283ea"}]}