)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"786076ac60061f5df7daef868b0536d4c07bf411","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     wanghongxu \u003cwang19930902@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-12-06 15:42:08 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix AttributeError of dell emc driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"initiator_check is defined as a type of bool,"},{"line_number":10,"context_line":"but is used as a str type."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_6076fe72","line":7,"updated":"2017-12-11 09:54:36.000000000","message":"Please, start commit message from the upper-case letter","commit_id":"0889902e17146cbe12aba7157fa401781d71c73d"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"8cc4fd413615a17bd07ba8742d0ef90c2d42ab5d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     wanghongxu \u003cwang19930902@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-12-06 15:42:08 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix AttributeError of dell emc driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"initiator_check is defined as a type of bool,"},{"line_number":10,"context_line":"but is used as a str type."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_a048812c","line":7,"in_reply_to":"df87a7cf_6076fe72","updated":"2017-12-11 14:05:13.000000000","message":"Also best to start with the driver name.","commit_id":"0889902e17146cbe12aba7157fa401781d71c73d"}],"cinder/volume/drivers/dell_emc/vmax/common.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"786076ac60061f5df7daef868b0536d4c07bf411","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        :returns:  flag"},{"line_number":166,"context_line":"        \"\"\""},{"line_number":167,"context_line":"        initiator_check \u003d self.configuration.safe_get(\u0027initiator_check\u0027)"},{"line_number":168,"context_line":"        return initiator_check or False"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    def _get_replication_info(self):"},{"line_number":171,"context_line":"        \"\"\"Gather replication information, if provided.\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_80cd4286","line":168,"updated":"2017-12-11 09:54:36.000000000","message":"A default value is False, we can just do \u0027initiator_check\u0027 do \u0027return self.configuration.safe_get(\u0027initiator_check\u0027)\u0027 or even mark this method as @property","commit_id":"0889902e17146cbe12aba7157fa401781d71c73d"}]}