)]}'
{"neutron/agent/securitygroups_rpc.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"b966a893817bfcbc45f82ad96e16ea29d79c6eff","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    cfg.StrOpt("},{"line_number":30,"context_line":"        \u0027firewall_driver\u0027,"},{"line_number":31,"context_line":"        default\u003d\u0027neutron.agent.firewall.NoopFirewallDriver\u0027,"},{"line_number":32,"context_line":"        help\u003d\u0027Driver for Firewall\u0027)"},{"line_number":33,"context_line":"]"},{"line_number":34,"context_line":"cfg.CONF.register_opts(security_group_opts, \u0027SECURITYGROUP\u0027)"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5HY%3D","line":32,"updated":"2013-07-19 17:27:10.000000000","message":"this line does not look to be i18n-ized like the others.","commit_id":"62070f9f1f61964828a932d7fd16ec48e680fcc1"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"change_message_id":"e3525bd85251e2e8c153272dc139fbfb347cb0ca","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    cfg.StrOpt("},{"line_number":30,"context_line":"        \u0027firewall_driver\u0027,"},{"line_number":31,"context_line":"        default\u003d\u0027neutron.agent.firewall.NoopFirewallDriver\u0027,"},{"line_number":32,"context_line":"        help\u003d_(\u0027Driver for Firewall\u0027))"},{"line_number":33,"context_line":"]"},{"line_number":34,"context_line":"cfg.CONF.register_opts(security_group_opts, \u0027SECURITYGROUP\u0027)"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F35U%3D","line":32,"updated":"2013-07-20 03:11:49.000000000","message":"This is probably a nit, but keeping in mind that there would a firewall driver also as a part of the FWaaS feature, can we be more specific here and change this to:\n\nDriver for Security Groups firewall.","commit_id":"6bd870bdada54c6ff851efef7da5a80c22402cf2"},{"author":{"_account_id":6593,"name":"Dirk Mueller","email":"dirk@dmllr.de","username":"dmllr"},"change_message_id":"6256d2bcd92c9397cfa53bf8adbbcb0b06bcce87","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    cfg.StrOpt("},{"line_number":30,"context_line":"        \u0027firewall_driver\u0027,"},{"line_number":31,"context_line":"        default\u003d\u0027neutron.agent.firewall.NoopFirewallDriver\u0027,"},{"line_number":32,"context_line":"        help\u003d_(\u0027Driver for Firewall\u0027))"},{"line_number":33,"context_line":"]"},{"line_number":34,"context_line":"cfg.CONF.register_opts(security_group_opts, \u0027SECURITYGROUP\u0027)"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F3x8%3D","line":32,"in_reply_to":"AAAAQn%2F%2F35U%3D","updated":"2013-07-20 06:34:49.000000000","message":"Done","commit_id":"6bd870bdada54c6ff851efef7da5a80c22402cf2"}],"neutron/common/config.py":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"67e78604373ce15838cbe81cb0e5c277adc62271","unresolved":false,"context_lines":[{"line_number":88,"context_line":"core_cli_opts \u003d ["},{"line_number":89,"context_line":"    cfg.StrOpt(\u0027state_path\u0027,"},{"line_number":90,"context_line":"               default\u003d\u0027/var/lib/neutron\u0027,"},{"line_number":91,"context_line":"               help\u003d_(\"Where to store Neutron state files.\""},{"line_number":92,"context_line":"                      \"This directory must be writable by the agent.\")),"},{"line_number":93,"context_line":"]"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F4n8%3D","line":91,"updated":"2013-07-19 20:14:05.000000000","message":"missing space after the period?","commit_id":"6bd870bdada54c6ff851efef7da5a80c22402cf2"},{"author":{"_account_id":6593,"name":"Dirk Mueller","email":"dirk@dmllr.de","username":"dmllr"},"change_message_id":"6256d2bcd92c9397cfa53bf8adbbcb0b06bcce87","unresolved":false,"context_lines":[{"line_number":88,"context_line":"core_cli_opts \u003d ["},{"line_number":89,"context_line":"    cfg.StrOpt(\u0027state_path\u0027,"},{"line_number":90,"context_line":"               default\u003d\u0027/var/lib/neutron\u0027,"},{"line_number":91,"context_line":"               help\u003d_(\"Where to store Neutron state files.\""},{"line_number":92,"context_line":"                      \"This directory must be writable by the agent.\")),"},{"line_number":93,"context_line":"]"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F3x4%3D","line":91,"in_reply_to":"AAAAQn%2F%2F4n8%3D","updated":"2013-07-20 06:34:49.000000000","message":"Done","commit_id":"6bd870bdada54c6ff851efef7da5a80c22402cf2"}],"neutron/plugins/bigswitch/plugin.py":[{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"change_message_id":"e3525bd85251e2e8c153272dc139fbfb347cb0ca","unresolved":false,"context_lines":[{"line_number":139,"context_line":"nova_opts.append(cfg.ListOpt(\u0027vif_types\u0027,"},{"line_number":140,"context_line":"                             default\u003dportbindings.VIF_TYPES,"},{"line_number":141,"context_line":"                             help\u003d_(\u0027List of allowed vif_type values\u0027)))"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"cfg.CONF.register_opts(nova_opts, \"NOVA\")"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F35A%3D","line":142,"updated":"2013-07-20 03:11:49.000000000","message":"I don\u0027t believe that this option is being configured in the config file, so help is not required here.","commit_id":"6bd870bdada54c6ff851efef7da5a80c22402cf2"},{"author":{"_account_id":6593,"name":"Dirk Mueller","email":"dirk@dmllr.de","username":"dmllr"},"change_message_id":"6256d2bcd92c9397cfa53bf8adbbcb0b06bcce87","unresolved":false,"context_lines":[{"line_number":139,"context_line":"nova_opts.append(cfg.ListOpt(\u0027vif_types\u0027,"},{"line_number":140,"context_line":"                             default\u003dportbindings.VIF_TYPES,"},{"line_number":141,"context_line":"                             help\u003d_(\u0027List of allowed vif_type values\u0027)))"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"cfg.CONF.register_opts(nova_opts, \"NOVA\")"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F3x0%3D","line":142,"in_reply_to":"AAAAQn%2F%2F35A%3D","updated":"2013-07-20 06:34:49.000000000","message":"but it is configurable, so it eventually should be added to the sample config imho.","commit_id":"6bd870bdada54c6ff851efef7da5a80c22402cf2"}]}