{"action":"created","issue":{"url":"https://api.github.com/repos/contiv/netplugin/issues/265","repository_url":"https://api.github.com/repos/contiv/netplugin","labels_url":"https://api.github.com/repos/contiv/netplugin/issues/265/labels{/name}","comments_url":"https://api.github.com/repos/contiv/netplugin/issues/265/comments","events_url":"https://api.github.com/repos/contiv/netplugin/issues/265/events","html_url":"https://github.com/contiv/netplugin/pull/265","id":130934302,"number":265,"title":"netctl cli to take in router ip ","user":{"login":"abhinandanpb","id":784144,"avatar_url":"https://avatars.githubusercontent.com/u/784144?v=3","gravatar_id":"","url":"https://api.github.com/users/abhinandanpb","html_url":"https://github.com/abhinandanpb","followers_url":"https://api.github.com/users/abhinandanpb/followers","following_url":"https://api.github.com/users/abhinandanpb/following{/other_user}","gists_url":"https://api.github.com/users/abhinandanpb/gists{/gist_id}","starred_url":"https://api.github.com/users/abhinandanpb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abhinandanpb/subscriptions","organizations_url":"https://api.github.com/users/abhinandanpb/orgs","repos_url":"https://api.github.com/users/abhinandanpb/repos","events_url":"https://api.github.com/users/abhinandanpb/events{/privacy}","received_events_url":"https://api.github.com/users/abhinandanpb/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-02-03T07:05:17Z","updated_at":"2016-02-06T08:04:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/contiv/netplugin/pulls/265","html_url":"https://github.com/contiv/netplugin/pull/265","diff_url":"https://github.com/contiv/netplugin/pull/265.diff","patch_url":"https://github.com/contiv/netplugin/pull/265.patch"},"body":"The changes contain:\r\n1) Changes to netctl cli to take in router ip and as as additional parameters\r\n2) This would mean there is only 1 bgp config cli to be used to configure L3 mode.\r\n3) decoupled bgp server start from netplugin start and tied it to the cli.\r\n4) This wil ensure bgp config can be changed dynamically even after containers come up . \r\n\r\nnew cli  netctl bgp create <hostname> -router-ip=<> -as=<> -neighbor-ip=<> neighbor-as=<>\r\n \r\n"},"comment":{"url":"https://api.github.com/repos/contiv/netplugin/issues/comments/180717042","html_url":"https://github.com/contiv/netplugin/pull/265#issuecomment-180717042","issue_url":"https://api.github.com/repos/contiv/netplugin/issues/265","id":180717042,"user":{"login":"jainvipin","id":9005681,"avatar_url":"https://avatars.githubusercontent.com/u/9005681?v=3","gravatar_id":"","url":"https://api.github.com/users/jainvipin","html_url":"https://github.com/jainvipin","followers_url":"https://api.github.com/users/jainvipin/followers","following_url":"https://api.github.com/users/jainvipin/following{/other_user}","gists_url":"https://api.github.com/users/jainvipin/gists{/gist_id}","starred_url":"https://api.github.com/users/jainvipin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jainvipin/subscriptions","organizations_url":"https://api.github.com/users/jainvipin/orgs","repos_url":"https://api.github.com/users/jainvipin/repos","events_url":"https://api.github.com/users/jainvipin/events{/privacy}","received_events_url":"https://api.github.com/users/jainvipin/received_events","type":"User","site_admin":false},"created_at":"2016-02-06T08:04:14Z","updated_at":"2016-02-06T08:04:14Z","body":"@abhinandanpb - good set of changes; taking bgp configuration from intent is the right direction; also like the consistency for various CLIs (rm/ls/create/etc.)\r\nplease identify the changes needed for this PR and work on the remaining ones independently."}}