)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"ceb4d6b0546f42b5e0878056eea28e5e21bdd77d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sean McGinnis \u003csean_mcginnis@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-01-20 16:50:19 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve debug logging of Storage Center driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding debug message and expanding some existing log statements"},{"line_number":10,"context_line":"to make debugging and auditing easier. Some interesting calls"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a930d6b_8688ed23","line":7,"updated":"2015-01-21 12:42:03.000000000","message":"Could you please add \u0027Dell: \u0027 at the beginning of commit message title?","commit_id":"f846f6052aaa6c9c419185201dfe77a56373581e"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5894909f4063151620e26fdfd7b72ac25921f5fa","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sean McGinnis \u003csean_mcginnis@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-01-20 16:50:19 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improve debug logging of Storage Center driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding debug message and expanding some existing log statements"},{"line_number":10,"context_line":"to make debugging and auditing easier. Some interesting calls"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a930d6b_dfec74ca","line":7,"in_reply_to":"1a930d6b_8688ed23","updated":"2015-01-21 14:12:39.000000000","message":"Done","commit_id":"f846f6052aaa6c9c419185201dfe77a56373581e"}],"cinder/volume/drivers/dell/dell_storagecenter_common.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"5bfcae72acb14184686e9beff2f2537a8afcd34b","unresolved":false,"context_lines":[{"line_number":234,"context_line":"        \u0027\u0027\u0027delete_snapshot\u0027\u0027\u0027"},{"line_number":235,"context_line":"        volume_name \u003d snapshot.get(\u0027volume_id\u0027)"},{"line_number":236,"context_line":"        snapshot_id \u003d snapshot.get(\u0027id\u0027)"},{"line_number":237,"context_line":"        LOG.debug(\u0027Deleting snapshot %(s)s from volume %(v)s\u0027,"},{"line_number":238,"context_line":"                  {\u0027s\u0027: snapshot_id,"},{"line_number":239,"context_line":"                   \u0027v\u0027: volume_name})"},{"line_number":240,"context_line":"        with self._client.open_connection() as api:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_c8404e38","line":237,"updated":"2015-01-21 05:26:51.000000000","message":"Can you change \"s\" to \"snap\" and \"v\" to \"vol\"?","commit_id":"f846f6052aaa6c9c419185201dfe77a56373581e"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5894909f4063151620e26fdfd7b72ac25921f5fa","unresolved":false,"context_lines":[{"line_number":234,"context_line":"        \u0027\u0027\u0027delete_snapshot\u0027\u0027\u0027"},{"line_number":235,"context_line":"        volume_name \u003d snapshot.get(\u0027volume_id\u0027)"},{"line_number":236,"context_line":"        snapshot_id \u003d snapshot.get(\u0027id\u0027)"},{"line_number":237,"context_line":"        LOG.debug(\u0027Deleting snapshot %(s)s from volume %(v)s\u0027,"},{"line_number":238,"context_line":"                  {\u0027s\u0027: snapshot_id,"},{"line_number":239,"context_line":"                   \u0027v\u0027: volume_name})"},{"line_number":240,"context_line":"        with self._client.open_connection() as api:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_dfb554ce","line":237,"in_reply_to":"1a930d6b_c8404e38","updated":"2015-01-21 14:12:39.000000000","message":"Done","commit_id":"f846f6052aaa6c9c419185201dfe77a56373581e"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"5bfcae72acb14184686e9beff2f2537a8afcd34b","unresolved":false,"context_lines":[{"line_number":294,"context_line":"        \u0027\u0027\u0027Extend the size of the volume.\u0027\u0027\u0027"},{"line_number":295,"context_line":"        volume_name \u003d volume.get(\u0027id\u0027)"},{"line_number":296,"context_line":"        LOG.debug(\u0027Extending volume %(v)s to %(s)s\u0027,"},{"line_number":297,"context_line":"                  {\u0027v\u0027: volume_name, \u0027s\u0027: new_size})"},{"line_number":298,"context_line":"        if volume is not None:"},{"line_number":299,"context_line":"            with self._client.open_connection() as api:"},{"line_number":300,"context_line":"                ssn \u003d api.find_sc(self.configuration.dell_sc_ssn)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_88572602","line":297,"updated":"2015-01-21 05:26:51.000000000","message":"Same comment here regarding variable names. Can you change \"v\" to \"vol\" and \"s\" to \"size\"?  It will be easier to read if the variable name is meaningful.","commit_id":"f846f6052aaa6c9c419185201dfe77a56373581e"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5894909f4063151620e26fdfd7b72ac25921f5fa","unresolved":false,"context_lines":[{"line_number":294,"context_line":"        \u0027\u0027\u0027Extend the size of the volume.\u0027\u0027\u0027"},{"line_number":295,"context_line":"        volume_name \u003d volume.get(\u0027id\u0027)"},{"line_number":296,"context_line":"        LOG.debug(\u0027Extending volume %(v)s to %(s)s\u0027,"},{"line_number":297,"context_line":"                  {\u0027v\u0027: volume_name, \u0027s\u0027: new_size})"},{"line_number":298,"context_line":"        if volume is not None:"},{"line_number":299,"context_line":"            with self._client.open_connection() as api:"},{"line_number":300,"context_line":"                ssn \u003d api.find_sc(self.configuration.dell_sc_ssn)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_1fc7dc46","line":297,"in_reply_to":"1a930d6b_88572602","updated":"2015-01-21 14:12:39.000000000","message":"Done","commit_id":"f846f6052aaa6c9c419185201dfe77a56373581e"}]}
