)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"0af594b151f90ed7109ebb0a4da7640a126dfe06","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Removed deprecated option external_network_bridge"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"According to description in bug #1491668 in Neutron it\u0027s recommeded"},{"line_number":10,"context_line":"to revmove external_network_bridge option from configuration files"},{"line_number":11,"context_line":"for L3 agent."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8361efb713db39004d4408f438c546bf3f701912"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa69d971_469b03ca","line":10,"updated":"2015-12-30 21:56:34.000000000","message":"remove","commit_id":"12715f6ce25599a102210a637d1062b0144192ad"},{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"0d29456dff15fcba6b8c21342bdb7e1c98e504f0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Removed deprecated option external_network_bridge"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"According to description in bug #1491668 in Neutron it\u0027s recommeded"},{"line_number":10,"context_line":"to revmove external_network_bridge option from configuration files"},{"line_number":11,"context_line":"for L3 agent."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8361efb713db39004d4408f438c546bf3f701912"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa69d971_212d8046","line":10,"in_reply_to":"fa69d971_469b03ca","updated":"2016-01-01 06:31:06.000000000","message":"Done","commit_id":"12715f6ce25599a102210a637d1062b0144192ad"}],"manifests/agents/l3.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"f32c0568c958f497d591a4ab8f37a1cfe4782a7d","unresolved":false,"context_lines":[{"line_number":143,"context_line":"  }"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  if ! is_service_default ($external_network_bridge) {"},{"line_number":146,"context_line":"    notice(\u0027parameter external_network_bridge is deprecated\u0027)"},{"line_number":147,"context_line":"  }"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"  if $ha_enabled {"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_34e5cb08","line":146,"updated":"2015-12-31 12:29:07.000000000","message":"Deprecation is usually done by warning, not notice.","commit_id":"12715f6ce25599a102210a637d1062b0144192ad"},{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"0d29456dff15fcba6b8c21342bdb7e1c98e504f0","unresolved":false,"context_lines":[{"line_number":143,"context_line":"  }"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  if ! is_service_default ($external_network_bridge) {"},{"line_number":146,"context_line":"    notice(\u0027parameter external_network_bridge is deprecated\u0027)"},{"line_number":147,"context_line":"  }"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"  if $ha_enabled {"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_6104d8bf","line":146,"in_reply_to":"fa69d971_34e5cb08","updated":"2016-01-01 06:31:06.000000000","message":"Done","commit_id":"12715f6ce25599a102210a637d1062b0144192ad"},{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"5ad8aae1dc0cc1dcad67c7992471cc6b4acc573d","unresolved":false,"context_lines":[{"line_number":139,"context_line":"  Neutron_l3_agent_config\u003c||\u003e ~\u003e Service[\u0027neutron-l3\u0027]"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"  if $allow_automatic_l3agent_failover {"},{"line_number":142,"context_line":"    warning(\u0027parameter allow_automatic_l3agent_failover is deprecated, use parameter in neutron::server instead\u0027)"},{"line_number":143,"context_line":"  }"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  if ! is_service_default ($external_network_bridge) {"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"fa69d971_01e15cfe","line":142,"range":{"start_line":142,"start_character":4,"end_line":142,"end_character":11},"updated":"2016-01-01 06:40:38.000000000","message":"Although this should not be a part of this commit logically, but according to previous Emilien\u0027s comment about notice\u003c-\u003ewarning for deprecated options this change would not hurt :-)","commit_id":"f06cdbfa0f3bcf2bcfda67e7b58f5939496a75fe"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"1bbc07beb631eb2159081a1e9a37de4db40e6567","unresolved":false,"context_lines":[{"line_number":139,"context_line":"  Neutron_l3_agent_config\u003c||\u003e ~\u003e Service[\u0027neutron-l3\u0027]"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"  if $allow_automatic_l3agent_failover {"},{"line_number":142,"context_line":"    warning(\u0027parameter allow_automatic_l3agent_failover is deprecated, use parameter in neutron::server instead\u0027)"},{"line_number":143,"context_line":"  }"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"  if ! is_service_default ($external_network_bridge) {"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"fa69d971_5ac5ef8b","line":142,"range":{"start_line":142,"start_character":4,"end_line":142,"end_character":11},"in_reply_to":"fa69d971_01e15cfe","updated":"2016-01-04 00:27:53.000000000","message":"thanks, the warning is really more helpful for our operators.","commit_id":"f06cdbfa0f3bcf2bcfda67e7b58f5939496a75fe"}],"manifests/agents/vpnaas.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"8dde89db721a0a9435a9c02c9f2c0fbd5b548f97","unresolved":false,"context_lines":[{"line_number":83,"context_line":"  }"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"  if ! is_service_default ($external_network_bridge) {"},{"line_number":86,"context_line":"    notice(\u0027parameter external_network_bridge is deprecated\u0027)"},{"line_number":87,"context_line":"  }"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"  neutron_vpnaas_agent_config {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_c11ca415","line":86,"range":{"start_line":86,"start_character":4,"end_line":86,"end_character":10},"updated":"2016-01-01 06:34:41.000000000","message":"warning","commit_id":"3c4abc7351d819a881c9ff7e2203b0497da2a072"},{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"11e0c8513bda5d3ad32b974f986db68c0475a0b3","unresolved":false,"context_lines":[{"line_number":83,"context_line":"  }"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"  if ! is_service_default ($external_network_bridge) {"},{"line_number":86,"context_line":"    notice(\u0027parameter external_network_bridge is deprecated\u0027)"},{"line_number":87,"context_line":"  }"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"  neutron_vpnaas_agent_config {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_81ed4cf1","line":86,"range":{"start_line":86,"start_character":4,"end_line":86,"end_character":10},"in_reply_to":"fa69d971_c11ca415","updated":"2016-01-01 06:43:51.000000000","message":"Done","commit_id":"3c4abc7351d819a881c9ff7e2203b0497da2a072"}]}