)]}'
{"devstack/plugin.sh":[{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"70abc8b65b9a9bfa5dae9f6fd8a77d962978ebe3","unresolved":false,"context_lines":[{"line_number":262,"context_line":"        sed -i \"s/interval:.*/interval: ${CEILOMETER_PIPELINE_INTERVAL}/\" $CEILOMETER_CONF_DIR/pipeline.yaml"},{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^        - notifier://?topic\u003dalarm.all$\u0027; then"},{"line_number":266,"context_line":"            sed -i \u0027/        - notifier:\\/\\/$/a\\          - notifier://?topic\u003dalarm.all\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da20952f_708fe875","line":265,"updated":"2015-09-01 19:34:03.000000000","message":"it seems that here is missed a file to apply grep\nor I\u0027ve missed something","commit_id":"f6b3120c5b1e87790032f91943482bd49b47f4f2"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"4576e75ac6a7b194581d8d896f92d026ace4fd34","unresolved":false,"context_lines":[{"line_number":262,"context_line":"        sed -i \"s/interval:.*/interval: ${CEILOMETER_PIPELINE_INTERVAL}/\" $CEILOMETER_CONF_DIR/pipeline.yaml"},{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^        - notifier://?topic\u003dalarm.all$\u0027; then"},{"line_number":266,"context_line":"            sed -i \u0027/        - notifier:\\/\\/$/a\\          - notifier://?topic\u003dalarm.all\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da20952f_df503861","line":265,"in_reply_to":"da20952f_708fe875","updated":"2015-09-01 23:47:09.000000000","message":"you are right, i will fix it","commit_id":"f6b3120c5b1e87790032f91943482bd49b47f4f2"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"7f4b6dfd4895f0ddbe19ea8a471972e2c4124885","unresolved":false,"context_lines":[{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^        - notifier://?topic\u003dalarm.all$\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml; then"},{"line_number":266,"context_line":"            sed -i \u0027/        - notifier:\\/\\/$/a\\          - notifier://?topic\u003dalarm.all\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"},{"line_number":269,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"da20952f_6d49348e","line":266,"updated":"2015-09-02 08:26:15.000000000","message":"instead of only append to - notifier://, how about we append after publishers: ?\n\nalso, I don\u0027t like the space things here, can we do some backreference in sed?","commit_id":"9f60befdf4aabbc2cdf40ca36a678aeaf7307637"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"327f8f18cf1427379bde014791720a58c6582de2","unresolved":false,"context_lines":[{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^        - notifier://?topic\u003dalarm.all$\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml; then"},{"line_number":266,"context_line":"            sed -i \u0027/        - notifier:\\/\\/$/a\\          - notifier://?topic\u003dalarm.all\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"},{"line_number":269,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"da20952f_43ba0170","line":266,"in_reply_to":"da20952f_6d49348e","updated":"2015-09-02 09:06:59.000000000","message":"sounds nice, will update","commit_id":"9f60befdf4aabbc2cdf40ca36a678aeaf7307637"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"55e56eb70818c7956fb225401f892075e80ee923","unresolved":false,"context_lines":[{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^ *- notifier://?topic\u003dalarm.all$\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml; then"},{"line_number":266,"context_line":"            sed -i \u0027s|^\\( *\\)publishers:$|\u0026\\n\\1  - notifier://?topic\u003dalarm.all|\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"},{"line_number":269,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"da20952f_bd374baf","line":266,"updated":"2015-09-03 05:47:07.000000000","message":"I\u0027m afraid this doesn\u0027t work on my machine. Please see http://paste.openstack.org/show/443662/\n\nWe need to think more, looks like we still need to backreference notifier line :(","commit_id":"c80d216a16840f21cfba64e63f6cb4231729f387"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"226a0c7d4f600b367805f63932feeb359474b2ba","unresolved":false,"context_lines":[{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^ *- notifier://?topic\u003dalarm.all$\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml; then"},{"line_number":266,"context_line":"            sed -i \u0027s|^\\( *\\)publishers:$|\u0026\\n\\1  - notifier://?topic\u003dalarm.all|\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"},{"line_number":269,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"da20952f_06ee1fb4","line":266,"in_reply_to":"da20952f_a3440a1b","updated":"2015-09-07 14:09:12.000000000","message":"I posted new patch set, please check the latest one. you can find bit improvement.","commit_id":"c80d216a16840f21cfba64e63f6cb4231729f387"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"7100c72d071a064829ba30a81ee711e915806119","unresolved":false,"context_lines":[{"line_number":263,"context_line":"    fi"},{"line_number":264,"context_line":"    if [ \"$CEILOMETER_EVENT_ALARM\" \u003d\u003d \"True\" ]; then"},{"line_number":265,"context_line":"        if ! grep -q \u0027^ *- notifier://?topic\u003dalarm.all$\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml; then"},{"line_number":266,"context_line":"            sed -i \u0027s|^\\( *\\)publishers:$|\u0026\\n\\1  - notifier://?topic\u003dalarm.all|\u0027 $CEILOMETER_CONF_DIR/event_pipeline.yaml"},{"line_number":267,"context_line":"        fi"},{"line_number":268,"context_line":"    fi"},{"line_number":269,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"da20952f_a3440a1b","line":266,"in_reply_to":"da20952f_bd374baf","updated":"2015-09-03 06:56:33.000000000","message":"how about this one?\n\n    sed \u0027s|^\\( *\\)\\(  \\)publishers:$|\u0026\\n\\1\\1\\2- notifier://?topic\u003dalarm.all|\u0027 /etc/ceilometer/event_pipeline.yaml\n\nwhat do you mean \u0027backreference\u0027? Isn\u0027t it back reference available  in replace command \u0027s\u0027 in sed?","commit_id":"c80d216a16840f21cfba64e63f6cb4231729f387"}]}