)]}'
{"os_brick/tests/initiator/connectors/test_rbd.py":[{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"d061b2dbf520ac48ee614d12e6efa1d3e2944eeb","unresolved":false,"context_lines":[{"line_number":63,"context_line":"        # Ensure rados is instantiated correctly"},{"line_number":64,"context_line":"        mock_rados.Rados.assert_called_once_with("},{"line_number":65,"context_line":"            clustername\u003d\u0027ceph\u0027,"},{"line_number":66,"context_line":"            rados_id\u003dself.user,"},{"line_number":67,"context_line":"            conffile\u003d\u0027/etc/ceph/ceph.conf\u0027)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        # Ensure correct calls to connect to cluster"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_3b11f304","line":66,"updated":"2016-08-11 18:27:18.000000000","message":"I don\u0027t understand why you didn\u0027t replace with utils.convert_str() ?","commit_id":"ce5d75d184b94ac59df2b95a2946d33a3638d11c"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"b3dd05b2db3165d0b745cf36f81285c23cfadb24","unresolved":false,"context_lines":[{"line_number":63,"context_line":"        # Ensure rados is instantiated correctly"},{"line_number":64,"context_line":"        mock_rados.Rados.assert_called_once_with("},{"line_number":65,"context_line":"            clustername\u003d\u0027ceph\u0027,"},{"line_number":66,"context_line":"            rados_id\u003dself.user,"},{"line_number":67,"context_line":"            conffile\u003d\u0027/etc/ceph/ceph.conf\u0027)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        # Ensure correct calls to connect to cluster"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_ef6b1583","line":66,"in_reply_to":"9ad45d7e_3b11f304","updated":"2016-08-11 19:58:35.000000000","message":"Makes sense, I\u0027ll fix it in the next patch","commit_id":"ce5d75d184b94ac59df2b95a2946d33a3638d11c"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"759efd1a74cc4a51b7887437307e26e524b050dc","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        # Ensure rados is instantiated correctly"},{"line_number":65,"context_line":"        mock_rados.Rados.assert_called_once_with("},{"line_number":66,"context_line":"            clustername\u003dutils.convert_str(\u0027ceph\u0027),"},{"line_number":67,"context_line":"            rados_id\u003dself.user,"},{"line_number":68,"context_line":"            conffile\u003d\u0027/etc/ceph/ceph.conf\u0027)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # Ensure correct calls to connect to cluster"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_8a3887d4","line":67,"updated":"2016-08-11 20:15:33.000000000","message":"I think you meant to change rados_id, not clustername:\nrados_id\u003dutils.convert_str(self.user),","commit_id":"53ab0a11e92e76a50468e46a776a50866abce0d2"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"1b878d4f5ea801374c7d3cd4563d401873bbd332","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        # Ensure rados is instantiated correctly"},{"line_number":65,"context_line":"        mock_rados.Rados.assert_called_once_with("},{"line_number":66,"context_line":"            clustername\u003dutils.convert_str(\u0027ceph\u0027),"},{"line_number":67,"context_line":"            rados_id\u003dself.user,"},{"line_number":68,"context_line":"            conffile\u003d\u0027/etc/ceph/ceph.conf\u0027)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # Ensure correct calls to connect to cluster"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_eac78361","line":67,"in_reply_to":"9ad45d7e_8a3887d4","updated":"2016-08-11 20:24:57.000000000","message":"You\u0027re right. I have to stop working too late","commit_id":"53ab0a11e92e76a50468e46a776a50866abce0d2"}],"os_brick/tests/initiator/test_linuxrbd.py":[{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"d061b2dbf520ac48ee614d12e6efa1d3e2944eeb","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"            # Verify object attributes are assigned as expected"},{"line_number":30,"context_line":"            self.assertEqual(\u0027/etc/ceph/ceph.conf\u0027, client.rbd_conf)"},{"line_number":31,"context_line":"            self.assertEqual(\u0027test_user\u0027, client.rbd_user)"},{"line_number":32,"context_line":"            self.assertEqual(\u0027test_pool\u0027, client.rbd_pool)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"            # Assert connect is called with correct paramaters"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_fbbf5bcd","line":31,"updated":"2016-08-11 18:27:18.000000000","message":"same question, why don\u0027t you use utils.convert_str() here?","commit_id":"ce5d75d184b94ac59df2b95a2946d33a3638d11c"}]}