)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add Message Registry File resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is the second patch to support Redfish Message Registry."},{"line_number":10,"context_line":"It delivers Redfish Message Registry File and Collection parsing to sushy"},{"line_number":11,"context_line":"fields."},{"line_number":12,"context_line":"Any other processing (e.g., loading files) will be added in followup patches."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_9c7aa922","line":10,"updated":"2018-07-31 13:06:32.000000000","message":"nit: may be shorten the line a bit?","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add Message Registry File resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is the second patch to support Redfish Message Registry."},{"line_number":10,"context_line":"It delivers Redfish Message Registry File and Collection parsing to sushy"},{"line_number":11,"context_line":"fields."},{"line_number":12,"context_line":"Any other processing (e.g., loading files) will be added in followup patches."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_428f4821","line":10,"in_reply_to":"5f7c97a3_9c7aa922","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"It adds link to Registries resource from the service root."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Currently it is not intended to be exposed to sushy users."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I3953840cf0145407d5915ee8d0a8a5f909301cc2"},{"line_number":19,"context_line":"Story: 2001791"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_1c8c592c","line":16,"range":{"start_line":16,"start_character":10,"end_line":16,"end_character":12},"updated":"2018-07-31 13:06:32.000000000","message":"nit: may be name what exactly is not intended","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"It adds link to Registries resource from the service root."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Currently it is not intended to be exposed to sushy users."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I3953840cf0145407d5915ee8d0a8a5f909301cc2"},{"line_number":19,"context_line":"Story: 2001791"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_62704442","line":16,"range":{"start_line":16,"start_character":10,"end_line":16,"end_character":12},"in_reply_to":"5f7c97a3_1c8c592c","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"}],"sushy/main.py":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"0728f718e9fb100f3ccbf2f0e4c53c5527cdbf4b","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        \"\"\""},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"        if self._registries_path:"},{"line_number":176,"context_line":"            messageregistryfile.MessageRegistryFileCollection("},{"line_number":177,"context_line":"                self._conn,"},{"line_number":178,"context_line":"                self._registries_path,"},{"line_number":179,"context_line":"                redfish_version\u003dself.redfish_version)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_725f7fe3","line":176,"updated":"2018-08-02 12:39:10.000000000","message":"Does this method return anything other than None?","commit_id":"246d6189a7da648afb4603629eec099ba94aa867"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"afdaf59cb6865fedb015e63dd7e6a671bfcae522","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        \"\"\""},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"        if self._registries_path:"},{"line_number":176,"context_line":"            messageregistryfile.MessageRegistryFileCollection("},{"line_number":177,"context_line":"                self._conn,"},{"line_number":178,"context_line":"                self._registries_path,"},{"line_number":179,"context_line":"                redfish_version\u003dself.redfish_version)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_e9c39d39","line":176,"in_reply_to":"5f7c97a3_725f7fe3","updated":"2018-08-02 15:58:02.000000000","message":"Done","commit_id":"246d6189a7da648afb4603629eec099ba94aa867"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"2eaafca2e53d97047042149560ae86848191df25","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        return session.Session(self._conn, identity,"},{"line_number":164,"context_line":"                               redfish_version\u003dself.redfish_version)"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def _get_registry_collection(self):"},{"line_number":167,"context_line":"        \"\"\"Get MessageRegistryFileCollection object"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        This resource is optional and can be empty."}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_0d16a94b","line":166,"updated":"2018-09-28 13:12:00.000000000","message":"While this collection is mostly for internal use, I see no harm in exposing it to users (just let us call it get_message_registry_file_collection?). This can be done in a follow-up.","commit_id":"dac8bc3498b4d2e24e12a20c5aaec7f20fa6df17"}],"sushy/resources/registry/messageregistryfile.py":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Location for each registry file of languages supported"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_dcdfc108","line":24,"range":{"start_line":24,"start_character":21,"end_line":24,"end_character":22},"updated":"2018-07-31 13:06:32.000000000","message":"*a*","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Location for each registry file of languages supported"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_bcc3e538","line":24,"range":{"start_line":24,"start_character":4,"end_line":24,"end_character":11},"updated":"2018-07-31 13:06:32.000000000","message":"nit: bullet points?","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Location for each registry file of languages supported"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_bcc8c54c","line":24,"range":{"start_line":24,"start_character":12,"end_line":24,"end_character":18},"updated":"2018-07-31 13:06:32.000000000","message":"nit: locally, as a single file\n?","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Location for each registry file of languages supported"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_67d37603","line":24,"range":{"start_line":24,"start_character":4,"end_line":24,"end_character":11},"in_reply_to":"5f7c97a3_bcc3e538","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Location for each registry file of languages supported"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_87ced255","line":24,"range":{"start_line":24,"start_character":12,"end_line":24,"end_character":18},"in_reply_to":"5f7c97a3_bcc8c54c","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    \"\"\"Location for each registry file of languages supported"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_27033e78","line":24,"range":{"start_line":24,"start_character":21,"end_line":24,"end_character":22},"in_reply_to":"5f7c97a3_dcdfc108","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_9cf62990","line":25,"range":{"start_line":25,"start_character":21,"end_line":25,"end_character":22},"updated":"2018-07-31 13:06:32.000000000","message":"*a*","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    There are 3 options where the file can be hosted:"},{"line_number":24,"context_line":"    locally hosted as single file,"},{"line_number":25,"context_line":"    locally hosted as part of archive (zip or other),"},{"line_number":26,"context_line":"    hosted publicly on the Internet."},{"line_number":27,"context_line":"    \"\"\""},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_823ec0b3","line":25,"range":{"start_line":25,"start_character":21,"end_line":25,"end_character":22},"in_reply_to":"5f7c97a3_9cf62990","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"}],"sushy/tests/unit/test_main.py":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"de61d28d55b23271954e12d437799abdac152aca","unresolved":false,"context_lines":[{"line_number":123,"context_line":"    @mock.patch.object(messageregistryfile,"},{"line_number":124,"context_line":"                       \u0027MessageRegistryFileCollection\u0027,"},{"line_number":125,"context_line":"                       autospec\u003dTrue)"},{"line_number":126,"context_line":"    def test__get_registry_collection(self,"},{"line_number":127,"context_line":"                                      MessageRegistryFileCollection_mock):"},{"line_number":128,"context_line":"        self.root._get_registry_collection()"},{"line_number":129,"context_line":"        MessageRegistryFileCollection_mock.assert_called_once_with("}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_7c6a2d18","line":126,"range":{"start_line":126,"start_character":38,"end_line":126,"end_character":42},"updated":"2018-07-31 13:06:32.000000000","message":"nit: may be move all args to the next line?","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"e903862d46c059eca3ebadb30f94489abfafa0f7","unresolved":false,"context_lines":[{"line_number":123,"context_line":"    @mock.patch.object(messageregistryfile,"},{"line_number":124,"context_line":"                       \u0027MessageRegistryFileCollection\u0027,"},{"line_number":125,"context_line":"                       autospec\u003dTrue)"},{"line_number":126,"context_line":"    def test__get_registry_collection(self,"},{"line_number":127,"context_line":"                                      MessageRegistryFileCollection_mock):"},{"line_number":128,"context_line":"        self.root._get_registry_collection()"},{"line_number":129,"context_line":"        MessageRegistryFileCollection_mock.assert_called_once_with("}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_a2c2dc84","line":126,"range":{"start_line":126,"start_character":38,"end_line":126,"end_character":42},"in_reply_to":"5f7c97a3_7c6a2d18","updated":"2018-07-31 13:53:27.000000000","message":"Done","commit_id":"c34b2188a96b07b77da0eaf728ed285d901e0358"}]}