)]}'
{"recipes/db_migration.rb":[{"author":{"_account_id":26008,"name":"Bogdan Katynski","email":"bogdan.katynski@workday.com","username":"bodgix"},"change_message_id":"0fccba5094b680524e98d57f10629ba4178f0835","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# The node[\u0027openstack\u0027][\u0027network\u0027][\u0027plugin_config_file\u0027] attribute is set in the default.rb recipe"},{"line_number":24,"context_line":"bash \u0027migrate network database\u0027 do"},{"line_number":25,"context_line":"  timeout timeout"},{"line_number":26,"context_line":"  migrate_command \u003d \u0027neutron-db-manage --config-file /etc/neutron/neutron.conf\u0027"},{"line_number":27,"context_line":"  code \u003c\u003c-EOF"},{"line_number":28,"context_line":"#{migrate_command} upgrade heads"},{"line_number":29,"context_line":"EOF"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"9f436f4f_092c4f00","line":26,"range":{"start_line":26,"start_character":21,"end_line":26,"end_character":78},"updated":"2017-08-11 11:06:36.000000000","message":"would it make sense to save this as a base command outside of the block so other blocks would only append task-specific options?","commit_id":"25349d61875fcb3231fba20d6336dfc227c17329"},{"author":{"_account_id":19193,"name":"Christoph Albers","email":"c.albers@x-ion.de","username":"calbers"},"change_message_id":"cff8d7d6e932cf6c3d957be70efc9c4a0f9cb533","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# The node[\u0027openstack\u0027][\u0027network\u0027][\u0027plugin_config_file\u0027] attribute is set in the default.rb recipe"},{"line_number":24,"context_line":"bash \u0027migrate network database\u0027 do"},{"line_number":25,"context_line":"  timeout timeout"},{"line_number":26,"context_line":"  migrate_command \u003d \u0027neutron-db-manage --config-file /etc/neutron/neutron.conf\u0027"},{"line_number":27,"context_line":"  code \u003c\u003c-EOF"},{"line_number":28,"context_line":"#{migrate_command} upgrade heads"},{"line_number":29,"context_line":"EOF"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"9f436f4f_8a894dcb","line":26,"range":{"start_line":26,"start_character":21,"end_line":26,"end_character":78},"in_reply_to":"9f436f4f_092c4f00","updated":"2017-08-11 12:39:05.000000000","message":"it is possible to build an library / resource which does this job. IMO this isn\u0027t needed at the moment. Its more like nice to have, cause this would be an complete refactor of that recipe / resource","commit_id":"25349d61875fcb3231fba20d6336dfc227c17329"},{"author":{"_account_id":14790,"name":"Samuel Cassiba","email":"samuel@cassi.ba","username":"scassiba"},"change_message_id":"088b94b8f29e9d39a5bcdcc3eaedce31e7e71281","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# The node[\u0027openstack\u0027][\u0027network\u0027][\u0027plugin_config_file\u0027] attribute is set in the default.rb recipe"},{"line_number":24,"context_line":"bash \u0027migrate network database\u0027 do"},{"line_number":25,"context_line":"  timeout timeout"},{"line_number":26,"context_line":"  migrate_command \u003d \u0027neutron-db-manage --config-file /etc/neutron/neutron.conf\u0027"},{"line_number":27,"context_line":"  code \u003c\u003c-EOF"},{"line_number":28,"context_line":"#{migrate_command} upgrade heads"},{"line_number":29,"context_line":"EOF"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"9f436f4f_5eaa06d2","line":26,"range":{"start_line":26,"start_character":21,"end_line":26,"end_character":78},"in_reply_to":"9f436f4f_8a894dcb","updated":"2017-08-11 12:47:18.000000000","message":"If custom resources are the eventual prescribed path, this whole logic will be lifted out to that. For now, however, we\u0027re focusing on stabilization. Basically, Chef 13 will tell us how we need to approach these types of recipes.","commit_id":"25349d61875fcb3231fba20d6336dfc227c17329"}],"recipes/ml2_linuxbridge.rb":[{"author":{"_account_id":11915,"name":"Jan Klare","email":"klare@osism.tech","username":"jklare"},"change_message_id":"174009755ce7dd8e3b7b01cf4c2d701542672885","unresolved":false,"context_lines":[{"line_number":27,"context_line":"include_recipe \u0027openstack-network\u0027"},{"line_number":28,"context_line":"node.default[\u0027openstack\u0027][\u0027network\u0027][\u0027plugins\u0027][\u0027ml2\u0027][\u0027conf\u0027][\u0027ml2\u0027][\u0027type_drivers\u0027] \u003d \u0027local,flat,vlan,gre,vxlan\u0027"},{"line_number":29,"context_line":"node.default[\u0027openstack\u0027][\u0027network\u0027][\u0027plugins\u0027][\u0027ml2\u0027][\u0027conf\u0027][\u0027ml2\u0027][\u0027mechanism_drivers\u0027] \u003d \u0027linuxbridge,l2population\u0027"},{"line_number":30,"context_line":"node.default[\u0027openstack\u0027][\u0027network\u0027][\u0027plugins\u0027][\u0027ml2\u0027][\u0027conf\u0027][\u0027ml2_type_vxlan\u0027][\u0027vni_ranges\u0027] \u003d \u0027VNI_START:VNI_END\u0027"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"platform_options \u003d node[\u0027openstack\u0027][\u0027network\u0027][\u0027platform\u0027]"},{"line_number":33,"context_line":"platform_options[\u0027neutron_linuxbridge_agent_packages\u0027].each do |pkg|"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"9f436f4f_02db2aed","line":30,"updated":"2017-08-10 13:02:46.000000000","message":"i think these changes should probably be mentioned and explained in the commit message","commit_id":"ca048318fb98a8358bdb5846e55a4243a8073685"}]}