)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"05cfce627934eb035aebe61797124004a0a19627","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sergey Skripnick \u003csskripnick@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-07-11 11:34:30 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add option to enable profiling"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To enable OSprofiler we should be able to add \"profiler\" notification"},{"line_number":10,"context_line":"topic to ceilometer notification topics."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baada198_5d8d677b","line":7,"updated":"2014-07-11 10:50:14.000000000","message":"It\u0027s already not the \"option to enable profiling\" addition, but exposing ceilo notification queues with the reason for doing it to make folks able to enable profiling/","commit_id":"1379f3fd95e2da30b4086137a852df5a5ac50318"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"7e154aa8dd57f0d583a893c92d273a2bb5fe64e4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sergey Skripnick \u003csskripnick@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-07-11 11:34:30 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add option to enable profiling"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To enable OSprofiler we should be able to add \"profiler\" notification"},{"line_number":10,"context_line":"topic to ceilometer notification topics."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baada198_0fe016f9","line":7,"in_reply_to":"baada198_5d8d677b","updated":"2014-07-11 11:27:22.000000000","message":"Done","commit_id":"1379f3fd95e2da30b4086137a852df5a5ac50318"}],"devstack-vm-gate.sh":[{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"d59949a24bc2271ddf768c9190b95645bb822348","unresolved":false,"context_lines":[{"line_number":221,"context_line":"        echo \"TEMPEST_AUTH_VERSION\u003dv3\" \u003e\u003elocalrc"},{"line_number":222,"context_line":"    fi"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    if [[ \"DEVSTACK_GATE_ENABLE_PROFILING\" -eq \"1\" ]]; then"},{"line_number":225,"context_line":"        # To enable OSprofiler we should add \"profiler\" notification"},{"line_number":226,"context_line":"        # topic to ceilometer notification topics."},{"line_number":227,"context_line":"        echo \"CEILOMETER_NOTIFICATION_TOPICS\u003dnotifications,profiler\" \u003e\u003elocalrc"}],"source_content_type":"text/x-sh","patch_set":2,"id":"baada198_0eb01f42","line":224,"updated":"2014-07-11 08:23:32.000000000","message":"Hm, unfortunately I have one concern about it - it looks like we\u0027re hiding the CEILOMETER_NOTIFICATION_TOPICS parameter behind the DEVSTACK_GATE_ENABLE_PROFILING, so, I think it\u0027ll be better to just expose CEILOMETER_NOTIFICATION_TOPICS and set it to the correct value if you need to enable profiling.","commit_id":"6fce7b51605f844df4a10bec999b1d1357281aae"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"02a9ec32936e58370b6cba824746116c502c3631","unresolved":false,"context_lines":[{"line_number":221,"context_line":"        echo \"TEMPEST_AUTH_VERSION\u003dv3\" \u003e\u003elocalrc"},{"line_number":222,"context_line":"    fi"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    if [[ \"DEVSTACK_GATE_ENABLE_PROFILING\" -eq \"1\" ]]; then"},{"line_number":225,"context_line":"        # To enable OSprofiler we should add \"profiler\" notification"},{"line_number":226,"context_line":"        # topic to ceilometer notification topics."},{"line_number":227,"context_line":"        echo \"CEILOMETER_NOTIFICATION_TOPICS\u003dnotifications,profiler\" \u003e\u003elocalrc"}],"source_content_type":"text/x-sh","patch_set":2,"id":"baada198_0e965f4c","line":224,"in_reply_to":"baada198_0eb01f42","updated":"2014-07-11 08:35:06.000000000","message":"Done","commit_id":"6fce7b51605f844df4a10bec999b1d1357281aae"},{"author":{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},"change_message_id":"9354f2a0804948bc350bc731eeaed9959e96cb9f","unresolved":false,"context_lines":[{"line_number":228,"context_line":"        echo \"ENABLE_HTTPD_MOD_WSGI_SERVICES\u003dFalse\" \u003e\u003e localrc"},{"line_number":229,"context_line":"    fi"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    if [[ \"CEILOMETER_NOTIFICATION_TOPICS\" ]]; then"},{"line_number":232,"context_line":"        # Add specified ceilometer notification topics to localrc"},{"line_number":233,"context_line":"        # Set to notifications,profiler to enable profiling"},{"line_number":234,"context_line":"        echo \"CEILOMETER_NOTIFICATION_TOPICS\u003d$CEILOMETER_NOTIFICATION_TOPICS\" \u003e\u003elocalrc"}],"source_content_type":"text/x-sh","patch_set":5,"id":"baada198_2c01ebb5","line":231,"updated":"2014-07-29 05:59:21.000000000","message":"missing $","commit_id":"416a5008bb93630650d294b10ce925eba11162c7"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"13d6113281849a596d6459e3167a869458443d3f","unresolved":false,"context_lines":[{"line_number":228,"context_line":"        echo \"ENABLE_HTTPD_MOD_WSGI_SERVICES\u003dFalse\" \u003e\u003e localrc"},{"line_number":229,"context_line":"    fi"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    if [[ \"CEILOMETER_NOTIFICATION_TOPICS\" ]]; then"},{"line_number":232,"context_line":"        # Add specified ceilometer notification topics to localrc"},{"line_number":233,"context_line":"        # Set to notifications,profiler to enable profiling"},{"line_number":234,"context_line":"        echo \"CEILOMETER_NOTIFICATION_TOPICS\u003d$CEILOMETER_NOTIFICATION_TOPICS\" \u003e\u003elocalrc"}],"source_content_type":"text/x-sh","patch_set":5,"id":"baada198_1108a92d","line":231,"in_reply_to":"baada198_2c01ebb5","updated":"2014-07-29 15:20:33.000000000","message":"Done","commit_id":"416a5008bb93630650d294b10ce925eba11162c7"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"cd4e40117c5cac90f4e632bc2d9d56b0daee6e36","unresolved":false,"context_lines":[{"line_number":228,"context_line":"        echo \"ENABLE_HTTPD_MOD_WSGI_SERVICES\u003dFalse\" \u003e\u003e localrc"},{"line_number":229,"context_line":"    fi"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    if [[ \"CEILOMETER_NOTIFICATION_TOPICS\" ]]; then"},{"line_number":232,"context_line":"        # Add specified ceilometer notification topics to localrc"},{"line_number":233,"context_line":"        # Set to notifications,profiler to enable profiling"},{"line_number":234,"context_line":"        echo \"CEILOMETER_NOTIFICATION_TOPICS\u003d$CEILOMETER_NOTIFICATION_TOPICS\" \u003e\u003elocalrc"}],"source_content_type":"text/x-sh","patch_set":5,"id":"baada198_decc8365","line":231,"in_reply_to":"baada198_2c01ebb5","updated":"2014-07-29 13:53:32.000000000","message":"yup, $ is needed here.","commit_id":"416a5008bb93630650d294b10ce925eba11162c7"}]}