)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"549a4106c3133ffbce574e6777dca02127647714","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NEC driver reads default_backends"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"NEC driver reads default configuration values from default_backends."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I441e4c1776db663e942c6c8e1995a0ac6971d80f"},{"line_number":12,"context_line":"Closes-Bug: #1705001"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df3967d1_c6a4e9bb","line":9,"updated":"2017-08-02 17:21:23.000000000","message":"I\u0027d like to see what the problem is and what the fix is included in this patch.","commit_id":"c8492e5846caf78b5691fd01ba0c15dadd90feae"}],"cinder/volume/drivers/nec/volume_common.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5410b17769c3571b3736777da757c83e7bd7da44","unresolved":false,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        confobj \u003d self._configuration"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        # The parameter `nec_ism_cli_fip` and `nec_pools` must be read from"},{"line_number":255,"context_line":"        # oslo_config.cfg, otherwise they cannot be read in migrate_volume()."},{"line_number":256,"context_line":"        return {"},{"line_number":257,"context_line":"            \u0027cli_fip\u0027: conf.nec_ismcli_fip,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf3e73b7_ae1ebe6d","line":254,"range":{"start_line":254,"start_character":25,"end_line":254,"end_character":40},"updated":"2017-08-07 12:05:22.000000000","message":"This doesn\u0027t seem right. All backend setting should be in the backend config or the backend_defaults.","commit_id":"b4370b733b3bbf55966f400c734b1cf10bdf048e"},{"author":{"_account_id":23885,"name":"Shunei Shiono","email":"shunei.shiono@gmail.com","username":"s-shiono"},"change_message_id":"49a32e6ae9c8819f1a0cdb0723d00d19adf7814b","unresolved":false,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        confobj \u003d self._configuration"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        # The parameter `nec_ism_cli_fip` and `nec_pools` must be read from"},{"line_number":255,"context_line":"        # oslo_config.cfg, otherwise they cannot be read in migrate_volume()."},{"line_number":256,"context_line":"        return {"},{"line_number":257,"context_line":"            \u0027cli_fip\u0027: conf.nec_ismcli_fip,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_78bd5567","line":254,"range":{"start_line":254,"start_character":25,"end_line":254,"end_character":40},"in_reply_to":"bf3e73b7_ae1ebe6d","updated":"2017-08-08 04:38:47.000000000","message":"Yes, our migration implementation should be improved.\nWe are looking for a better way for reading the parameters.\nWe would like to fix our ci first by this patch.","commit_id":"b4370b733b3bbf55966f400c734b1cf10bdf048e"}]}