{"action":"created","issue":{"url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup/issues/20","repository_url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup","labels_url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup/issues/20/labels{/name}","comments_url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup/issues/20/comments","events_url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup/issues/20/events","html_url":"https://github.com/theforeman/foreman_default_hostgroup/issues/20","id":162018958,"number":20,"title":"Plugin doesn't work in Foreman 1.12","user":{"login":"sh41","id":6237786,"avatar_url":"https://avatars.githubusercontent.com/u/6237786?v=3","gravatar_id":"","url":"https://api.github.com/users/sh41","html_url":"https://github.com/sh41","followers_url":"https://api.github.com/users/sh41/followers","following_url":"https://api.github.com/users/sh41/following{/other_user}","gists_url":"https://api.github.com/users/sh41/gists{/gist_id}","starred_url":"https://api.github.com/users/sh41/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sh41/subscriptions","organizations_url":"https://api.github.com/users/sh41/orgs","repos_url":"https://api.github.com/users/sh41/repos","events_url":"https://api.github.com/users/sh41/events{/privacy}","received_events_url":"https://api.github.com/users/sh41/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-06-23T20:45:07Z","updated_at":"2016-06-27T01:57:42Z","closed_at":null,"body":"Sadly it looks like theforeman/foreman#3242 has stopped this plugin from being able to carry out its really useful work. \r\n\r\nAs far as I can tell this plugin currently overrides the `import_host_and_facts` function to add in the functionality that sets a default hostgroup. The PR deprecates that function in foreman (https://github.com/theforeman/foreman/blob/d4ec64418b8db6093375f8d4b690d506705163f7/app/models/host/managed.rb#L466) and changes the code  so that the overridden function is never called. Instead two different functions called `import_host` and `import_facts` are called: https://github.com/theforeman/foreman/blob/d4ec64418b8db6093375f8d4b690d506705163f7/app/controllers/api/v2/hosts_controller.rb#L236\r\n\r\nIt looks like this PR will be included in foreman 1.12 (I've noticed the problem in 1.12rc2). \r\n\r\n"},"comment":{"url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup/issues/comments/228639105","html_url":"https://github.com/theforeman/foreman_default_hostgroup/issues/20#issuecomment-228639105","issue_url":"https://api.github.com/repos/theforeman/foreman_default_hostgroup/issues/20","id":228639105,"user":{"login":"GregSutcliffe","id":827786,"avatar_url":"https://avatars.githubusercontent.com/u/827786?v=3","gravatar_id":"","url":"https://api.github.com/users/GregSutcliffe","html_url":"https://github.com/GregSutcliffe","followers_url":"https://api.github.com/users/GregSutcliffe/followers","following_url":"https://api.github.com/users/GregSutcliffe/following{/other_user}","gists_url":"https://api.github.com/users/GregSutcliffe/gists{/gist_id}","starred_url":"https://api.github.com/users/GregSutcliffe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GregSutcliffe/subscriptions","organizations_url":"https://api.github.com/users/GregSutcliffe/orgs","repos_url":"https://api.github.com/users/GregSutcliffe/repos","events_url":"https://api.github.com/users/GregSutcliffe/events{/privacy}","received_events_url":"https://api.github.com/users/GregSutcliffe/received_events","type":"User","site_admin":false},"created_at":"2016-06-27T01:57:42Z","updated_at":"2016-06-27T01:57:42Z","body":"@sh41 I've just sent a PR with a ton of changes for 1.12 - on the chance that you're running a sandbox somewhere, would you be willing to give it a try and see if it works for you. Tests say it should ;)"}}
{"actor":{"display_login":"GregSutcliffe"}}