)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"a2db1fc19792bdf4e2f3eaf566955eac699bf63e","unresolved":true,"context_lines":[{"line_number":1,"context_line":"Parent:     0091bfc9 (Merge \"Volume Target support for Ironic on OpenStack SDK\")"},{"line_number":2,"context_line":"Author:     Gregory Thiemonge \u003cgthiemon@redhat.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2020-11-23 15:46:17 +0100"},{"line_number":4,"context_line":"Commit:     anuradha1904 \u003canuradha750871@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-12-09 20:41:05 +0530"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bb36185e_b773a4e4","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":51},"updated":"2020-12-10 07:20:24.000000000","message":"you probably amended a previous commit, because I\u0027m 100% sure I\u0027m not the author of this patch ;-)","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"}],"openstack/baremetal/v1/condutor.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"class Conductor(_common.ListMixin, resource.Resource):"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    resources_key \u003d \u0027/conductors\u0027"},{"line_number":20,"context_line":"    base_path \u003d \u0027/conductors\u0027"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    # capabilities"}],"source_content_type":"text/x-python","patch_set":1,"id":"6b937e07_51fef5bf","line":19,"range":{"start_line":19,"start_character":21,"end_line":19,"end_character":32},"updated":"2020-12-09 20:46:10.000000000","message":"the resources_key are the key you have in the response, it doesn\u0027t contain the /","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    created_at \u003d resource.Body(\u0027created_at\u0027)"},{"line_number":36,"context_line":"    updated_at \u003d resource.Body(\u0027updated_at\u0027)"},{"line_number":37,"context_line":"    hostname \u003d resource.Body(\u0027hostname\u0027, type\u003dlist)"},{"line_number":38,"context_line":"    conductor_group \u003d resource.Body(\u0027conductor_group\u0027)"},{"line_number":39,"context_line":"    alive \u003d resource.Body(\u0027alive\u0027)"},{"line_number":40,"context_line":"    links \u003d resource.Body(\u0027links\u0027, type\u003dlist)"}],"source_content_type":"text/x-python","patch_set":1,"id":"35148663_ec568a42","line":37,"range":{"start_line":37,"start_character":4,"end_line":37,"end_character":51},"updated":"2020-12-09 20:46:10.000000000","message":"You can remove type\u003dlist, I just noticed the api-ref we have says hostname is type array (list in python) but it should be string \u003d)","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    updated_at \u003d resource.Body(\u0027updated_at\u0027)"},{"line_number":37,"context_line":"    hostname \u003d resource.Body(\u0027hostname\u0027, type\u003dlist)"},{"line_number":38,"context_line":"    conductor_group \u003d resource.Body(\u0027conductor_group\u0027)"},{"line_number":39,"context_line":"    alive \u003d resource.Body(\u0027alive\u0027)"},{"line_number":40,"context_line":"    links \u003d resource.Body(\u0027links\u0027, type\u003dlist)"},{"line_number":41,"context_line":"    drivers \u003d resource.Body(\u0027drivers\u0027, type\u003dlist)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e78b3cd7_2657f75e","line":39,"range":{"start_line":39,"start_character":4,"end_line":39,"end_character":34},"updated":"2020-12-09 20:46:10.000000000","message":"this one you can add type\u003dbool","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"}],"openstack/exceptions.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":211,"context_line":"        try:"},{"line_number":212,"context_line":"            content \u003d response.json()"},{"line_number":213,"context_line":"            messages \u003d [_extract_message(obj) for obj in content.values()]"},{"line_number":214,"context_line":"            if not any(messages):"},{"line_number":215,"context_line":"                # Exception dict may be the root dict in projects that use WSME"},{"line_number":216,"context_line":"                messages \u003d [_extract_message(content)]"},{"line_number":217,"context_line":"            # Join all of the messages together nicely and filter out any"},{"line_number":218,"context_line":"            # objects that don\u0027t have a \"message\" attr."},{"line_number":219,"context_line":"            details \u003d \u0027\\n\u0027.join(msg for msg in messages if msg)"},{"line_number":220,"context_line":"        except Exception:"}],"source_content_type":"text/x-python","patch_set":1,"id":"92f34be7_663f0151","line":217,"range":{"start_line":214,"start_character":0,"end_line":217,"end_character":58},"updated":"2020-12-09 20:46:10.000000000","message":"I\u0027m wondering why this change came on your patch since this is already in the repo .-.","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"}],"openstack/tests/functional/baremetal/test_baremetal_conductor.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":18,"context_line":"class TestBareMetalConductor(base.BaseBaremetalTest):"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    def test_conductor_list(self):"},{"line_number":21,"context_line":"        conductor \u003d self.conn.baremetal.get_conductor()"},{"line_number":22,"context_line":"        self.assertNotEqual([], conductor)"},{"line_number":23,"context_line":"        conductor_name \u003d self.conn.baremetal.get_conductor(\u0027hostname\u0027)"},{"line_number":24,"context_line":"        self.assertEqual(conductor, conductor_name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c6fe8530_45c181cb","line":21,"range":{"start_line":21,"start_character":40,"end_line":21,"end_character":53},"updated":"2020-12-09 20:46:10.000000000","message":"I couldn\u0027t find the implementation of get_conductor on your patch \n\nget_conductor should be used when you want a specific conductor, you need to have a call that will be conductors() that will return the list of available conductors","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":20,"context_line":"    def test_conductor_list(self):"},{"line_number":21,"context_line":"        conductor \u003d self.conn.baremetal.get_conductor()"},{"line_number":22,"context_line":"        self.assertNotEqual([], conductor)"},{"line_number":23,"context_line":"        conductor_name \u003d self.conn.baremetal.get_conductor(\u0027hostname\u0027)"},{"line_number":24,"context_line":"        self.assertEqual(conductor, conductor_name)"},{"line_number":25,"context_line":"        "},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"b33f74c4_2eac661d","line":24,"range":{"start_line":23,"start_character":8,"end_line":24,"end_character":51},"updated":"2020-12-09 20:46:10.000000000","message":"The idea would be iterate over the list of conductors you got on line 21, and you would try to call get_conductor(hostname) for each one you have in the list and you would assert it has information.","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"}],"openstack/tests/unit/test_exceptions.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ec9456c8d1f55a86df36fe5279d5d26f24bcd506","unresolved":true,"context_lines":[{"line_number":214,"context_line":"        self.assertEqual(self.message, exc.details)"},{"line_number":215,"context_line":"        self.assertIn(self.message, str(exc))"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    def test_raise_wsme_format(self):"},{"line_number":218,"context_line":"        response \u003d mock.Mock()"},{"line_number":219,"context_line":"        response.status_code \u003d 404"},{"line_number":220,"context_line":"        response.headers \u003d {"},{"line_number":221,"context_line":"            \u0027content-type\u0027: \u0027application/json\u0027,"},{"line_number":222,"context_line":"        }"},{"line_number":223,"context_line":"        response.json.return_value \u003d {"},{"line_number":224,"context_line":"            \u0027faultstring\u0027: self.message,"},{"line_number":225,"context_line":"            \u0027faultcode\u0027: \u0027Client\u0027,"},{"line_number":226,"context_line":"            \u0027debuginfo\u0027: None,"},{"line_number":227,"context_line":"        }"},{"line_number":228,"context_line":"        exc \u003d self.assertRaises(exceptions.NotFoundException,"},{"line_number":229,"context_line":"                                self._do_raise, response,"},{"line_number":230,"context_line":"                                error_message\u003dself.message)"},{"line_number":231,"context_line":"        self.assertEqual(response.status_code, exc.status_code)"},{"line_number":232,"context_line":"        self.assertEqual(self.message, exc.details)"},{"line_number":233,"context_line":"        self.assertIn(self.message, str(exc))"}],"source_content_type":"text/x-python","patch_set":1,"id":"42f69c42_182537a1","line":233,"range":{"start_line":217,"start_character":0,"end_line":233,"end_character":45},"updated":"2020-12-09 20:46:10.000000000","message":"This change was from other commit that is already in the repo,\nmaybe you did some ammend with the last commit instead of creating an new branch to do your work?","commit_id":"3b411dc0ba1dc87491b52781073fba89ea19bf1a"}]}