{"action":"created","issue":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2778","labels_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2778/labels{/name}","comments_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2778/comments","events_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2778/events","html_url":"https://github.com/SatelliteQE/robottelo/pull/2778","id":106566524,"number":2778,"title":"[DO NOT MERGE] CLI: Move SSHCommandResult processing to Base class","user":{"login":"abalakh","id":11719030,"avatar_url":"https://avatars.githubusercontent.com/u/11719030?v=3","gravatar_id":"","url":"https://api.github.com/users/abalakh","html_url":"https://github.com/abalakh","followers_url":"https://api.github.com/users/abalakh/followers","following_url":"https://api.github.com/users/abalakh/following{/other_user}","gists_url":"https://api.github.com/users/abalakh/gists{/gist_id}","starred_url":"https://api.github.com/users/abalakh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abalakh/subscriptions","organizations_url":"https://api.github.com/users/abalakh/orgs","repos_url":"https://api.github.com/users/abalakh/repos","events_url":"https://api.github.com/users/abalakh/events{/privacy}","received_events_url":"https://api.github.com/users/abalakh/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/CLI","name":"CLI","color":"009800"},{"url":"https://api.github.com/repos/SatelliteQE/robottelo/labels/Infrastructure","name":"Infrastructure","color":"5319e7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-15T14:20:23Z","updated_at":"2015-09-15T14:28:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/pulls/2778","html_url":"https://github.com/SatelliteQE/robottelo/pull/2778","diff_url":"https://github.com/SatelliteQE/robottelo/pull/2778.diff","patch_url":"https://github.com/SatelliteQE/robottelo/pull/2778.patch"},"body":"Core part of #2776\r\n* Added new exception - `CLICommandExecutionError` which contains `return_code`, `stderr` and `msg` (message)\r\n* Created `_handle_response()` func which checks `return_code` and raises `CLICommandExecutionError` if needed. Returns `stdout`\r\n* Injected `_handle_response()` in `execute()` (that's the easiest place to inject, no existing commands won't be affected and nothing should be rewritten)\r\n* Removed `_remove_task_status()` in favour of `_handle_response()` `ignore_stderr` argument"},"comment":{"url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/comments/140410271","html_url":"https://github.com/SatelliteQE/robottelo/pull/2778#issuecomment-140410271","issue_url":"https://api.github.com/repos/SatelliteQE/robottelo/issues/2778","id":140410271,"user":{"login":"abalakh","id":11719030,"avatar_url":"https://avatars.githubusercontent.com/u/11719030?v=3","gravatar_id":"","url":"https://api.github.com/users/abalakh","html_url":"https://github.com/abalakh","followers_url":"https://api.github.com/users/abalakh/followers","following_url":"https://api.github.com/users/abalakh/following{/other_user}","gists_url":"https://api.github.com/users/abalakh/gists{/gist_id}","starred_url":"https://api.github.com/users/abalakh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abalakh/subscriptions","organizations_url":"https://api.github.com/users/abalakh/orgs","repos_url":"https://api.github.com/users/abalakh/repos","events_url":"https://api.github.com/users/abalakh/events{/privacy}","received_events_url":"https://api.github.com/users/abalakh/received_events","type":"User","site_admin":false},"created_at":"2015-09-15T14:28:10Z","updated_at":"2015-09-15T14:28:10Z","body":"`[DO NOT MERGE]` tag as there're some questions. This PR is for `issue_2776` branch, not `master`, so it can be easily merged.\r\nWould be happy to hear some feedback please, especially @elyezer and @Ichimonji10 :smile: "}}