{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/36065","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/36065/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/36065/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/36065/events","html_url":"https://github.com/saltstack/salt/pull/36065","id":175208675,"number":36065,"title":"[FEATURE] retcode passthrough exitstatus (take 3)","user":{"login":"plastikos","id":3281727,"avatar_url":"https://avatars.githubusercontent.com/u/3281727?v=3","gravatar_id":"","url":"https://api.github.com/users/plastikos","html_url":"https://github.com/plastikos","followers_url":"https://api.github.com/users/plastikos/followers","following_url":"https://api.github.com/users/plastikos/following{/other_user}","gists_url":"https://api.github.com/users/plastikos/gists{/gist_id}","starred_url":"https://api.github.com/users/plastikos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plastikos/subscriptions","organizations_url":"https://api.github.com/users/plastikos/orgs","repos_url":"https://api.github.com/users/plastikos/repos","events_url":"https://api.github.com/users/plastikos/events{/privacy}","received_events_url":"https://api.github.com/users/plastikos/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-09-06T10:06:26Z","updated_at":"2016-09-14T19:22:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/saltstack/salt/pulls/36065","html_url":"https://github.com/saltstack/salt/pull/36065","diff_url":"https://github.com/saltstack/salt/pull/36065.diff","patch_url":"https://github.com/saltstack/salt/pull/36065.patch"},"body":"### What does this PR do?\r\nIt passes the `retcode` through to the `salt` and `salt-call` executable exit status.\r\n\r\nThis is a rebase of #35282\r\nThis no longer requires commit back-outs used in #35986 \r\n\r\n### What issues does this PR fix or reference?\r\n#18510 \r\n\r\n### Previous Behavior\r\nThe `retcode` from states was not reflected in the command line executable exit status\r\n\r\n### New Behavior\r\nNon-zero `retcode` from salt states is used for the exit status of `salt` and `salt-call` - unless a \"higher priority\" error occurs in the specific executables.\r\n\r\n### Tests written?\r\nYes\r\n"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/247125175","html_url":"https://github.com/saltstack/salt/pull/36065#issuecomment-247125175","issue_url":"https://api.github.com/repos/saltstack/salt/issues/36065","id":247125175,"user":{"login":"rallytime","id":6547906,"avatar_url":"https://avatars.githubusercontent.com/u/6547906?v=3","gravatar_id":"","url":"https://api.github.com/users/rallytime","html_url":"https://github.com/rallytime","followers_url":"https://api.github.com/users/rallytime/followers","following_url":"https://api.github.com/users/rallytime/following{/other_user}","gists_url":"https://api.github.com/users/rallytime/gists{/gist_id}","starred_url":"https://api.github.com/users/rallytime/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rallytime/subscriptions","organizations_url":"https://api.github.com/users/rallytime/orgs","repos_url":"https://api.github.com/users/rallytime/repos","events_url":"https://api.github.com/users/rallytime/events{/privacy}","received_events_url":"https://api.github.com/users/rallytime/received_events","type":"User","site_admin":false},"created_at":"2016-09-14T19:22:33Z","updated_at":"2016-09-14T19:22:33Z","body":"@plastikos Unfortunately, there are merge conflicts with this PR. Can you rebase it? Hopefully we can get this in really quickly once it is rebased so you don't have to keep re-doing this change.\r\n\r\n@thatch45 and @cachedout let's all keep an eye on this so we can get it into develop when it's ready."}}
{"actor":{"display_login":"rallytime"}}