)]}'
{"trove/guestagent/volume.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"956057c19a20250dc9f3ba910e32fcead2959ee0","unresolved":false,"context_lines":[{"line_number":102,"context_line":"        try:"},{"line_number":103,"context_line":"            # check if the device is mounted at mount_point before e2fsck"},{"line_number":104,"context_line":"            if not os.path.ismount(mount_point):"},{"line_number":105,"context_line":"                utils.execute(\"sudo\", \"e2fsck\", \"-f\", \"-p\", self.device_path)"},{"line_number":106,"context_line":"            utils.execute(\"sudo\", \"resize2fs\", self.device_path)"},{"line_number":107,"context_line":"        except ProcessExecutionError as err:"},{"line_number":108,"context_line":"            LOG.error(err)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2Ft%2Fk%3D","line":105,"updated":"2014-03-14 08:52:05.000000000","message":"out of scope of given bug report, sumbit another bug with full description of this issue.","commit_id":"d54bfb227db28d7426278d0b7a29c17f96851d25"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"8adea69528ab65fffbe65e4457d169efde6b77df","unresolved":false,"context_lines":[{"line_number":102,"context_line":"        try:"},{"line_number":103,"context_line":"            # check if the device is mounted at mount_point before e2fsck"},{"line_number":104,"context_line":"            if not os.path.ismount(mount_point):"},{"line_number":105,"context_line":"                utils.execute(\"sudo\", \"e2fsck\", \"-f\", \"-p\", self.device_path)"},{"line_number":106,"context_line":"            utils.execute(\"sudo\", \"resize2fs\", self.device_path)"},{"line_number":107,"context_line":"        except ProcessExecutionError as err:"},{"line_number":108,"context_line":"            LOG.error(err)"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAXH%2F%2Fda0%3D","line":105,"updated":"2014-04-21 15:52:06.000000000","message":"Should probably use run_as_root\u003dtrue, rather than specifying sudo directly.","commit_id":"de1591530f6b5f7e7c466eb9ca7fc32be44f2c95"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"d9625acfe81bacf60bab58e3aec6d14262fe4669","unresolved":false,"context_lines":[{"line_number":104,"context_line":"        try:"},{"line_number":105,"context_line":"            # check if the device is mounted at mount_point before e2fsck"},{"line_number":106,"context_line":"            if not os.path.ismount(mount_point):"},{"line_number":107,"context_line":"                utils.execute(\"sudo\", \"e2fsck\", \"-f\", \"-p\", self.device_path,"},{"line_number":108,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":109,"context_line":"            utils.execute(\"sudo\", \"resize2fs\", self.device_path,"},{"line_number":110,"context_line":"                          run_as_root\u003dTrue, root_helper\u003d\"sudo\")"}],"source_content_type":"text/x-python","patch_set":13,"id":"AAAAXH%2F%2FdKA%3D","line":107,"updated":"2014-04-21 16:23:41.000000000","message":"You forgot to take out the sudo.","commit_id":"eb0e6a491e98b9dd84e49163b2723f88324afca4"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"89475ce2368a55c729561e2610f216c0f595c4e4","unresolved":false,"context_lines":[{"line_number":107,"context_line":"                utils.execute(\"e2fsck\", \"-f\", \"-p\", self.device_path,"},{"line_number":108,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":109,"context_line":"            utils.execute(\"resize2fs\", self.device_path,"},{"line_number":110,"context_line":"                          run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":111,"context_line":"        except ProcessExecutionError as err:"},{"line_number":112,"context_line":"            LOG.error(err)"},{"line_number":113,"context_line":"            raise GuestError(\"Error resizing the filesystem: %s\" %"}],"source_content_type":"text/x-python","patch_set":14,"id":"AAAAXH%2F%2FDWw%3D","line":110,"updated":"2014-04-24 11:03:10.000000000","message":"i suggest to log the stderr outpul also to know why is it failed","commit_id":"50b1198c366f17c3e56c8480da02e5bd77578098"}],"trove/taskmanager/models.py":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"30fd9c64ed67dd2949ae43f4c9691dab2db8f22d","unresolved":false,"context_lines":[{"line_number":1069,"context_line":"                  \u0027id\u0027: self.instance.id, \u0027dev\u0027: CONF.device_path})"},{"line_number":1070,"context_line":"        self.instance.nova_client.volumes.create_server_volume(nova_server_id,"},{"line_number":1071,"context_line":"                                                               volume_id,"},{"line_number":1072,"context_line":"                                                               CONF.device_path)"},{"line_number":1073,"context_line":""},{"line_number":1074,"context_line":"        def volume_in_use():"},{"line_number":1075,"context_line":"            volume \u003d self.instance.volume_client.volumes.get("}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2FuWs%3D","line":1072,"updated":"2014-03-14 07:53:49.000000000","message":"Looks like the line \u003e 80 chars.","commit_id":"9a314108adb952c15097546f5a5cc434b2194ab0"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"d8f492e7b7f31ad341822f40886b139dda586c7b","unresolved":false,"context_lines":[{"line_number":1042,"context_line":""},{"line_number":1043,"context_line":"    @try_recover"},{"line_number":1044,"context_line":"    def _detach_volume(self):"},{"line_number":1045,"context_line":"        nova_server_id \u003d self.instance.server.id"},{"line_number":1046,"context_line":"        volume_id \u003d self.instance.volume_id"},{"line_number":1047,"context_line":"        LOG.debug(_(\"Detach volume %(vol_id)s from instance %(id)s\") % {"},{"line_number":1048,"context_line":"                  \u0027vol_id\u0027: volume_id,"}],"source_content_type":"text/x-python","patch_set":15,"id":"9ad9bd40_03138e0d","line":1045,"updated":"2014-05-05 17:24:42.000000000","message":"no need to allocate another one var, use self.instance.server.id directly","commit_id":"1552a476a5c7ae1b359ed2e4adbe1f29ffeb8d3e"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"d8f492e7b7f31ad341822f40886b139dda586c7b","unresolved":false,"context_lines":[{"line_number":1043,"context_line":"    @try_recover"},{"line_number":1044,"context_line":"    def _detach_volume(self):"},{"line_number":1045,"context_line":"        nova_server_id \u003d self.instance.server.id"},{"line_number":1046,"context_line":"        volume_id \u003d self.instance.volume_id"},{"line_number":1047,"context_line":"        LOG.debug(_(\"Detach volume %(vol_id)s from instance %(id)s\") % {"},{"line_number":1048,"context_line":"                  \u0027vol_id\u0027: volume_id,"},{"line_number":1049,"context_line":"                  \u0027id\u0027: self.instance.id})"}],"source_content_type":"text/x-python","patch_set":15,"id":"9ad9bd40_43f836c3","line":1046,"updated":"2014-05-05 17:24:42.000000000","message":"same","commit_id":"1552a476a5c7ae1b359ed2e4adbe1f29ffeb8d3e"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"d8f492e7b7f31ad341822f40886b139dda586c7b","unresolved":false,"context_lines":[{"line_number":1064,"context_line":""},{"line_number":1065,"context_line":"    @try_recover"},{"line_number":1066,"context_line":"    def _attach_volume(self):"},{"line_number":1067,"context_line":"        nova_server_id \u003d self.instance.server.id"},{"line_number":1068,"context_line":"        volume_id \u003d self.instance.volume_id"},{"line_number":1069,"context_line":"        LOG.debug(_(\"Attach volume %(vol_id)s to instance %(id)s at \""},{"line_number":1070,"context_line":"                  \"%(dev)s\") % {\u0027vol_id\u0027: volume_id,"}],"source_content_type":"text/x-python","patch_set":15,"id":"9ad9bd40_a32e6241","line":1067,"updated":"2014-05-05 17:24:42.000000000","message":"same","commit_id":"1552a476a5c7ae1b359ed2e4adbe1f29ffeb8d3e"}],"trove/tests/unittests/taskmanager/test_models.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"956057c19a20250dc9f3ba910e32fcead2959ee0","unresolved":false,"context_lines":[{"line_number":343,"context_line":"                          recover_func\u003dself.action._recover_mount_restart)"},{"line_number":344,"context_line":"        self.assertEqual(1, self.instance.guest.mount_volume.call_count)"},{"line_number":345,"context_line":"        self.assertEqual(1, self.instance.restart.call_count)"},{"line_number":346,"context_line":"        self.instance.nova_client.volumes.delete_server_volume.side_effect \u003d \\"},{"line_number":347,"context_line":"            None"},{"line_number":348,"context_line":"        self.instance.reset_mock()"},{"line_number":349,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2Ft%2Fs%3D","line":346,"updated":"2014-03-14 08:52:05.000000000","message":"avoid \\","commit_id":"d54bfb227db28d7426278d0b7a29c17f96851d25"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"956057c19a20250dc9f3ba910e32fcead2959ee0","unresolved":false,"context_lines":[{"line_number":357,"context_line":"        self.assertEqual(1, cnt)"},{"line_number":358,"context_line":"        self.assertEqual(1, self.instance.guest.mount_volume.call_count)"},{"line_number":359,"context_line":"        self.assertEqual(1, self.instance.restart.call_count)"},{"line_number":360,"context_line":"        self.instance.nova_client.volumes.create_server_volume.side_effect \u003d \\"},{"line_number":361,"context_line":"            None"},{"line_number":362,"context_line":"        self.instance.reset_mock()"},{"line_number":363,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2Ft%2Fo%3D","line":360,"updated":"2014-03-14 08:52:05.000000000","message":"avoid \\","commit_id":"d54bfb227db28d7426278d0b7a29c17f96851d25"},{"author":{"_account_id":6156,"name":"Justin Hopper","email":"justin.hopper@hp.com","username":"justin-hopper"},"change_message_id":"2a268c1ec567cd41721b0411eb1def2e4d7701d5","unresolved":false,"context_lines":[{"line_number":382,"context_line":"                          recover_func\u003dself.action._recover_mount_restart)"},{"line_number":383,"context_line":"        self.assertEqual(1, self.instance.guest.mount_volume.call_count)"},{"line_number":384,"context_line":"        self.assertEqual(1, self.instance.restart.call_count)"},{"line_number":385,"context_line":"        self.instance.nova_client.volumes.delete_server_volume.side_effect \u003d \\"},{"line_number":386,"context_line":"            None"},{"line_number":387,"context_line":"        self.instance.reset_mock()"},{"line_number":388,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"AAAAXX%2F%2F%2FWM%3D","line":385,"updated":"2014-04-24 20:00:01.000000000","message":"it maybe seem trivial but the pattern lately is to surround the right hand side with () so rather than use the \\.  The parens allow for the same possible indentation","commit_id":"50b1198c366f17c3e56c8480da02e5bd77578098"}]}