{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/1321","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/1321/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/1321/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/1321/events","html_url":"https://github.com/ManageIQ/manageiq/pull/1321","id":53706987,"number":1321,"title":"pass ems to save_ems_inventory in microsoft","user":{"login":"kbrock","id":1930,"avatar_url":"https://avatars.githubusercontent.com/u/1930?v=3","gravatar_id":"","url":"https://api.github.com/users/kbrock","html_url":"https://github.com/kbrock","followers_url":"https://api.github.com/users/kbrock/followers","following_url":"https://api.github.com/users/kbrock/following{/other_user}","gists_url":"https://api.github.com/users/kbrock/gists{/gist_id}","starred_url":"https://api.github.com/users/kbrock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kbrock/subscriptions","organizations_url":"https://api.github.com/users/kbrock/orgs","repos_url":"https://api.github.com/users/kbrock/repos","events_url":"https://api.github.com/users/kbrock/events{/privacy}","received_events_url":"https://api.github.com/users/kbrock/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-08T02:12:36Z","updated_at":"2015-01-08T14:45:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/1321","html_url":"https://github.com/ManageIQ/manageiq/pull/1321","diff_url":"https://github.com/ManageIQ/manageiq/pull/1321.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/1321.patch"},"body":"save_ems_inventory's first parameter is stated to be an ems.\r\n\r\nit was passing in a host, changed to pass in an ems.\r\n\r\nDid notice the test was refreshing a host that had no ems. Is this a valid use case? thnx\r\n\r\n/cc @Fryguy does this look right to you?\r\n\r\nrelated to #1305"},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/69188181","html_url":"https://github.com/ManageIQ/manageiq/pull/1321#issuecomment-69188181","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/1321","id":69188181,"user":{"login":"kbrock","id":1930,"avatar_url":"https://avatars.githubusercontent.com/u/1930?v=3","gravatar_id":"","url":"https://api.github.com/users/kbrock","html_url":"https://github.com/kbrock","followers_url":"https://api.github.com/users/kbrock/followers","following_url":"https://api.github.com/users/kbrock/following{/other_user}","gists_url":"https://api.github.com/users/kbrock/gists{/gist_id}","starred_url":"https://api.github.com/users/kbrock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kbrock/subscriptions","organizations_url":"https://api.github.com/users/kbrock/orgs","repos_url":"https://api.github.com/users/kbrock/repos","events_url":"https://api.github.com/users/kbrock/events{/privacy}","received_events_url":"https://api.github.com/users/kbrock/received_events","type":"User","site_admin":false},"created_at":"2015-01-08T14:45:43Z","updated_at":"2015-01-08T14:45:43Z","body":"Also curious if we want to remove `Host` from [save_inventory.rb].\r\n\r\n\r\n/cc @blomquisg @jrafanie in case either of you have an idea why we would not want to pass in a host into the field labeled \"ems\". Assuming that in order to refresh a Host, it has to have a non `nil` `ext_management_system`.\r\n\r\n\r\n[save_inventory.rb]: https://github.com/ManageIQ/manageiq/blob/4192bebafd189e9796fa8eb96172812bd28f2162/vmdb/app/models/ems_refresh/save_inventory.rb#L6"}}