)]}'
{"ceilometer/profiler/notifications.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"c09d3e533868d0b4ba349c6e118b3e3df09412e1","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from ceilometer import sample"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# TODO(boris-42): remove after adding keystone audit plugins."},{"line_number":24,"context_line":"cfg.CONF.import_opt(\u0027keystone_control_exchange\u0027,"},{"line_number":25,"context_line":"                    \u0027ceilometer.identity.notifications\u0027)"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_b8cb7949","line":23,"updated":"2014-07-15 15:58:25.000000000","message":"Can you explain the meaning of this TODO?\n\nAs in, why does the exchange have to be removed after adding keystone audit plugins?","commit_id":"52b2779e1bae060d8dcf0edfc9307859d6cf4e93"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"58f298ef1c57e218ea9ab39fb38b7a9b92c7eaed","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from ceilometer import sample"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# TODO(boris-42): remove after adding keystone audit plugins."},{"line_number":24,"context_line":"cfg.CONF.import_opt(\u0027keystone_control_exchange\u0027,"},{"line_number":25,"context_line":"                    \u0027ceilometer.identity.notifications\u0027)"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_7bb9db78","line":23,"in_reply_to":"baada198_b8cb7949","updated":"2014-07-15 16:30:38.000000000","message":"We don\u0027t load \"ceilometer.identity.notifications\" module via stevedore. So ceilometer.identity.notifications is not seted. \n\nWhen I add first notification plugin for keystone \u003d\u003e I will add it as well to stevedore \u003d\u003e it will be loaded \u003d\u003e We won\u0027t need here this code.","commit_id":"52b2779e1bae060d8dcf0edfc9307859d6cf4e93"}]}