)]}'
{"nova/conf/network.py":[{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"a1768b6546d41bf03a152ee710dcad65fb84690b","unresolved":false,"context_lines":[{"line_number":277,"context_line":"                deprecated_for_removal\u003dTrue),"},{"line_number":278,"context_line":"]"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"ALL_DEFAULT_OPTS \u003d ["},{"line_number":281,"context_line":"    linux_net_opts,"},{"line_number":282,"context_line":"    network_opts,"},{"line_number":283,"context_line":"    ldap_dns_opts,"},{"line_number":284,"context_line":"    security_group_opts]"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"def register_opts(conf):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_1c993020","line":284,"range":{"start_line":280,"start_character":19,"end_line":284,"end_character":24},"updated":"2016-04-08 18:57:34.000000000","message":"I think it should be linux_net_opts + network_opts + ldap_dns_opts +  security_group_opts.","commit_id":"590d2a7d021264194dcea607ee81a38e7c596167"},{"author":{"_account_id":18335,"name":"Pushkar Umaranikar","email":"pushkar.umaranikar@intel.com","username":"pushkarumaranikar"},"change_message_id":"749a4696595b9babe48abba33129a62763ac1a67","unresolved":false,"context_lines":[{"line_number":278,"context_line":"]"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"ALL_DEFAULT_OPTS \u003d ["},{"line_number":281,"context_line":"    linux_net_opts,"},{"line_number":282,"context_line":"    network_opts,"},{"line_number":283,"context_line":"    ldap_dns_opts,"},{"line_number":284,"context_line":"    security_group_opts]"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"def register_opts(conf):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a061dce_72a4d507","line":284,"range":{"start_line":281,"start_character":0,"end_line":284,"end_character":24},"updated":"2016-04-08 19:52:06.000000000","message":"Agree with Anusha\u0027s comment on patchset 2. We want to concatenate the lists. This implementation creates nested lists. Hence, I think we need to use list1 + list 2 + list3","commit_id":"f08223e69c544294b2620ce8e640573b759c29a3"},{"author":{"_account_id":18335,"name":"Pushkar Umaranikar","email":"pushkar.umaranikar@intel.com","username":"pushkarumaranikar"},"change_message_id":"0b621c92a183bed8e3f3a57af7224ba645793d92","unresolved":false,"context_lines":[{"line_number":277,"context_line":"                deprecated_for_removal\u003dTrue),"},{"line_number":278,"context_line":"]"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"ALL_DEFAULT_OPTS \u003d linux_net_opts + network_opts + ldap_dns_opts \\"},{"line_number":281,"context_line":"                   + security_group_opts"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"def register_opts(conf):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a061dce_8b6cd409","line":281,"range":{"start_line":280,"start_character":0,"end_line":281,"end_character":40},"updated":"2016-04-08 21:05:28.000000000","message":"Nit : PEP8 recommendation of wrapping multiple lines is using parenthesis. https://www.python.org/dev/peps/pep-0008/#maximum-line-length","commit_id":"0c2b3b387550ee990d0f83e0aff93f58a9955fa3"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"928b73a68274c7b362f1b9769b2bf47840e9ce6f","unresolved":false,"context_lines":[{"line_number":277,"context_line":"                deprecated_for_removal\u003dTrue),"},{"line_number":278,"context_line":"]"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"ALL_DEFAULT_OPTS \u003d linux_net_opts + network_opts + ldap_dns_opts \\"},{"line_number":281,"context_line":"                   + security_group_opts"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"def register_opts(conf):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a1b2928_b795bb41","line":281,"range":{"start_line":280,"start_character":0,"end_line":281,"end_character":40},"in_reply_to":"9a061dce_8b6cd409","updated":"2016-04-11 16:01:00.000000000","message":"Done","commit_id":"0c2b3b387550ee990d0f83e0aff93f58a9955fa3"}]}
