{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/issues/390","repository_url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending","labels_url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/issues/390/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/issues/390/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/issues/390/events","html_url":"https://github.com/ManageIQ/manageiq-gems-pending/pull/390","id":374146793,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI1OTE1MjEx","number":390,"title":"[WIP] Handle pipes for pg_restore","user":{"login":"NickLaMuro","id":314014,"node_id":"MDQ6VXNlcjMxNDAxNA==","avatar_url":"https://avatars0.githubusercontent.com/u/314014?v=4","gravatar_id":"","url":"https://api.github.com/users/NickLaMuro","html_url":"https://github.com/NickLaMuro","followers_url":"https://api.github.com/users/NickLaMuro/followers","following_url":"https://api.github.com/users/NickLaMuro/following{/other_user}","gists_url":"https://api.github.com/users/NickLaMuro/gists{/gist_id}","starred_url":"https://api.github.com/users/NickLaMuro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NickLaMuro/subscriptions","organizations_url":"https://api.github.com/users/NickLaMuro/orgs","repos_url":"https://api.github.com/users/NickLaMuro/repos","events_url":"https://api.github.com/users/NickLaMuro/events{/privacy}","received_events_url":"https://api.github.com/users/NickLaMuro/received_events","type":"User","site_admin":false},"labels":[{"id":534047521,"node_id":"MDU6TGFiZWw1MzQwNDc1MjE=","url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/labels/wip","name":"wip","color":"eeeeee","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-25T21:49:45Z","updated_at":"2018-10-25T21:56:23Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/pulls/390","html_url":"https://github.com/ManageIQ/manageiq-gems-pending/pull/390","diff_url":"https://github.com/ManageIQ/manageiq-gems-pending/pull/390.diff","patch_url":"https://github.com/ManageIQ/manageiq-gems-pending/pull/390.patch"},"body":"This is built off of #385\r\n\r\nBetter diff can be found here:\r\n\r\nhttps://github.com/NickLaMuro/manageiq-gems-pending/compare/postgres_admin_restore_and_dump_specs...handle-piping-for-postgres-admin-restore-pg-dump\r\n\r\n* * *\r\n\r\n\r\nFor whatever the reason, `pg_dump` doesn't like reading from a fifo when it isn't passed in as `STDIN`.  This assigns the same treatment to it as was done to `pg_dump`/`pg_restore`, which is passes it through `STDIN`.\r\n\r\nSince we don't really have good access to STDIN via `AwesomeSpawn`, we use vanilla `Kernel.spawn` to handle the actually command execution.  We call out to `AwesomeSpawn` for the command builder, as well as emulate the errors that it would throw so there is limited change in how this library behaves to the user."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/issues/comments/433219439","html_url":"https://github.com/ManageIQ/manageiq-gems-pending/pull/390#issuecomment-433219439","issue_url":"https://api.github.com/repos/ManageIQ/manageiq-gems-pending/issues/390","id":433219439,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMzIxOTQzOQ==","user":{"login":"miq-bot","id":7806257,"node_id":"MDQ6VXNlcjc4MDYyNTc=","avatar_url":"https://avatars1.githubusercontent.com/u/7806257?v=4","gravatar_id":"","url":"https://api.github.com/users/miq-bot","html_url":"https://github.com/miq-bot","followers_url":"https://api.github.com/users/miq-bot/followers","following_url":"https://api.github.com/users/miq-bot/following{/other_user}","gists_url":"https://api.github.com/users/miq-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/miq-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miq-bot/subscriptions","organizations_url":"https://api.github.com/users/miq-bot/orgs","repos_url":"https://api.github.com/users/miq-bot/repos","events_url":"https://api.github.com/users/miq-bot/events{/privacy}","received_events_url":"https://api.github.com/users/miq-bot/received_events","type":"User","site_admin":false},"created_at":"2018-10-25T21:56:23Z","updated_at":"2018-10-25T21:56:23Z","author_association":"MEMBER","body":"<rubocop />Checked commits https://github.com/NickLaMuro/manageiq-gems-pending/compare/653beddbf931173d7c1e70bb1f94527c4a3fe41f~...ac3b9ca7c026b04e80a115536c8425691a2fa8bb with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0\n8 files checked, 1 offense detected\n\n**spec/support/mixins/pg_environment_updater.rb**\n- [ ] :exclamation: - [Line 62](https://github.com/NickLaMuro/manageiq-gems-pending/blob/ac3b9ca7c026b04e80a115536c8425691a2fa8bb/spec/support/mixins/pg_environment_updater.rb#L62), Col 5 - [Style/SafeNavigation](http://rubydoc.info/gems/rubocop/0.52.1/RuboCop/Cop/Style/SafeNavigation) - Use safe navigation (`&.`) instead of checking if an object exists before calling the method.\n"}}
{"actor":{"display_login":"miq-bot"}}