)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"271c015d0e74abe6730516657fceb58e8ceba28c","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The DRBD Manage\u0027 DrbdManageClientHelp class is designed to accumulate"},{"line_number":10,"context_line":"code that is common across various DRBD Manage clients; using this"},{"line_number":11,"context_line":"shared code eases maintenance."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ie33082d4c99b33d1af8bfcdc5842e11d85988064"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3ac371cc_9cc72463","line":11,"updated":"2016-08-18 15:36:58.000000000","message":"If there are other projects that might benefit from this refactoring, it\u0027s worth creating a blueprint so each of the affected patches and projects can track against that same blueprint for this effort.","commit_id":"ea9c7c8f363129768e102715706498ca1b56c65c"},{"author":{"_account_id":14339,"name":"Philipp Marek","email":"philipp.marek@linbit.com","username":"pmarek"},"change_message_id":"d92d4650a329ebed03c3ec8c89f6e3cee99b4f1a","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The DRBD Manage\u0027 DrbdManageClientHelp class is designed to accumulate"},{"line_number":10,"context_line":"code that is common across various DRBD Manage clients; using this"},{"line_number":11,"context_line":"shared code eases maintenance."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ie33082d4c99b33d1af8bfcdc5842e11d85988064"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3ac371cc_a2a61551","line":11,"in_reply_to":"3ac371cc_9cc72463","updated":"2016-08-19 06:39:35.000000000","message":"Sorry about the misunderstanding.\n\nThe other clients for DRBD Manage are not OpenStack projects, but other cloud-, storage-, and virtualization platforms.","commit_id":"ea9c7c8f363129768e102715706498ca1b56c65c"}],"cinder/volume/drivers/drbdmanagedrv.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"783ff1063482b39fb632256c6a3e905107f6ae4e","unresolved":false,"context_lines":[{"line_number":231,"context_line":"    def sleep(self, amount):"},{"line_number":232,"context_line":"        eventlet.sleep(amount)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def _LW(self, msg):"},{"line_number":235,"context_line":"        # This looks strange, but avoids the \"flake8\" error message"},{"line_number":236,"context_line":"        #   H701  Empty localization string"},{"line_number":237,"context_line":"        l \u003d cinder.i18n._LW"}],"source_content_type":"text/x-python","patch_set":7,"id":"9ad45d7e_eff110cf","line":234,"updated":"2016-08-08 22:22:31.000000000","message":"this is really strange to do this.  Why can\u0027t  you do what everyone else does and use the markers normally in your code.","commit_id":"8882d64fb3a2b5d4ae0df15190e7a4fcf6bbaf1c"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"783ff1063482b39fb632256c6a3e905107f6ae4e","unresolved":false,"context_lines":[{"line_number":729,"context_line":"        if not d_res_name:"},{"line_number":730,"context_line":"            # resource already gone?"},{"line_number":731,"context_line":"            LOG.warning(self._LW(\"snapshot: %s not found, \""},{"line_number":732,"context_line":"                                 \"skipping delete operation\"),"},{"line_number":733,"context_line":"                        snapshot[\u0027id\u0027])"},{"line_number":734,"context_line":"            LOG.info(self._LI(\u0027Successfully deleted snapshot: %s\u0027),"},{"line_number":735,"context_line":"                     snapshot[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":7,"id":"9ad45d7e_4fb93ce9","line":732,"updated":"2016-08-08 22:22:31.000000000","message":"_LW() instead just like the rest of Cinder does.","commit_id":"8882d64fb3a2b5d4ae0df15190e7a4fcf6bbaf1c"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"271c015d0e74abe6730516657fceb58e8ceba28c","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    clienthelper \u003d None"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    class dm_c_h(object):"},{"line_number":64,"context_line":"        odm \u003d None"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        def _fetch_answer_data(**args):"}],"source_content_type":"text/x-python","patch_set":9,"id":"3ac371cc_7c706840","line":63,"updated":"2016-08-18 15:36:58.000000000","message":"can we create a more meaningful and readable name than this please?","commit_id":"ea9c7c8f363129768e102715706498ca1b56c65c"},{"author":{"_account_id":14339,"name":"Philipp Marek","email":"philipp.marek@linbit.com","username":"pmarek"},"change_message_id":"d92d4650a329ebed03c3ec8c89f6e3cee99b4f1a","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    clienthelper \u003d None"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    class dm_c_h(object):"},{"line_number":64,"context_line":"        odm \u003d None"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        def _fetch_answer_data(**args):"}],"source_content_type":"text/x-python","patch_set":9,"id":"3ac371cc_e2348dfc","line":63,"in_reply_to":"3ac371cc_7c706840","updated":"2016-08-19 06:39:35.000000000","message":"\"dm_c_h\" is the short form for a (faked) DrbdManageClientHelper, in case DRBD Manage isn\u0027t\ninstalled.","commit_id":"ea9c7c8f363129768e102715706498ca1b56c65c"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"271c015d0e74abe6730516657fceb58e8ceba28c","unresolved":false,"context_lines":[{"line_number":145,"context_line":"CS_UPD_CON \u003d None"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"class DrbdManageBaseDriver(driver.VolumeDriver, dm_c_h):"},{"line_number":149,"context_line":"    \"\"\"Cinder driver that uses DRBDmanage for storage.\"\"\""},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"    VERSION \u003d \u00271.1.1\u0027"}],"source_content_type":"text/x-python","patch_set":9,"id":"3ac371cc_bce2e0d2","line":148,"updated":"2016-08-18 15:36:58.000000000","message":"I don\u0027t know what dm_c_h is, and it\u0027s not very readable.","commit_id":"ea9c7c8f363129768e102715706498ca1b56c65c"},{"author":{"_account_id":14339,"name":"Philipp Marek","email":"philipp.marek@linbit.com","username":"pmarek"},"change_message_id":"d92d4650a329ebed03c3ec8c89f6e3cee99b4f1a","unresolved":false,"context_lines":[{"line_number":145,"context_line":"CS_UPD_CON \u003d None"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"class DrbdManageBaseDriver(driver.VolumeDriver, dm_c_h):"},{"line_number":149,"context_line":"    \"\"\"Cinder driver that uses DRBDmanage for storage.\"\"\""},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"    VERSION \u003d \u00271.1.1\u0027"}],"source_content_type":"text/x-python","patch_set":9,"id":"3ac371cc_02172955","line":148,"in_reply_to":"3ac371cc_bce2e0d2","updated":"2016-08-19 06:39:35.000000000","message":"The DrbdManageClientHelper class.\n\nWill rename.","commit_id":"ea9c7c8f363129768e102715706498ca1b56c65c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"1a8922d8eb9ca5ec585a6b96d0c5bb1ad5576437","unresolved":false,"context_lines":[{"line_number":46,"context_line":"try:"},{"line_number":47,"context_line":"    import dbus"},{"line_number":48,"context_line":"    from drbdmanage import clienthelper"},{"line_number":49,"context_line":"    from drbdmanage.clienthelper import DrbdManageClientHelper \\"},{"line_number":50,"context_line":"        as dm_client_helper"},{"line_number":51,"context_line":"    import drbdmanage.consts as dm_const"},{"line_number":52,"context_line":"    import drbdmanage.exceptions as dm_exc"}],"source_content_type":"text/x-python","patch_set":13,"id":"da6895a0_c65ce7ef","line":49,"range":{"start_line":49,"start_character":40,"end_line":49,"end_character":62},"updated":"2016-10-25 13:23:50.000000000","message":"I believe this is a class? According to guidelines the module should be imported and not the individual class. That can also get rid of the use of \\ for multiline.","commit_id":"20ad493b1f9843c96a18268766025c7b864bd7de"}]}