)]}'
{"cinder/volume/drivers/hpe/hpe_3par_common.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"713ae7229658f0016504f95222d3039c33abd09e","unresolved":false,"context_lines":[{"line_number":4225,"context_line":""},{"line_number":4226,"context_line":"        # Check and see if we are in periodic mode. If we are, update"},{"line_number":4227,"context_line":"        # Remote Copy Group to have a sync period."},{"line_number":4228,"context_line":"        if len(rcg[\u0027volumes\u0027]) and \u0027syncPeriod\u0027 in rcg[\u0027targets\u0027][0]:"},{"line_number":4229,"context_line":"            if replication_sync_period !\u003d int(rcg[\u0027targets\u0027][0][\u0027syncPeriod\u0027]):"},{"line_number":4230,"context_line":"                for target in self._replication_targets:"},{"line_number":4231,"context_line":"                    if target[\u0027replication_mode\u0027] \u003d\u003d replication_mode_num:"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f93b717_59e3cba4","line":4228,"range":{"start_line":4228,"start_character":48,"end_line":4228,"end_character":69},"updated":"2018-02-01 03:16:47.000000000","message":"Do we need to check the length of targets here?","commit_id":"8039c39e8d468d833e5f335614dcd14769727108"},{"author":{"_account_id":24952,"name":"Vivek Soni","email":"viveksoni31390@gmail.com","username":"sonivi"},"change_message_id":"6a3790f3e2d9bdb40bc2aa73aec7c4d6b7684d5b","unresolved":false,"context_lines":[{"line_number":4225,"context_line":""},{"line_number":4226,"context_line":"        # Check and see if we are in periodic mode. If we are, update"},{"line_number":4227,"context_line":"        # Remote Copy Group to have a sync period."},{"line_number":4228,"context_line":"        if len(rcg[\u0027volumes\u0027]) and \u0027syncPeriod\u0027 in rcg[\u0027targets\u0027][0]:"},{"line_number":4229,"context_line":"            if replication_sync_period !\u003d int(rcg[\u0027targets\u0027][0][\u0027syncPeriod\u0027]):"},{"line_number":4230,"context_line":"                for target in self._replication_targets:"},{"line_number":4231,"context_line":"                    if target[\u0027replication_mode\u0027] \u003d\u003d replication_mode_num:"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f93b717_949a7ab6","line":4228,"range":{"start_line":4228,"start_character":48,"end_line":4228,"end_character":69},"in_reply_to":"5f93b717_59e3cba4","updated":"2018-02-01 03:46:33.000000000","message":"This function will be called in case of replication only, which will always have targets. so no need to check its length here","commit_id":"8039c39e8d468d833e5f335614dcd14769727108"}]}
