{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/47233","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/47233/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/47233/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/47233/events","html_url":"https://github.com/saltstack/salt/pull/47233","id":316620579,"number":47233,"title":"Handle yum.Errors.YumBaseError","user":{"login":"leeclemens","id":1250267,"avatar_url":"https://avatars1.githubusercontent.com/u/1250267?v=4","gravatar_id":"","url":"https://api.github.com/users/leeclemens","html_url":"https://github.com/leeclemens","followers_url":"https://api.github.com/users/leeclemens/followers","following_url":"https://api.github.com/users/leeclemens/following{/other_user}","gists_url":"https://api.github.com/users/leeclemens/gists{/gist_id}","starred_url":"https://api.github.com/users/leeclemens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/leeclemens/subscriptions","organizations_url":"https://api.github.com/users/leeclemens/orgs","repos_url":"https://api.github.com/users/leeclemens/repos","events_url":"https://api.github.com/users/leeclemens/events{/privacy}","received_events_url":"https://api.github.com/users/leeclemens/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-04-22T22:52:50Z","updated_at":"2018-05-01T17:48:47Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/saltstack/salt/pulls/47233","html_url":"https://github.com/saltstack/salt/pull/47233","diff_url":"https://github.com/saltstack/salt/pull/47233.diff","patch_url":"https://github.com/saltstack/salt/pull/47233.patch"},"body":"Happens at least in event rpmdb is corrupt/cannot be opened\r\n\r\n### What does this PR do?\r\n\r\nExcepts `yum.Errors.YumBaseError` and raises `CommandExecutionError` for consistency and simplified display\r\n\r\n### What issues does this PR fix or reference?\r\n\r\nFixes #44331\r\n\r\n### Previous Behavior\r\n\r\nFull Traceback was printed in Comment\r\n\r\n### New Behavior\r\n\r\n`CommandExecutionError` is raised with simplified message from the original `yum.Errors.YumBaseError`\r\n\r\n### Tests written?\r\n\r\nNo\r\n\r\n### Commits signed with GPG?\r\n\r\nYes\r\n\r\nPlease review [Salt's Contributing Guide](https://docs.saltstack.com/en/latest/topics/development/contributing.html) for best practices.\r\n\r\nSee GitHub's [page on GPG signing](https://help.github.com/articles/signing-commits-using-gpg/) for more information about signing commits with GPG.\r\n"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/385738149","html_url":"https://github.com/saltstack/salt/pull/47233#issuecomment-385738149","issue_url":"https://api.github.com/repos/saltstack/salt/issues/47233","id":385738149,"user":{"login":"leeclemens","id":1250267,"avatar_url":"https://avatars1.githubusercontent.com/u/1250267?v=4","gravatar_id":"","url":"https://api.github.com/users/leeclemens","html_url":"https://github.com/leeclemens","followers_url":"https://api.github.com/users/leeclemens/followers","following_url":"https://api.github.com/users/leeclemens/following{/other_user}","gists_url":"https://api.github.com/users/leeclemens/gists{/gist_id}","starred_url":"https://api.github.com/users/leeclemens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/leeclemens/subscriptions","organizations_url":"https://api.github.com/users/leeclemens/orgs","repos_url":"https://api.github.com/users/leeclemens/repos","events_url":"https://api.github.com/users/leeclemens/events{/privacy}","received_events_url":"https://api.github.com/users/leeclemens/received_events","type":"User","site_admin":false},"created_at":"2018-05-01T17:48:47Z","updated_at":"2018-05-01T17:48:47Z","author_association":"CONTRIBUTOR","body":"@cachedout I just want to clarify, as this has gotten a little confusing, but I was unable to reproduce the OP's issue with the current version. Although this fix shouldn't introduce any issues, it doesn't necessary resolve any either. I haven't debugged it, but it seems the error is gracefully handled somewhere before the `except` block I added."}}
{"actor":{"display_login":"leeclemens"}}