)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7b34e8e9530034639e875136f19ba6eea2b9e113","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This patch adds unit tests for the methods create_agent, update_agent"},{"line_number":12,"context_line":"and delete_agent."},{"line_number":13,"context_line":"In addition, this patch changes update_agent method because _parse_resp"},{"line_number":14,"context_line":"cannot handle bytes data and py34 unit tests cannot pass without this"},{"line_number":15,"context_line":"change."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I85cf43007c0654bb7747ff3518a19894b34271f8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3a50d1a3_1faa2ad5","line":14,"updated":"2015-07-20 10:58:39.000000000","message":"Yes, this change looks fine but now _parse_resp can handle this in lib-0.7.0.\nMay be just update commit msg will be good.","commit_id":"2072b02b83a8a55a4f28c57b74dd57bc4557165e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1c454d05e183bdd4ed74764b0af14d43c1355001","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This patch adds unit tests for the methods create_agent, update_agent"},{"line_number":12,"context_line":"and delete_agent."},{"line_number":13,"context_line":"In addition, this patch changes update_agent method because _parse_resp"},{"line_number":14,"context_line":"cannot handle bytes data and py34 unit tests cannot pass without this"},{"line_number":15,"context_line":"change."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I85cf43007c0654bb7747ff3518a19894b34271f8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3a50d1a3_23000c01","line":14,"in_reply_to":"3a50d1a3_1faa2ad5","updated":"2015-07-21 00:15:30.000000000","message":"Nice catch.\nMy venv/tempest-lib was old.\nI will change the commit message soon.","commit_id":"2072b02b83a8a55a4f28c57b74dd57bc4557165e"}],"tempest/tests/services/compute/test_agents_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"31985a3e80bf1fe22982ee844c03f4dae18fa493","unresolved":false,"context_lines":[{"line_number":26,"context_line":"class TestAgentsClient(base.TestCase):"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    def setUp(self):"},{"line_number":29,"context_line":"        super(TestAgentsClient, self).setUp()"},{"line_number":30,"context_line":"        self.useFixture(fake_config.ConfigFixture())"},{"line_number":31,"context_line":"        fake_auth \u003d fake_auth_provider.FakeAuthProvider()"},{"line_number":32,"context_line":"        self.client \u003d agents_client.AgentsClient(fake_auth,"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a50d1a3_30991b20","line":29,"updated":"2015-07-21 10:11:54.000000000","message":"Sorry i missed this comment earlier. \n\nDo we need ConfigFixture here, as these tests mock get/post methods? and config option will be required here.\n\nIt runs without config fixture too. Or addition new tests can use that?","commit_id":"180f09f5179519d97666d53067627b2a7ee1c5b1"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d4575366604da7c497b0f8e18838661584c29471","unresolved":false,"context_lines":[{"line_number":26,"context_line":"class TestAgentsClient(base.TestCase):"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    def setUp(self):"},{"line_number":29,"context_line":"        super(TestAgentsClient, self).setUp()"},{"line_number":30,"context_line":"        self.useFixture(fake_config.ConfigFixture())"},{"line_number":31,"context_line":"        fake_auth \u003d fake_auth_provider.FakeAuthProvider()"},{"line_number":32,"context_line":"        self.client \u003d agents_client.AgentsClient(fake_auth,"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a50d1a3_f68d402b","line":29,"in_reply_to":"3a50d1a3_30991b20","updated":"2015-07-23 22:00:27.000000000","message":"Nice catch, I also feel here is unnecessary.\nso, will remove it.","commit_id":"180f09f5179519d97666d53067627b2a7ee1c5b1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"db251eb4e76171de0e1b53622c54e7abf8bc5b62","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from tempest.services.compute.json import agents_client"},{"line_number":21,"context_line":"from tempest.tests import base"},{"line_number":22,"context_line":"from tempest.tests import fake_auth_provider"},{"line_number":23,"context_line":"from tempest.tests import fake_config"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class TestAgentsClient(base.TestCase):"}],"source_content_type":"text/x-python","patch_set":8,"id":"3a50d1a3_9cc65a48","line":23,"updated":"2015-07-24 01:07:44.000000000","message":"fake_config  too.","commit_id":"56c4351bb0dda75be4e95b6ad54aea48f8ba56cf"}]}
