)]}'
{"ansible/group_vars/all.yml":[{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"313a3caaadadce24a4bd3085f959f1e4640edd47","unresolved":false,"context_lines":[{"line_number":43,"context_line":"kolla_enable_sanity_cinder: \"{{ kolla_enable_sanity_checks }}\""},{"line_number":44,"context_line":"kolla_enable_sanity_swift: \"{{ kolla_enable_sanity_checks }}\""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"kolla_enable_cadf_notifications: \"no\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"####################"},{"line_number":49,"context_line":"# kolla-kubernetes"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1ac06dbe_f5629615","line":46,"updated":"2016-08-19 14:01:11.000000000","message":"s/kolla_//","commit_id":"10426776f71b3cc26887208012a2c3a013ad766e"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"9719a773acaa5607a0e5f5e1139a65b0fdfb84db","unresolved":false,"context_lines":[{"line_number":42,"context_line":"kolla_enable_sanity_glance: \"{{ kolla_enable_sanity_checks }}\""},{"line_number":43,"context_line":"kolla_enable_sanity_cinder: \"{{ kolla_enable_sanity_checks }}\""},{"line_number":44,"context_line":"kolla_enable_sanity_swift: \"{{ kolla_enable_sanity_checks }}\""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"####################"},{"line_number":47,"context_line":"# kolla-kubernetes"},{"line_number":48,"context_line":"####################"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa7ab95a_12fc983d","line":45,"updated":"2016-08-31 12:41:09.000000000","message":"Please do not touch this.","commit_id":"d39a4856a7be0c417d867e2f874b0cb54c7b7929"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"9719a773acaa5607a0e5f5e1139a65b0fdfb84db","unresolved":false,"context_lines":[{"line_number":203,"context_line":"    password: \"{{ keystone_admin_password }}\""},{"line_number":204,"context_line":"    project_name: \"admin\""},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"enable_cadf_notifications: \"no\""},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"# These roles are required for Kolla to be operation, however a savvy deployer"},{"line_number":209,"context_line":"# could disable some of these required roles and run their own services."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa7ab95a_3290bcec","line":206,"updated":"2016-08-31 12:41:09.000000000","message":"why not put this into the enable_* list?","commit_id":"d39a4856a7be0c417d867e2f874b0cb54c7b7929"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"ea6d9a9e3078a64c1a005346cc9a714b4ee01040","unresolved":false,"context_lines":[{"line_number":215,"context_line":"enable_nova: \"yes\""},{"line_number":216,"context_line":"enable_rabbitmq: \"yes\""},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"# Additional optional OpenStack features and services are specified here"},{"line_number":219,"context_line":"enable_cadf_notifications: \"no\""},{"line_number":220,"context_line":"enable_ceilometer: \"no\""},{"line_number":221,"context_line":"enable_central_logging: \"no\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"fa7ab95a_4d04c1c5","line":218,"range":{"start_line":218,"start_character":32,"end_line":218,"end_character":40},"updated":"2016-08-31 12:45:53.000000000","message":"I added features here because cadf notifications is not an openstack service.","commit_id":"e59ba6ea583b1c95ec0e7c8b7fe757493656c61c"}],"ansible/roles/keystone/templates/keystone.conf.j2":[{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"0005580b895356b2b4631117b9cbebd84cacbec1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"debug \u003d {{ keystone_logging_debug }}"},{"line_number":3,"context_line":"notification_format \u003d cadf"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"# NOTE(elemoine) log_dir alone does not work for Keystone"},{"line_number":6,"context_line":"log_file \u003d /var/log/kolla/keystone/keystone.log"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"bacf61ea_69b81ac9","line":3,"range":{"start_line":2,"start_character":0,"end_line":3,"end_character":26},"updated":"2016-07-29 20:42:22.000000000","message":"Does it have to be \"cadf\" for all cases? I mean some folks might not be interested in this. Would it not better to make it conditional and enable it only when ceilometer is enabled since it allows to parse cadf messages? Just a thought..","commit_id":"7ec057d5f0aa5e12d3c649a442e34c6127020abe"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"dbed7bd07c30ab37e7361578cc25d72b21adcc03","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"debug \u003d {{ keystone_logging_debug }}"},{"line_number":3,"context_line":"notification_format \u003d cadf"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"# NOTE(elemoine) log_dir alone does not work for Keystone"},{"line_number":6,"context_line":"log_file \u003d /var/log/kolla/keystone/keystone.log"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"bacf61ea_077cdb1d","line":3,"updated":"2016-07-30 06:44:30.000000000","message":"yes. if the consumer is ceilometer, please add the condition.","commit_id":"7ec057d5f0aa5e12d3c649a442e34c6127020abe"},{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"1f7567081b06a47694e45b282e73fabc3f0b9b5c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"debug \u003d {{ keystone_logging_debug }}"},{"line_number":3,"context_line":"{% if kolla_enable_enhanced_security %}"},{"line_number":4,"context_line":"notification_format \u003d cadf"},{"line_number":5,"context_line":"{% endif %}"},{"line_number":6,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"1ac06dbe_859339fa","line":3,"updated":"2016-08-19 13:02:47.000000000","message":"This var is missing. Also unsure about the name. Could we just use enable_cadf_notificaions here?","commit_id":"3feeb84616b6f8f55db6481ef78c8bd6e7643fa7"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"155fbd76417162e3a080bd0f7e65f3b13354bd4d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"debug \u003d {{ keystone_logging_debug }}"},{"line_number":3,"context_line":"{% if kolla_enable_enhanced_security %}"},{"line_number":4,"context_line":"notification_format \u003d cadf"},{"line_number":5,"context_line":"{% endif %}"},{"line_number":6,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"1ac06dbe_363c392f","line":3,"in_reply_to":"1ac06dbe_859339fa","updated":"2016-08-19 13:39:11.000000000","message":"It should be enable_cadf_notifications...","commit_id":"3feeb84616b6f8f55db6481ef78c8bd6e7643fa7"}]}