{"action":"created","issue":{"url":"https://api.github.com/repos/sosreport/sos/issues/616","labels_url":"https://api.github.com/repos/sosreport/sos/issues/616/labels{/name}","comments_url":"https://api.github.com/repos/sosreport/sos/issues/616/comments","events_url":"https://api.github.com/repos/sosreport/sos/issues/616/events","html_url":"https://github.com/sosreport/sos/pull/616","id":97912872,"number":616,"title":"[networking] brctl command is run when bridge kernel module is loaded only","user":{"login":"knoha-rh","id":13554034,"avatar_url":"https://avatars.githubusercontent.com/u/13554034?v=3","gravatar_id":"","url":"https://api.github.com/users/knoha-rh","html_url":"https://github.com/knoha-rh","followers_url":"https://api.github.com/users/knoha-rh/followers","following_url":"https://api.github.com/users/knoha-rh/following{/other_user}","gists_url":"https://api.github.com/users/knoha-rh/gists{/gist_id}","starred_url":"https://api.github.com/users/knoha-rh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/knoha-rh/subscriptions","organizations_url":"https://api.github.com/users/knoha-rh/orgs","repos_url":"https://api.github.com/users/knoha-rh/repos","events_url":"https://api.github.com/users/knoha-rh/events{/privacy}","received_events_url":"https://api.github.com/users/knoha-rh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-29T11:04:22Z","updated_at":"2015-07-29T14:11:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/sosreport/sos/pulls/616","html_url":"https://github.com/sosreport/sos/pull/616","diff_url":"https://github.com/sosreport/sos/pull/616.diff","patch_url":"https://github.com/sosreport/sos/pull/616.patch"},"body":"Previously, bridge information were collected even if bridge kernel module were not loaded.\r\nbrctl command requires and loads bridge and related kernel modules to accomplish its purporse.\r\nThis behaviour causes unexpected and unnecessary kernel module loading.\r\n\r\nsosreport should not change the system configuration.\r\n\r\nThis patch will change this behaviour to run brctl command only if bridge module is loaded.\r\n\r\nSigned-off-by: Keigo Noha <knoha@redhat.com>"},"comment":{"url":"https://api.github.com/repos/sosreport/sos/issues/comments/125963715","html_url":"https://github.com/sosreport/sos/pull/616#issuecomment-125963715","issue_url":"https://api.github.com/repos/sosreport/sos/issues/616","id":125963715,"user":{"login":"battlemidget","id":51892,"avatar_url":"https://avatars.githubusercontent.com/u/51892?v=3","gravatar_id":"","url":"https://api.github.com/users/battlemidget","html_url":"https://github.com/battlemidget","followers_url":"https://api.github.com/users/battlemidget/followers","following_url":"https://api.github.com/users/battlemidget/following{/other_user}","gists_url":"https://api.github.com/users/battlemidget/gists{/gist_id}","starred_url":"https://api.github.com/users/battlemidget/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/battlemidget/subscriptions","organizations_url":"https://api.github.com/users/battlemidget/orgs","repos_url":"https://api.github.com/users/battlemidget/repos","events_url":"https://api.github.com/users/battlemidget/events{/privacy}","received_events_url":"https://api.github.com/users/battlemidget/received_events","type":"User","site_admin":false},"created_at":"2015-07-29T14:11:55Z","updated_at":"2015-07-29T14:11:55Z","body":"Not sure if we want that second commit message in there, maybe do a force push to combine those commits? Other than that LGTM"}}