)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7535,"name":"Nikolay Starodubtsev","email":"starodubcevna@gmail.com","username":"Kezar"},"change_message_id":"035e469f6d051250be76baa045a3d2118e24f514","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Initial devstack support implemented"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* Install climate, climate-nova and climate client"},{"line_number":10,"context_line":"* Sets all neded climate configs, even the nova ones in order to use"},{"line_number":11,"context_line":"climate-nova"},{"line_number":12,"context_line":"* Starts/Stops climate-api and climate-manager"},{"line_number":13,"context_line":"* Creates the Keystone V3 service/endpoint if don\u0027t exists."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"AAAAUX%2F%2B6uw%3D","line":10,"updated":"2014-02-18 08:01:07.000000000","message":"typo \"needed\"","commit_id":"ad1106aba620831b778ec54f8c572746d945691c"}],"contrib/devstack/README.rst":[{"author":{"_account_id":10217,"name":"Diego Pamio","email":"diego.pamio@intel.com","username":"dpamio"},"change_message_id":"e338f8e641c6ea32c3bdfc00f76fb208b2a93e86","unresolved":false,"context_lines":[{"line_number":13,"context_line":"To configure devstack to run climate:"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    $ cd ${DEVSTACK_DIR}"},{"line_number":16,"context_line":"    $ echo \"enable_service climate\" \u003e\u003e localrc"}],"source_content_type":"text/x-rst","patch_set":4,"id":"AAAAUX%2F%2Fz88%3D","line":16,"updated":"2014-02-10 17:48:10.000000000","message":"Shouldn\u0027t we also add:\necho \"enable_service climate-a\" \u003e\u003e localrc\nand \necho \"enable_service climate-m\" \u003e\u003e localrc\n?","commit_id":"f30148ece5501c40952fb301fe89338958c07d80"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"0641775a380db39e8704aeb66c744ce56d9eec90","unresolved":false,"context_lines":[{"line_number":13,"context_line":"To configure devstack to run climate:"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    $ cd ${DEVSTACK_DIR}"},{"line_number":16,"context_line":"    $ echo \"enable_service climate\" \u003e\u003e localrc"}],"source_content_type":"text/x-rst","patch_set":4,"id":"AAAAUX%2F%2Fylc%3D","line":16,"in_reply_to":"AAAAUX%2F%2Fz88%3D","updated":"2014-02-10 19:51:58.000000000","message":"Good catch. Done.","commit_id":"f30148ece5501c40952fb301fe89338958c07d80"},{"author":{"_account_id":7535,"name":"Nikolay Starodubtsev","email":"starodubcevna@gmail.com","username":"Kezar"},"change_message_id":"0a5dbaa3c4500ecde4e9fbb426f45fe16b34052b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"To install:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    $ DEVSTACK_DIR\u003d.../path/to/devstack"},{"line_number":10,"context_line":"    $ cp lib/climate ${DEVSTACK_DIR}/lib"},{"line_number":11,"context_line":"    $ cp extras.d/70-climate.sh ${DEVSTACK_DIR}/extras.d"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"To configure devstack to run climate:"}],"source_content_type":"text/x-rst","patch_set":9,"id":"AAAAUX%2F%2B6sQ%3D","line":10,"updated":"2014-02-18 08:13:13.000000000","message":"maybe full path such as climate/contrib/devstack/lib/climate will be better?","commit_id":"ad1106aba620831b778ec54f8c572746d945691c"},{"author":{"_account_id":7535,"name":"Nikolay Starodubtsev","email":"starodubcevna@gmail.com","username":"Kezar"},"change_message_id":"f67bcf75cb27a99e2ab8ffdd65bd6e9fa5afcae3","unresolved":false,"context_lines":[{"line_number":7,"context_line":"To install:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    $ DEVSTACK_DIR\u003d.../path/to/devstack"},{"line_number":10,"context_line":"    $ cp lib/climate ${DEVSTACK_DIR}/lib"},{"line_number":11,"context_line":"    $ cp extras.d/70-climate.sh ${DEVSTACK_DIR}/extras.d"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"To configure devstack to run climate:"}],"source_content_type":"text/x-rst","patch_set":9,"id":"AAAAUX%2F%2B6Mw%3D","line":10,"in_reply_to":"AAAAUX%2F%2B6Nk%3D","updated":"2014-02-18 09:11:46.000000000","message":"Ok, I get it","commit_id":"ad1106aba620831b778ec54f8c572746d945691c"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"f38e5cd8ee23103bdd1ea6b680f032e472b734af","unresolved":false,"context_lines":[{"line_number":7,"context_line":"To install:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    $ DEVSTACK_DIR\u003d.../path/to/devstack"},{"line_number":10,"context_line":"    $ cp lib/climate ${DEVSTACK_DIR}/lib"},{"line_number":11,"context_line":"    $ cp extras.d/70-climate.sh ${DEVSTACK_DIR}/extras.d"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"To configure devstack to run climate:"}],"source_content_type":"text/x-rst","patch_set":9,"id":"AAAAUX%2F%2B6Nk%3D","line":10,"in_reply_to":"AAAAUX%2F%2B6sQ%3D","updated":"2014-02-18 09:09:39.000000000","message":"in that case you also should modify other path with extras.d. I think it\u0027s ok to go to needed dir and then copy all this stuff.","commit_id":"ad1106aba620831b778ec54f8c572746d945691c"},{"author":{"_account_id":7535,"name":"Nikolay Starodubtsev","email":"starodubcevna@gmail.com","username":"Kezar"},"change_message_id":"0a5dbaa3c4500ecde4e9fbb426f45fe16b34052b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    $ DEVSTACK_DIR\u003d.../path/to/devstack"},{"line_number":10,"context_line":"    $ cp lib/climate ${DEVSTACK_DIR}/lib"},{"line_number":11,"context_line":"    $ cp extras.d/70-climate.sh ${DEVSTACK_DIR}/extras.d"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"To configure devstack to run climate:"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"AAAAUX%2F%2B6sM%3D","line":11,"updated":"2014-02-18 08:13:13.000000000","message":"the same","commit_id":"ad1106aba620831b778ec54f8c572746d945691c"}],"contrib/devstack/extras.d/70-climate.sh":[{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"f0ca3fde25de6476e96622fcb9463be25638e653","unresolved":false,"context_lines":[{"line_number":12,"context_line":"        echo_summary \"Starting Climate\""},{"line_number":13,"context_line":"        start_climate"},{"line_number":14,"context_line":"    elif [[ \"$1\" \u003d\u003d \"unstack\" ]]; then"},{"line_number":15,"context_line":"        echo_summary \"Stoping Climate\""},{"line_number":16,"context_line":"        stop_climate"},{"line_number":17,"context_line":"    fi"},{"line_number":18,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAUH%2F9tQg%3D","line":15,"updated":"2014-02-04 19:39:11.000000000","message":"This function isn\u0027t imported here (is defined in stack.sh)\nThis is a piece of unstack.sh output:\n\nextras.d/70-climate.sh: line 15: echo_summary: command not found\n\nThe other scripts in extras.d don\u0027t log anything here.","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"}],"contrib/devstack/lib/climate":[{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"c4516f7062891adea7650ed7ef7a00074583bb52","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# --------"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"CLIMATE_DEBUG\u003d${CLIMATE_DEBUG:-True}"},{"line_number":26,"context_line":"CLIMATE_LOG_EXCHANGE\u003d${SAVANNA_LOG_EXCHANGE:-True}"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# Set up default repos"},{"line_number":29,"context_line":"CLIMATE_REPO\u003d${CLIMATE_REPO:-${GIT_BASE}/stackforge/climate.git}"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9uRA%3D","line":26,"updated":"2014-02-04 17:19:50.000000000","message":"I think this was the result of some inspiration in a Savanna file ;)\nCLIMATE_LOG_EXCHANGE\u003d${CLIMATE_LOG_EXCHANGE:-True}","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"4938071a0e83da8a5f5159bcaa0f466d7d03b0ba","unresolved":false,"context_lines":[{"line_number":34,"context_line":"# Set up default directories"},{"line_number":35,"context_line":"CLIMATE_DIR\u003d$DEST/climate"},{"line_number":36,"context_line":"CLIMATE_CONF_DIR\u003d${CLIMATE_CONF_DIR:-/etc/climate}"},{"line_number":37,"context_line":"CLIMATE_CONF_FILE\u003dclimate.conf"},{"line_number":38,"context_line":"CLIMATECLIENT_DIR\u003d$DEST/python-climateclient"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"CLIMATE_SERVICE_HOST\u003d${CLIMATE_SERVICE_HOST:-$SERVICE_HOST}"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9mu8%3D","line":37,"updated":"2014-02-05 18:07:11.000000000","message":"It should include CLIMATE_CONF_DIR","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"4938071a0e83da8a5f5159bcaa0f466d7d03b0ba","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT os_admin_tenant_name $SERVICE_TENANT_NAME"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":68,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT log-exchange $SAVANNA_LOG_EXCHANGE"},{"line_number":69,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT debug $SAVANNA_DEBUG"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE manager plugins\u003ddummy.vm.plugin"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9mus%3D","line":68,"updated":"2014-02-05 18:07:11.000000000","message":"one more savanna thing","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"c4516f7062891adea7650ed7ef7a00074583bb52","unresolved":false,"context_lines":[{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":68,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT log-exchange $SAVANNA_LOG_EXCHANGE"},{"line_number":69,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT debug $SAVANNA_DEBUG"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE manager plugins\u003ddummy.vm.plugin"},{"line_number":72,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9uPc%3D","line":69,"updated":"2014-02-04 17:19:50.000000000","message":"The same Savanna thing.","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"4081c0e45dfa30379710ea202d341823fd32c8e9","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT log-exchange $SAVANNA_LOG_EXCHANGE"},{"line_number":69,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE DEFAULT debug $SAVANNA_DEBUG"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE manager plugins\u003ddummy.vm.plugin"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    recreate_database climate utf8"},{"line_number":74,"context_line":"    iniset $CLIMATE_CONF_DIR/$CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9tBQ%3D","line":71,"updated":"2014-02-04 20:38:06.000000000","message":"This sets in /etc/climate/climate.conf\n\n[manager]                                                                       \nplugins\u003ddummy.vm.plugin \u003d\n\nI think this should be plugins dummy.vm.plugin","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"b69070095628ce1f37dd399f3588435cef6b59b2","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        --role_id $ADMIN_ROLE"},{"line_number":101,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":102,"context_line":"        CLIMATE_SERVICE\u003d$(keystone service-create \\"},{"line_number":103,"context_line":"            --name\u003dCLIMATE \\"},{"line_number":104,"context_line":"            --type\u003dshare \\"},{"line_number":105,"context_line":"            --description\u003d\"Climate Reservations Service\" \\"},{"line_number":106,"context_line":"            | grep \" id \" | get_field 2)"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9onA%3D","line":103,"updated":"2014-02-05 14:25:13.000000000","message":"Maybe the name should be lower case. The rest of the services created by devstack are lower case.","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"4938071a0e83da8a5f5159bcaa0f466d7d03b0ba","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        --role_id $ADMIN_ROLE"},{"line_number":101,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":102,"context_line":"        CLIMATE_SERVICE\u003d$(keystone service-create \\"},{"line_number":103,"context_line":"            --name\u003dCLIMATE \\"},{"line_number":104,"context_line":"            --type\u003dshare \\"},{"line_number":105,"context_line":"            --description\u003d\"Climate Reservations Service\" \\"},{"line_number":106,"context_line":"            | grep \" id \" | get_field 2)"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9mug%3D","line":103,"in_reply_to":"AAAAUH%2F9onA%3D","updated":"2014-02-05 18:07:11.000000000","message":"agreed","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"b69070095628ce1f37dd399f3588435cef6b59b2","unresolved":false,"context_lines":[{"line_number":101,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":102,"context_line":"        CLIMATE_SERVICE\u003d$(keystone service-create \\"},{"line_number":103,"context_line":"            --name\u003dCLIMATE \\"},{"line_number":104,"context_line":"            --type\u003dshare \\"},{"line_number":105,"context_line":"            --description\u003d\"Climate Reservations Service\" \\"},{"line_number":106,"context_line":"            | grep \" id \" | get_field 2)"},{"line_number":107,"context_line":"        keystone endpoint-create \\"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9oiM%3D","line":104,"updated":"2014-02-05 14:25:13.000000000","message":"I don\u0027t know much about keystone services, but seems that climateclient is asking for a service of \u0027reservation\u0027 type.","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"4938071a0e83da8a5f5159bcaa0f466d7d03b0ba","unresolved":false,"context_lines":[{"line_number":101,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":102,"context_line":"        CLIMATE_SERVICE\u003d$(keystone service-create \\"},{"line_number":103,"context_line":"            --name\u003dCLIMATE \\"},{"line_number":104,"context_line":"            --type\u003dshare \\"},{"line_number":105,"context_line":"            --description\u003d\"Climate Reservations Service\" \\"},{"line_number":106,"context_line":"            | grep \" id \" | get_field 2)"},{"line_number":107,"context_line":"        keystone endpoint-create \\"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F9muY%3D","line":104,"in_reply_to":"AAAAUH%2F9oiM%3D","updated":"2014-02-05 18:07:11.000000000","message":"agreed","commit_id":"527e82399163a7f2c89a22c0f01f8c9d674f2d18"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"aeafc017fcfbdd19504ec5dfd158a9a9ddb7d823","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_tenant_name $SERVICE_TENANT_NAME"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":74,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT log-exchange $CLIMATE_LOG_EXCHANGE"},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT debug $CLIMATE_DEBUG"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins dummy.vm.plugin"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAUH%2F9euw%3D","line":74,"updated":"2014-02-06 16:25:32.000000000","message":"There is no need to expose this config, user could use DevStack\u0027s local.conf","commit_id":"89040208a81d5a556102c98490a9529fe69a8c91"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"aeafc017fcfbdd19504ec5dfd158a9a9ddb7d823","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    fi"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    iniadd $NOVA_CONF DEFAULT scheduler_available_filters \"climatenova.scheduler.filters.climate_filter.ClimateFilter\""},{"line_number":94,"context_line":"    "},{"line_number":95,"context_line":"    iniadd $NOVA_CONF DEFAULT osapi_compute_extension \"climatenova.api.extensions.default_reservation.Default_reservation\""},{"line_number":96,"context_line":"    iniadd $NOVA_CONF DEFAULT osapi_compute_extension \"climatenova.api.extensions.reservation.Reservation\""},{"line_number":97,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAUH%2F9e6c%3D","line":94,"updated":"2014-02-06 16:25:32.000000000","message":"Please, remove whitespaces in empty line.","commit_id":"89040208a81d5a556102c98490a9529fe69a8c91"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"47bfcebb6fcd25b5c026c818f1225d5cb067d1d4","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT debug $CLIMATE_DEBUG"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins dummy.vm.plugin"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    recreate_database climate utf8"},{"line_number":78,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuN8%3D","line":75,"updated":"2014-02-11 08:26:28.000000000","message":"Please update with the correct trunk values","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"14d6e2011c4f549c6ef5023c42b48be38f4a273a","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT debug $CLIMATE_DEBUG"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins dummy.vm.plugin"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    recreate_database climate utf8"},{"line_number":78,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Fqfc%3D","line":75,"in_reply_to":"AAAAUX%2F%2FuAI%3D","updated":"2014-02-11 17:40:05.000000000","message":"Done","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"f2f221028b355e49fa5654ed00aa83d2a6b131a8","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT debug $CLIMATE_DEBUG"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins dummy.vm.plugin"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    recreate_database climate utf8"},{"line_number":78,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuAI%3D","line":75,"in_reply_to":"AAAAUX%2F%2FuDg%3D","updated":"2014-02-11 08:54:27.000000000","message":"Nah, that\u0027s true, sorry :)","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"efe21bfa2562d796a3e4236d31261ba16293a69d","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT debug $CLIMATE_DEBUG"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins dummy.vm.plugin"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    recreate_database climate utf8"},{"line_number":78,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuDg%3D","line":75,"in_reply_to":"AAAAUX%2F%2FuFo%3D","updated":"2014-02-11 08:50:24.000000000","message":"No, please think globally ;-)\n\nhttps://github.com/stackforge/climate/blob/master/etc/climate.conf.example#L14","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"4d1faa334d8fb05be4b733b1bae2c596667b7124","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT host $HOST_IP"},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT debug $CLIMATE_DEBUG"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins dummy.vm.plugin"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    recreate_database climate utf8"},{"line_number":78,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuFo%3D","line":75,"in_reply_to":"AAAAUX%2F%2FuN8%3D","updated":"2014-02-11 08:46:21.000000000","message":"it should be basic.vm.plugin here","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"47bfcebb6fcd25b5c026c818f1225d5cb067d1d4","unresolved":false,"context_lines":[{"line_number":129,"context_line":"            --internalurl \"$CLIMATE_SERVICE_PROTOCOL://$CLIMATE_SERVICE_HOST:$CLIMATE_SERVICE_PORT/v1\""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"        KEYSTONEV3_SERVICE\u003d$(keystone service-list \\"},{"line_number":132,"context_line":"                             | grep keystone3 | get_field 1)"},{"line_number":133,"context_line":"        if [[ -z \"$KEYSTONEV3_SERVICE\" ]]; then"},{"line_number":134,"context_line":"            KEYSTONEV3_SERVICE\u003d$(keystone service-create \\"},{"line_number":135,"context_line":"                --name\u003dkeystonev3 \\"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuNo%3D","line":132,"updated":"2014-02-11 08:26:28.000000000","message":"grep should be matching \"keystonev3\", typo here","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"14d6e2011c4f549c6ef5023c42b48be38f4a273a","unresolved":false,"context_lines":[{"line_number":129,"context_line":"            --internalurl \"$CLIMATE_SERVICE_PROTOCOL://$CLIMATE_SERVICE_HOST:$CLIMATE_SERVICE_PORT/v1\""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"        KEYSTONEV3_SERVICE\u003d$(keystone service-list \\"},{"line_number":132,"context_line":"                             | grep keystone3 | get_field 1)"},{"line_number":133,"context_line":"        if [[ -z \"$KEYSTONEV3_SERVICE\" ]]; then"},{"line_number":134,"context_line":"            KEYSTONEV3_SERVICE\u003d$(keystone service-create \\"},{"line_number":135,"context_line":"                --name\u003dkeystonev3 \\"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FqfI%3D","line":132,"in_reply_to":"AAAAUX%2F%2FuNo%3D","updated":"2014-02-11 17:40:05.000000000","message":"Done","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"47bfcebb6fcd25b5c026c818f1225d5cb067d1d4","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    git_clone $CLIMATECLIENT_REPO $CLIMATECLIENT_DIR $CLIMATECLIENT_BRANCH"},{"line_number":154,"context_line":"    git_clone $CLIMATENOVA_REPO $CLIMATENOVA_DIR $CLIMATENOVA_BRANCH"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    setup_develop $CLIMATE_DIR"},{"line_number":157,"context_line":"    setup_develop $CLIMATECLIENT_DIR"},{"line_number":158,"context_line":"    setup_develop $CLIMATENOVA_DIR"},{"line_number":159,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuNE%3D","line":156,"updated":"2014-02-11 08:26:28.000000000","message":"Due to the previous issues we had with python clients set to trunk, could we consider deploying Climate into a venv ?","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"f2f221028b355e49fa5654ed00aa83d2a6b131a8","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    git_clone $CLIMATECLIENT_REPO $CLIMATECLIENT_DIR $CLIMATECLIENT_BRANCH"},{"line_number":154,"context_line":"    git_clone $CLIMATENOVA_REPO $CLIMATENOVA_DIR $CLIMATENOVA_BRANCH"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    setup_develop $CLIMATE_DIR"},{"line_number":157,"context_line":"    setup_develop $CLIMATECLIENT_DIR"},{"line_number":158,"context_line":"    setup_develop $CLIMATENOVA_DIR"},{"line_number":159,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Ft%2Bw%3D","line":156,"in_reply_to":"AAAAUX%2F%2FuBs%3D","updated":"2014-02-11 08:54:27.000000000","message":"Yep, I understand your concerns. I only say that by default devstack uses current masters - and that\u0027s the reason I suppose we should continue that logic. I don\u0027t say we should not provide later integration tests, that will check that - but here should work Tempest, I suppose, not devstack.","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"977cd3740431dfd15629483dd80415eb81579b0c","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    git_clone $CLIMATECLIENT_REPO $CLIMATECLIENT_DIR $CLIMATECLIENT_BRANCH"},{"line_number":154,"context_line":"    git_clone $CLIMATENOVA_REPO $CLIMATENOVA_DIR $CLIMATENOVA_BRANCH"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    setup_develop $CLIMATE_DIR"},{"line_number":157,"context_line":"    setup_develop $CLIMATECLIENT_DIR"},{"line_number":158,"context_line":"    setup_develop $CLIMATENOVA_DIR"},{"line_number":159,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Ft%2Fg%3D","line":156,"in_reply_to":"AAAAUX%2F%2FuBs%3D","updated":"2014-02-11 08:54:11.000000000","message":"You need this devstack integration and dsvm job to early detect such problems, so, it should be installed as all other projects installed in OpenStack.","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"efe21bfa2562d796a3e4236d31261ba16293a69d","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    git_clone $CLIMATECLIENT_REPO $CLIMATECLIENT_DIR $CLIMATECLIENT_BRANCH"},{"line_number":154,"context_line":"    git_clone $CLIMATENOVA_REPO $CLIMATENOVA_DIR $CLIMATENOVA_BRANCH"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    setup_develop $CLIMATE_DIR"},{"line_number":157,"context_line":"    setup_develop $CLIMATECLIENT_DIR"},{"line_number":158,"context_line":"    setup_develop $CLIMATENOVA_DIR"},{"line_number":159,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuBs%3D","line":156,"in_reply_to":"AAAAUX%2F%2FuE0%3D","updated":"2014-02-11 08:50:24.000000000","message":"If we consider having requirements, we should test those requirements, hence the venv.\n\nWe lost 1 week for the keystoneclient issue, we\u0027re mitigating the novaclient issue because both were not identified at the development stage. I do understand this is not done the same way for other projects, but that\u0027s not a reason to shoot one bullet in our foot :-)","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"4d1faa334d8fb05be4b733b1bae2c596667b7124","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    git_clone $CLIMATECLIENT_REPO $CLIMATECLIENT_DIR $CLIMATECLIENT_BRANCH"},{"line_number":154,"context_line":"    git_clone $CLIMATENOVA_REPO $CLIMATENOVA_DIR $CLIMATENOVA_BRANCH"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    setup_develop $CLIMATE_DIR"},{"line_number":157,"context_line":"    setup_develop $CLIMATECLIENT_DIR"},{"line_number":158,"context_line":"    setup_develop $CLIMATENOVA_DIR"},{"line_number":159,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuE0%3D","line":156,"in_reply_to":"AAAAUX%2F%2FuNE%3D","updated":"2014-02-11 08:46:21.000000000","message":"Don\u0027t think so, it\u0027s devstack, so by default it installs everything to the testing node itself. I suppose we should continue this way of doing things - but we can just switch branches while deep-dive testing.","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"47bfcebb6fcd25b5c026c818f1225d5cb067d1d4","unresolved":false,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"# start_climate() - Start running processes, including screen"},{"line_number":163,"context_line":"function start_climate() {"},{"line_number":164,"context_line":"    screen_it climate-a \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-api --config-file $CLIMATE_CONF_FILE\""},{"line_number":165,"context_line":"    screen_it climate-m \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-manager --config-file $CLIMATE_CONF_FILE\""},{"line_number":166,"context_line":"}"},{"line_number":167,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuM8%3D","line":164,"updated":"2014-02-11 08:26:28.000000000","message":"enabling debugging is worth it","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"14d6e2011c4f549c6ef5023c42b48be38f4a273a","unresolved":false,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"# start_climate() - Start running processes, including screen"},{"line_number":163,"context_line":"function start_climate() {"},{"line_number":164,"context_line":"    screen_it climate-a \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-api --config-file $CLIMATE_CONF_FILE\""},{"line_number":165,"context_line":"    screen_it climate-m \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-manager --config-file $CLIMATE_CONF_FILE\""},{"line_number":166,"context_line":"}"},{"line_number":167,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Fqdk%3D","line":164,"in_reply_to":"AAAAUX%2F%2Ft%2FI%3D","updated":"2014-02-11 17:40:05.000000000","message":"Done","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"14d6e2011c4f549c6ef5023c42b48be38f4a273a","unresolved":false,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"# start_climate() - Start running processes, including screen"},{"line_number":163,"context_line":"function start_climate() {"},{"line_number":164,"context_line":"    screen_it climate-a \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-api --config-file $CLIMATE_CONF_FILE\""},{"line_number":165,"context_line":"    screen_it climate-m \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-manager --config-file $CLIMATE_CONF_FILE\""},{"line_number":166,"context_line":"}"},{"line_number":167,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Fqdo%3D","line":164,"in_reply_to":"AAAAUX%2F%2FuM8%3D","updated":"2014-02-11 17:40:05.000000000","message":"Done","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"977cd3740431dfd15629483dd80415eb81579b0c","unresolved":false,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"# start_climate() - Start running processes, including screen"},{"line_number":163,"context_line":"function start_climate() {"},{"line_number":164,"context_line":"    screen_it climate-a \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-api --config-file $CLIMATE_CONF_FILE\""},{"line_number":165,"context_line":"    screen_it climate-m \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-manager --config-file $CLIMATE_CONF_FILE\""},{"line_number":166,"context_line":"}"},{"line_number":167,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Ft%2FI%3D","line":164,"in_reply_to":"AAAAUX%2F%2FuM8%3D","updated":"2014-02-11 08:54:11.000000000","message":"debug prop could be easily bypassed to climate config using devstack\u0027s local.conf, but here, I think that debug by default could be useful.","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"47bfcebb6fcd25b5c026c818f1225d5cb067d1d4","unresolved":false,"context_lines":[{"line_number":162,"context_line":"# start_climate() - Start running processes, including screen"},{"line_number":163,"context_line":"function start_climate() {"},{"line_number":164,"context_line":"    screen_it climate-a \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-api --config-file $CLIMATE_CONF_FILE\""},{"line_number":165,"context_line":"    screen_it climate-m \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-manager --config-file $CLIMATE_CONF_FILE\""},{"line_number":166,"context_line":"}"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuM0%3D","line":165,"updated":"2014-02-11 08:26:28.000000000","message":"ditto L164","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"14d6e2011c4f549c6ef5023c42b48be38f4a273a","unresolved":false,"context_lines":[{"line_number":162,"context_line":"# start_climate() - Start running processes, including screen"},{"line_number":163,"context_line":"function start_climate() {"},{"line_number":164,"context_line":"    screen_it climate-a \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-api --config-file $CLIMATE_CONF_FILE\""},{"line_number":165,"context_line":"    screen_it climate-m \"cd $CLIMATE_DIR \u0026\u0026 $CLIMATE_BIN_DIR/climate-manager --config-file $CLIMATE_CONF_FILE\""},{"line_number":166,"context_line":"}"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Fqdg%3D","line":165,"in_reply_to":"AAAAUX%2F%2FuM0%3D","updated":"2014-02-11 17:40:05.000000000","message":"Done","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"47bfcebb6fcd25b5c026c818f1225d5cb067d1d4","unresolved":false,"context_lines":[{"line_number":170,"context_line":"function stop_climate() {"},{"line_number":171,"context_line":"    # Kill the climate screen windows"},{"line_number":172,"context_line":"    for serv in climate-a climate-m; do"},{"line_number":173,"context_line":"        screen_stop $serv"},{"line_number":174,"context_line":"    done"},{"line_number":175,"context_line":"}"},{"line_number":176,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuMY%3D","line":173,"updated":"2014-02-11 08:26:28.000000000","message":"theere is a possibility that the manager would still be running, we need to check if any processes left and kill them","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"f2f221028b355e49fa5654ed00aa83d2a6b131a8","unresolved":false,"context_lines":[{"line_number":170,"context_line":"function stop_climate() {"},{"line_number":171,"context_line":"    # Kill the climate screen windows"},{"line_number":172,"context_line":"    for serv in climate-a climate-m; do"},{"line_number":173,"context_line":"        screen_stop $serv"},{"line_number":174,"context_line":"    done"},{"line_number":175,"context_line":"}"},{"line_number":176,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2Ft%2F4%3D","line":173,"in_reply_to":"AAAAUX%2F%2FuBY%3D","updated":"2014-02-11 08:54:27.000000000","message":"I mean I\u0027m not sure if that bug will appear here as code itself and its stopping now seems ok for me. If it will work now without any workarounds - why not :)","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"efe21bfa2562d796a3e4236d31261ba16293a69d","unresolved":false,"context_lines":[{"line_number":170,"context_line":"function stop_climate() {"},{"line_number":171,"context_line":"    # Kill the climate screen windows"},{"line_number":172,"context_line":"    for serv in climate-a climate-m; do"},{"line_number":173,"context_line":"        screen_stop $serv"},{"line_number":174,"context_line":"    done"},{"line_number":175,"context_line":"}"},{"line_number":176,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuBY%3D","line":173,"in_reply_to":"AAAAUX%2F%2FuEg%3D","updated":"2014-02-11 08:50:24.000000000","message":"That\u0027s a known bug. Why shouldn\u0027t we provide a workaround here ?","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"4d1faa334d8fb05be4b733b1bae2c596667b7124","unresolved":false,"context_lines":[{"line_number":170,"context_line":"function stop_climate() {"},{"line_number":171,"context_line":"    # Kill the climate screen windows"},{"line_number":172,"context_line":"    for serv in climate-a climate-m; do"},{"line_number":173,"context_line":"        screen_stop $serv"},{"line_number":174,"context_line":"    done"},{"line_number":175,"context_line":"}"},{"line_number":176,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAUX%2F%2FuEg%3D","line":173,"in_reply_to":"AAAAUX%2F%2FuMY%3D","updated":"2014-02-11 08:46:21.000000000","message":"I suppose these lines should work as is - that should be tested separately.","commit_id":"9779e74cc212b84e9398453a46f2dd2362257d92"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"263b8b547d041bec9e3f2e9484429f5645a991c6","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_password $SERVICE_PASSWORD"},{"line_number":70,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_username climate"},{"line_number":71,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_tenant_name $SERVICE_TENANT_NAME"},{"line_number":72,"context_line":"    "},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT climate_username $CLIMATE_USER_NAME"},{"line_number":74,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT climate_password $SERVICE_PASSWORD"},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT climate_tenant_name $SERVICE_TENANT_NAME"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"AAAAUX%2F%2FqCc%3D","line":72,"updated":"2014-02-11 17:52:24.000000000","message":"nah, tabs should not be here.","commit_id":"80f94a127c6b11754e134fe0cc796ba24dcb89b0"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"f3374b4dcac3e0c108be0b24c308493545bd3b39","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_password $SERVICE_PASSWORD"},{"line_number":70,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_username climate"},{"line_number":71,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT os_admin_tenant_name $SERVICE_TENANT_NAME"},{"line_number":72,"context_line":"    "},{"line_number":73,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT climate_username $CLIMATE_USER_NAME"},{"line_number":74,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT climate_password $SERVICE_PASSWORD"},{"line_number":75,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT climate_tenant_name $SERVICE_TENANT_NAME"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"AAAAUX%2F%2Fp1E%3D","line":72,"in_reply_to":"AAAAUX%2F%2FqCc%3D","updated":"2014-02-11 18:19:38.000000000","message":"Done","commit_id":"80f94a127c6b11754e134fe0cc796ba24dcb89b0"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"263b8b547d041bec9e3f2e9484429f5645a991c6","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins basic.vm.plugin,physical.host.plugin"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    "},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    recreate_database climate utf8"},{"line_number":85,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"AAAAUX%2F%2FqCA%3D","line":82,"updated":"2014-02-11 17:52:24.000000000","message":"ditto line 72","commit_id":"80f94a127c6b11754e134fe0cc796ba24dcb89b0"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"f3374b4dcac3e0c108be0b24c308493545bd3b39","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    iniset $CLIMATE_CONF_FILE manager plugins basic.vm.plugin,physical.host.plugin"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    "},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    recreate_database climate utf8"},{"line_number":85,"context_line":"    iniset $CLIMATE_CONF_FILE database connection `database_connection_url climate`"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"AAAAUX%2F%2Fp1I%3D","line":82,"in_reply_to":"AAAAUX%2F%2FqCA%3D","updated":"2014-02-11 18:19:38.000000000","message":"Done","commit_id":"80f94a127c6b11754e134fe0cc796ba24dcb89b0"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"b63799a12c88c04663d7522b9267809fb5f4d560","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    iniset $CLIMATE_CONF_FILE DEFAULT use_syslog $SYSLOG"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    iniset_rpc_backend climate $CLIMATE_CONF_FILE DEFAULT"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    ACTUAL_FILTERS\u003d$(iniget $NOVA_CONF DEFAULT scheduler_default_filters)"},{"line_number":90,"context_line":"    if [[ -z \"$ACTUAL_FILTERS\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"AAAAUX%2F%2FRt4%3D","line":87,"updated":"2014-02-14 09:38:59.000000000","message":"It will be \n\nrpc_backend \u003d climate.openstack.common.rpc.impl_kombu\n\nline here. As we use oslo.messaging now, we should insert:\n\nrpc_backend \u003d rabbit","commit_id":"bf9bdf1758c991604cf75e227297986a7f1eaa4d"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"f6e16b5e8719af16b891422671636eeca21bab40","unresolved":false,"context_lines":[{"line_number":54,"context_line":"# Oslo.Messaging RPC iniupdate cofiguration"},{"line_number":55,"context_line":"function iniupdate_rpc_backend() {"},{"line_number":56,"context_line":"    local file\u003d$1"},{"line_number":57,"context_line":"    local section\u003d$2                                                            "},{"line_number":58,"context_line":"    if is_service_enabled zeromq; then"},{"line_number":59,"context_line":"        iniset $file $section rpc_backend zmq"},{"line_number":60,"context_line":"    elif is_service_enabled qpid || [ -n \"$QPID_HOST\" ]; then"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAAUX%2F%2FNg8%3D","line":57,"updated":"2014-02-14 20:45:24.000000000","message":"please fix the empty spaces here","commit_id":"61c18f0c3e899f0fffab535951f870595aec868d"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"7821c9bc3a830d64a6d4e9d0fa96dba01ed98873","unresolved":false,"context_lines":[{"line_number":54,"context_line":"# Oslo.Messaging RPC iniupdate cofiguration"},{"line_number":55,"context_line":"function iniupdate_rpc_backend() {"},{"line_number":56,"context_line":"    local file\u003d$1"},{"line_number":57,"context_line":"    local section\u003d$2                                                            "},{"line_number":58,"context_line":"    if is_service_enabled zeromq; then"},{"line_number":59,"context_line":"        iniset $file $section rpc_backend zmq"},{"line_number":60,"context_line":"    elif is_service_enabled qpid || [ -n \"$QPID_HOST\" ]; then"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAAUX%2F%2FErU%3D","line":57,"in_reply_to":"AAAAUX%2F%2FNg8%3D","updated":"2014-02-17 10:03:46.000000000","message":"+1 :)\nLook carefully on what do your text editor does sometimes :) It looks like it\u0027s doing smth naughty when you are not watching it :)","commit_id":"61c18f0c3e899f0fffab535951f870595aec868d"}]}