{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3737","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3737/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3737/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3737/events","html_url":"https://github.com/ManageIQ/manageiq/pull/3737","id":99304569,"number":3737,"title":"Migrate inventory collection from REXML to Nokogiri","user":{"login":"djberg96","id":78529,"avatar_url":"https://avatars.githubusercontent.com/u/78529?v=3","gravatar_id":"","url":"https://api.github.com/users/djberg96","html_url":"https://github.com/djberg96","followers_url":"https://api.github.com/users/djberg96/followers","following_url":"https://api.github.com/users/djberg96/following{/other_user}","gists_url":"https://api.github.com/users/djberg96/gists{/gist_id}","starred_url":"https://api.github.com/users/djberg96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djberg96/subscriptions","organizations_url":"https://api.github.com/users/djberg96/orgs","repos_url":"https://api.github.com/users/djberg96/repos","events_url":"https://api.github.com/users/djberg96/events{/privacy}","received_events_url":"https://api.github.com/users/djberg96/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/providers/scvmm","name":"providers/scvmm","color":"5319e7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-05T21:39:02Z","updated_at":"2015-08-06T15:59:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/3737","html_url":"https://github.com/ManageIQ/manageiq/pull/3737","diff_url":"https://github.com/ManageIQ/manageiq/pull/3737.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/3737.patch"},"body":"Alright, let's try again, with some of those commits squished down a bit...\r\n\r\nAs per the discussion at https://github.com/ManageIQ/manageiq/issues/241\r\n\r\nThis switches the SCVMM inventory collection from REXML to Nokogiri. The changes were largely to account for some remaining differences between those two libraries. There were also some minor formatting tweaks.\r\n\r\nThe benefits include a major reduction in memory usage and a minor speed boost."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/128418282","html_url":"https://github.com/ManageIQ/manageiq/pull/3737#issuecomment-128418282","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3737","id":128418282,"user":{"login":"djberg96","id":78529,"avatar_url":"https://avatars.githubusercontent.com/u/78529?v=3","gravatar_id":"","url":"https://api.github.com/users/djberg96","html_url":"https://github.com/djberg96","followers_url":"https://api.github.com/users/djberg96/followers","following_url":"https://api.github.com/users/djberg96/following{/other_user}","gists_url":"https://api.github.com/users/djberg96/gists{/gist_id}","starred_url":"https://api.github.com/users/djberg96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djberg96/subscriptions","organizations_url":"https://api.github.com/users/djberg96/orgs","repos_url":"https://api.github.com/users/djberg96/repos","events_url":"https://api.github.com/users/djberg96/events{/privacy}","received_events_url":"https://api.github.com/users/djberg96/received_events","type":"User","site_admin":false},"created_at":"2015-08-06T15:59:13Z","updated_at":"2015-08-06T15:59:13Z","body":"@Fryguy There are already tests. See gems/pending/test/xml/tc_nokogiri.rb, which I had to modify slightly."}}