)]}'
{"cinder/brick/local_dev/lvm.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"735e0b10e8ab0257d3a637393e4fa487d1175764","unresolved":false,"context_lines":[{"line_number":690,"context_line":"            self._execute(*cmd,"},{"line_number":691,"context_line":"                          root_helper\u003dself._root_helper,"},{"line_number":692,"context_line":"                          run_as_root\u003dTrue)"},{"line_number":693,"context_line":"        except putils.ProcessExecutionError as err:"},{"line_number":694,"context_line":"            LOG.exception(\u0027Error activating LV\u0027)"},{"line_number":695,"context_line":"            LOG.error(\u0027Cmd     :%s\u0027, err.cmd)"},{"line_number":696,"context_line":"            LOG.error(\u0027StdOut  :%s\u0027, err.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_83c11857","line":693,"updated":"2017-10-09 15:14:58.000000000","message":"How about only retrying if the exit code is 5? That\u0027s what you\u0027re hitting:\n\nhttp://logs.openstack.org/75/508175/4/check/gate-tempest-dsvm-py35-ubuntu-xenial/d46ef74/logs/screen-c-vol.txt#_Oct_09_12_08_37_819653\n\nAlso, would retrying with --refresh potentially help?\n\nAlso, how about running lvdisplay to dump some information at the point of failure before giving up?","commit_id":"a2b7b8d74a2dded1a38354321f5627d7375d7ee2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"33ea91df79705a955873011a391dd26ee1c61922","unresolved":false,"context_lines":[{"line_number":691,"context_line":"                          root_helper\u003dself._root_helper,"},{"line_number":692,"context_line":"                          run_as_root\u003dTrue)"},{"line_number":693,"context_line":"        except putils.ProcessExecutionError as err:"},{"line_number":694,"context_line":"            LOG.exception(\u0027Error activating LV\u0027)"},{"line_number":695,"context_line":"            LOG.error(\u0027Cmd     :%s\u0027, err.cmd)"},{"line_number":696,"context_line":"            LOG.error(\u0027StdOut  :%s\u0027, err.stdout)"},{"line_number":697,"context_line":"            LOG.error(\u0027StdErr  :%s\u0027, err.stderr)"},{"line_number":698,"context_line":"            raise"},{"line_number":699,"context_line":""},{"line_number":700,"context_line":"    @utils.retry(putils.ProcessExecutionError)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_63b4dc82","line":697,"range":{"start_line":694,"start_character":12,"end_line":697,"end_character":48},"updated":"2017-10-09 15:30:35.000000000","message":"Would be nice to not dump a stacktrace until the end when you actually timeout, otherwise this is noise in the logs. That means you\u0027d have to either not use the decorator, or do it on an inner method.","commit_id":"a2b7b8d74a2dded1a38354321f5627d7375d7ee2"}]}