)]}'
{"neutron/agent/linux/ip_lib.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"66548e06f7f2624109df64f412319ea88c128094","unresolved":false,"context_lines":[{"line_number":978,"context_line":"                            \u0027ns\u0027: ns_name})"},{"line_number":979,"context_line":""},{"line_number":980,"context_line":""},{"line_number":981,"context_line":"def sysctl(cmd, namespace\u003dNone):"},{"line_number":982,"context_line":"    \"\"\"Run sysctl command \u0027cmd\u0027"},{"line_number":983,"context_line":""},{"line_number":984,"context_line":"    @param cmd: a list containing the sysctl command to run"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_16ac4433","line":981,"range":{"start_line":981,"start_character":0,"end_line":981,"end_character":10},"updated":"2016-11-09 11:05:45.000000000","message":"I\u0027m just wondering whether this is a correct module to place this function. It doesn\u0027t really have anything in common with \u0027ip\u0027 commands.","commit_id":"219399b02d1918d544d6e2666e97b891c1163abe"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e37ae2fe27932d4286ee9f5b11ad44b0faa97ee1","unresolved":false,"context_lines":[{"line_number":978,"context_line":"                            \u0027ns\u0027: ns_name})"},{"line_number":979,"context_line":""},{"line_number":980,"context_line":""},{"line_number":981,"context_line":"def sysctl(cmd, namespace\u003dNone):"},{"line_number":982,"context_line":"    \"\"\"Run sysctl command \u0027cmd\u0027"},{"line_number":983,"context_line":""},{"line_number":984,"context_line":"    @param cmd: a list containing the sysctl command to run"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_ca73f8e5","line":981,"range":{"start_line":981,"start_character":0,"end_line":981,"end_character":10},"in_reply_to":"9a629dbe_16ac4433","updated":"2016-11-09 14:21:17.000000000","message":"Originally I just left it here since it was being relocated, but since it requires the IPWrapper class and typically takes a namespace (which requires /sbin/ip) it still seems correct.  Would we not create an import loop if we moved it to utils.py since L322 is calling it, but it needs IPWrapper?","commit_id":"219399b02d1918d544d6e2666e97b891c1163abe"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"97713991a54c94904b616d7953a21c58ebc4d733","unresolved":false,"context_lines":[{"line_number":978,"context_line":"                            \u0027ns\u0027: ns_name})"},{"line_number":979,"context_line":""},{"line_number":980,"context_line":""},{"line_number":981,"context_line":"def sysctl(cmd, namespace\u003dNone):"},{"line_number":982,"context_line":"    \"\"\"Run sysctl command \u0027cmd\u0027"},{"line_number":983,"context_line":""},{"line_number":984,"context_line":"    @param cmd: a list containing the sysctl command to run"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_c5912b54","line":981,"range":{"start_line":981,"start_character":0,"end_line":981,"end_character":10},"in_reply_to":"9a629dbe_ca73f8e5","updated":"2016-11-09 14:33:08.000000000","message":"Yeah, I hit that loop today too.","commit_id":"219399b02d1918d544d6e2666e97b891c1163abe"}]}