{"action":"created","issue":{"url":"https://api.github.com/repos/Metaswitch/calico/issues/448","labels_url":"https://api.github.com/repos/Metaswitch/calico/issues/448/labels{/name}","comments_url":"https://api.github.com/repos/Metaswitch/calico/issues/448/comments","events_url":"https://api.github.com/repos/Metaswitch/calico/issues/448/events","html_url":"https://github.com/Metaswitch/calico/pull/448","id":70448551,"number":448,"title":"Initial code review markups for iptables updater.","user":{"login":"plwhite","id":8448749,"avatar_url":"https://avatars.githubusercontent.com/u/8448749?v=3","gravatar_id":"","url":"https://api.github.com/users/plwhite","html_url":"https://github.com/plwhite","followers_url":"https://api.github.com/users/plwhite/followers","following_url":"https://api.github.com/users/plwhite/following{/other_user}","gists_url":"https://api.github.com/users/plwhite/gists{/gist_id}","starred_url":"https://api.github.com/users/plwhite/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plwhite/subscriptions","organizations_url":"https://api.github.com/users/plwhite/orgs","repos_url":"https://api.github.com/users/plwhite/repos","events_url":"https://api.github.com/users/plwhite/events{/privacy}","received_events_url":"https://api.github.com/users/plwhite/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Metaswitch/calico/labels/Felix","name":"Felix","color":"5319e7"}],"state":"closed","locked":false,"assignee":{"login":"plwhite","id":8448749,"avatar_url":"https://avatars.githubusercontent.com/u/8448749?v=3","gravatar_id":"","url":"https://api.github.com/users/plwhite","html_url":"https://github.com/plwhite","followers_url":"https://api.github.com/users/plwhite/followers","following_url":"https://api.github.com/users/plwhite/following{/other_user}","gists_url":"https://api.github.com/users/plwhite/gists{/gist_id}","starred_url":"https://api.github.com/users/plwhite/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plwhite/subscriptions","organizations_url":"https://api.github.com/users/plwhite/orgs","repos_url":"https://api.github.com/users/plwhite/repos","events_url":"https://api.github.com/users/plwhite/events{/privacy}","received_events_url":"https://api.github.com/users/plwhite/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/Metaswitch/calico/milestones/4","html_url":"https://github.com/Metaswitch/calico/milestones/0.18","labels_url":"https://api.github.com/repos/Metaswitch/calico/milestones/4/labels","id":1075614,"number":4,"title":"0.18","description":"","creator":{"login":"Lukasa","id":1382556,"avatar_url":"https://avatars.githubusercontent.com/u/1382556?v=3","gravatar_id":"","url":"https://api.github.com/users/Lukasa","html_url":"https://github.com/Lukasa","followers_url":"https://api.github.com/users/Lukasa/followers","following_url":"https://api.github.com/users/Lukasa/following{/other_user}","gists_url":"https://api.github.com/users/Lukasa/gists{/gist_id}","starred_url":"https://api.github.com/users/Lukasa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Lukasa/subscriptions","organizations_url":"https://api.github.com/users/Lukasa/orgs","repos_url":"https://api.github.com/users/Lukasa/repos","events_url":"https://api.github.com/users/Lukasa/events{/privacy}","received_events_url":"https://api.github.com/users/Lukasa/received_events","type":"User","site_admin":false},"open_issues":13,"closed_issues":5,"state":"open","created_at":"2015-04-20T16:37:46Z","updated_at":"2015-04-29T10:53:48Z","due_on":"2015-05-03T23:00:00Z","closed_at":null},"comments":4,"created_at":"2015-04-23T16:02:33Z","updated_at":"2015-04-29T14:49:11Z","closed_at":"2015-04-29T10:53:48Z","pull_request":{"url":"https://api.github.com/repos/Metaswitch/calico/pulls/448","html_url":"https://github.com/Metaswitch/calico/pull/448","diff_url":"https://github.com/Metaswitch/calico/pull/448.diff","patch_url":"https://github.com/Metaswitch/calico/pull/448.patch"},"body":"I have made some comment changes directly in line - other comments marked with \"PLW\".\r\n\r\nThis is a first pass, having found nothing very exciting but a few cases where the logic was not clear."},"comment":{"url":"https://api.github.com/repos/Metaswitch/calico/issues/comments/97454228","html_url":"https://github.com/Metaswitch/calico/pull/448#issuecomment-97454228","issue_url":"https://api.github.com/repos/Metaswitch/calico/issues/448","id":97454228,"user":{"login":"plwhite","id":8448749,"avatar_url":"https://avatars.githubusercontent.com/u/8448749?v=3","gravatar_id":"","url":"https://api.github.com/users/plwhite","html_url":"https://github.com/plwhite","followers_url":"https://api.github.com/users/plwhite/followers","following_url":"https://api.github.com/users/plwhite/following{/other_user}","gists_url":"https://api.github.com/users/plwhite/gists{/gist_id}","starred_url":"https://api.github.com/users/plwhite/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plwhite/subscriptions","organizations_url":"https://api.github.com/users/plwhite/orgs","repos_url":"https://api.github.com/users/plwhite/repos","events_url":"https://api.github.com/users/plwhite/events{/privacy}","received_events_url":"https://api.github.com/users/plwhite/received_events","type":"User","site_admin":false},"created_at":"2015-04-29T14:49:11Z","updated_at":"2015-04-29T14:49:11Z","body":"@fasaxc I had to revert this fix, as it broke the FVs. There was a typo in member names of the transaction (missing underscores for the updates field), and I decided to hold off to get some other fixes wrapped up today. Can you sanity check that this is all that was wrong and get it checked in?\r\n"}}