)]}'
{"attributes/default.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"a658ec8e1f52823f9101ae58a1e159d034135e6c","unresolved":false,"context_lines":[{"line_number":307,"context_line":"# the agent will enforce that only a single external networks exists and"},{"line_number":308,"context_line":"# use that external network id"},{"line_number":309,"context_line":"default[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_id\u0027] \u003d nil"},{"line_number":310,"context_line":"# If this name is specified, the ID will be looked up (on every run)."},{"line_number":311,"context_line":"default[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_name\u0027] \u003d nil"},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"# Indicates that this L3 agent should also handle routers that do not have"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"1ae5cdf2_b7d3f336","line":310,"updated":"2014-06-18 14:57:47.000000000","message":"Since your doing a node.set on the network_id, could the check be changed such that the node network_id is used?    To force every run, would need to just set network_id to nil.","commit_id":"885593577db22ae6281b9b36d9e1f270aa28ddbe"},{"author":{"_account_id":10110,"name":"Stephan Renatus","email":"stephan.renatus@gmail.com","username":"sr"},"change_message_id":"7dc3bb2d01ef0d2e1ded9c0da6d41a5958ae719c","unresolved":false,"context_lines":[{"line_number":307,"context_line":"# the agent will enforce that only a single external networks exists and"},{"line_number":308,"context_line":"# use that external network id"},{"line_number":309,"context_line":"default[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_id\u0027] \u003d nil"},{"line_number":310,"context_line":"# If this name is specified, the ID will be looked up (on every run)."},{"line_number":311,"context_line":"default[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_name\u0027] \u003d nil"},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"# Indicates that this L3 agent should also handle routers that do not have"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"1ae5cdf2_48776ed4","line":310,"in_reply_to":"1ae5cdf2_b7d3f336","updated":"2014-06-18 15:57:38.000000000","message":"Done","commit_id":"885593577db22ae6281b9b36d9e1f270aa28ddbe"}],"recipes/l3_agent.rb":[{"author":{"_account_id":1032,"name":"Justin Shepherd","email":"jshepher@rackspace.com","username":"jshepher"},"change_message_id":"92d23b67aadeef33e04602d393915f44cbfe77ad","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"include_recipe \u0027openstack-network::common\u0027"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"if !node[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_id\u0027] \u0026\u0026 node[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_name\u0027]"},{"line_number":26,"context_line":"  external_name \u003d node[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_name\u0027]"},{"line_number":27,"context_line":"  admin_user \u003d node[\u0027openstack\u0027][\u0027identity\u0027][\u0027admin_user\u0027]"},{"line_number":28,"context_line":"  admin_tenant \u003d node[\u0027openstack\u0027][\u0027identity\u0027][\u0027admin_tenant_name\u0027]"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"1ae5cdf2_9858cd32","line":25,"updated":"2014-06-19 02:40:43.000000000","message":"i would prefer \n\nnode[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_id\u0027].nil?\n\ninstead of \n\n!node[\u0027openstack\u0027][\u0027network\u0027][\u0027l3\u0027][\u0027gateway_external_network_id\u0027]","commit_id":"2bc6464142ea9e999403f3250eec9f060b69f2de"}]}