)]}'
{"cinder/volume/api.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"289754f74f90b951b97de4d9a09632699a439993","unresolved":false,"context_lines":[{"line_number":1432,"context_line":"        # Build required conditions for conditional update"},{"line_number":1433,"context_line":"        expected \u003d {\u0027status\u0027: (\u0027available\u0027, \u0027in-use\u0027),"},{"line_number":1434,"context_line":"                    \u0027migration_status\u0027: self.AVAILABLE_MIGRATION_STATUS,"},{"line_number":1435,"context_line":"                    \u0027replication_status\u0027: (None, \u0027disabled\u0027, \u0027not-capable\u0027),"},{"line_number":1436,"context_line":"                    \u0027consistencygroup_id\u0027: (None, \u0027\u0027),"},{"line_number":1437,"context_line":"                    \u0027group_id\u0027: (None, \u0027\u0027)}"},{"line_number":1438,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_a8f28898","line":1435,"updated":"2017-10-26 03:30:00.000000000","message":"Can you please use ReplicationStatus defined in fields.py for \u0027not-capable\u0027 and \u0027disabled\u0027?","commit_id":"4ed1b30e060b54500787d0f4dbf441e45abe08d6"},{"author":{"_account_id":11186,"name":"yixuan zhang","email":"yixuan_z@hotmail.com","username":"yixuanzhang"},"change_message_id":"3ddcb0f64c83e703c1ccc83a375183a97d63ffc8","unresolved":false,"context_lines":[{"line_number":1432,"context_line":"        # Build required conditions for conditional update"},{"line_number":1433,"context_line":"        expected \u003d {\u0027status\u0027: (\u0027available\u0027, \u0027in-use\u0027),"},{"line_number":1434,"context_line":"                    \u0027migration_status\u0027: self.AVAILABLE_MIGRATION_STATUS,"},{"line_number":1435,"context_line":"                    \u0027replication_status\u0027: (None, \u0027disabled\u0027, \u0027not-capable\u0027),"},{"line_number":1436,"context_line":"                    \u0027consistencygroup_id\u0027: (None, \u0027\u0027),"},{"line_number":1437,"context_line":"                    \u0027group_id\u0027: (None, \u0027\u0027)}"},{"line_number":1438,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_d439daf1","line":1435,"in_reply_to":"3f4b6375_a8f28898","updated":"2017-10-26 06:39:47.000000000","message":"Hi Xing, thanks a lot for your comment. It\u0027s addressed in the new patch.","commit_id":"4ed1b30e060b54500787d0f4dbf441e45abe08d6"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9e8ea4d5d1b46644d658314ce84401a196bc804a","unresolved":false,"context_lines":[{"line_number":1434,"context_line":"                    \u0027migration_status\u0027: self.AVAILABLE_MIGRATION_STATUS,"},{"line_number":1435,"context_line":"                    \u0027replication_status\u0027: (None,"},{"line_number":1436,"context_line":"                                           fields.ReplicationStatus.DISABLED,"},{"line_number":1437,"context_line":"                                           fields.ReplicationStatus."},{"line_number":1438,"context_line":"                                           NOT_CAPABLE),"},{"line_number":1439,"context_line":"                    \u0027consistencygroup_id\u0027: (None, \u0027\u0027),"},{"line_number":1440,"context_line":"                    \u0027group_id\u0027: (None, \u0027\u0027)}"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_15c3da58","line":1437,"updated":"2017-10-26 18:56:41.000000000","message":"Adjusting the indentation here so you don\u0027t have to split the field name into two lines would be preferred: this is a pain to grep for.","commit_id":"5f5650e1944bb8ae82cfc26a364cea0b56cf5760"},{"author":{"_account_id":11186,"name":"yixuan zhang","email":"yixuan_z@hotmail.com","username":"yixuanzhang"},"change_message_id":"4e5ef7ab371ce89215b555e02058eb3df46cae2b","unresolved":false,"context_lines":[{"line_number":1434,"context_line":"                    \u0027migration_status\u0027: self.AVAILABLE_MIGRATION_STATUS,"},{"line_number":1435,"context_line":"                    \u0027replication_status\u0027: (None,"},{"line_number":1436,"context_line":"                                           fields.ReplicationStatus.DISABLED,"},{"line_number":1437,"context_line":"                                           fields.ReplicationStatus."},{"line_number":1438,"context_line":"                                           NOT_CAPABLE),"},{"line_number":1439,"context_line":"                    \u0027consistencygroup_id\u0027: (None, \u0027\u0027),"},{"line_number":1440,"context_line":"                    \u0027group_id\u0027: (None, \u0027\u0027)}"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_32df026a","line":1437,"in_reply_to":"3f4b6375_15c3da58","updated":"2017-10-27 03:02:50.000000000","message":"Hi Eric, thanks a lot for your comment, it is addressed in the new patch","commit_id":"5f5650e1944bb8ae82cfc26a364cea0b56cf5760"}]}