)]}'
{"recipes/server-apache.rb":[{"author":{"_account_id":23300,"name":"Brian Fletcher","email":"brian.jf.fletcher@gmail.com","username":"brian.fletcher"},"change_message_id":"a5cc41050141ef9c78b1c78eada3dd47eef7ae12","unresolved":false,"context_lines":[{"line_number":348,"context_line":"execute \u0027Keystone apache restart\u0027 do"},{"line_number":349,"context_line":"  command \u0027uname\u0027"},{"line_number":350,"context_line":"  action :nothing"},{"line_number":351,"context_line":"  subscribes("},{"line_number":352,"context_line":"    :run,"},{"line_number":353,"context_line":"    \"template[#{node[\u0027apache\u0027][\u0027dir\u0027]}/sites-available/keystone-main.conf]\","},{"line_number":354,"context_line":"    :immediately"},{"line_number":355,"context_line":"  )"},{"line_number":356,"context_line":"  subscribes("},{"line_number":357,"context_line":"    :run,"},{"line_number":358,"context_line":"    \"template[#{node[\u0027apache\u0027][\u0027dir\u0027]}/sites-available/keystone-admin.conf]\","},{"line_number":359,"context_line":"    :immediately"},{"line_number":360,"context_line":"  )"},{"line_number":361,"context_line":"  notifies :restart, \u0027service[apache2]\u0027, :immediately"},{"line_number":362,"context_line":"  notifies :run, \u0027execute[Keystone: sleep]\u0027, :immediately"},{"line_number":363,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"9a89bdaa_92af3429","line":360,"range":{"start_line":351,"start_character":0,"end_line":360,"end_character":3},"updated":"2016-09-07 17:14:48.000000000","message":"Looks good. The site is enabled using the a2ensite script after this template is laid down. So this can mean that the service is restarted before the config is enabled.\n\nyou can use:\n\n    subscribes :restart, \"execute[a2ensite keystone-admin.conf]\", :immediately\n    subscribes :restart, \"execute[a2ensite keystone-main.conf]\", :immediately","commit_id":"03e78e75225c5e28e35f6eff4c8527df2d09b7d7"},{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"3e1793e6de5ab7f46976253c76a6c58e1c2c4d56","unresolved":false,"context_lines":[{"line_number":357,"context_line":"    :run,"},{"line_number":358,"context_line":"    \"template[#{node[\u0027apache\u0027][\u0027dir\u0027]}/sites-available/keystone-admin.conf]\","},{"line_number":359,"context_line":"    :immediately"},{"line_number":360,"context_line":"  )"},{"line_number":361,"context_line":"  notifies :restart, \u0027service[apache2]\u0027, :immediately"},{"line_number":362,"context_line":"  notifies :run, \u0027execute[Keystone: sleep]\u0027, :immediately"},{"line_number":363,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"9a89bdaa_90180720","line":360,"in_reply_to":"9a89bdaa_92af3429","updated":"2016-09-07 19:42:06.000000000","message":"Looks like this is needed as I\u0027m getting the 403 error again.","commit_id":"03e78e75225c5e28e35f6eff4c8527df2d09b7d7"},{"author":{"_account_id":11915,"name":"Jan Klare","email":"klare@osism.tech","username":"jklare"},"change_message_id":"f178073882a0c5ecd2a5cc644b24c34571a5783c","unresolved":false,"context_lines":[{"line_number":357,"context_line":"    :run,"},{"line_number":358,"context_line":"    \"template[#{node[\u0027apache\u0027][\u0027dir\u0027]}/sites-available/keystone-admin.conf]\","},{"line_number":359,"context_line":"    :immediately"},{"line_number":360,"context_line":"  )"},{"line_number":361,"context_line":"  notifies :restart, \u0027service[apache2]\u0027, :immediately"},{"line_number":362,"context_line":"  notifies :run, \u0027execute[Keystone: sleep]\u0027, :immediately"},{"line_number":363,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"9a89bdaa_22151e9b","line":360,"in_reply_to":"9a89bdaa_92af3429","updated":"2016-09-08 07:54:41.000000000","message":"Yeah that part is a bit ugly, but i am not sure if \"a2ensite\" will run with only a change in the config, since its guards just check if the symlink exists. Maybe we should stop patching here and start patching apache2 ?","commit_id":"03e78e75225c5e28e35f6eff4c8527df2d09b7d7"}]}