)]}'
{"driver_certs/cinder_driver_cert.sh":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1d79e2fe26ce8d1db29e8fb3093d1fb90e1465ba","unresolved":false,"context_lines":[{"line_number":6,"context_line":"TOP_DIR\u003d$(cd $CERT_DIR/..; pwd)"},{"line_number":7,"context_line":"source $TOP_DIR/functions"},{"line_number":8,"context_line":"source $TOP_DIR/stackrc"},{"line_number":9,"context_line":"source $TOP_DIR/localrc"},{"line_number":10,"context_line":"source $TOP_DIR/openrc"},{"line_number":11,"context_line":"TEMPFILE\u003d`mktemp`"},{"line_number":12,"context_line":"RECLONE\u003dTrue"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2Fico%3D","line":9,"updated":"2013-11-01 18:50:58.000000000","message":"Please don\u0027t source locallrc. stackrc already does and with the new local.conf support this will break.","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"97e40dcc999db528c0d4e31705101e0ce226ae79","unresolved":false,"context_lines":[{"line_number":6,"context_line":"TOP_DIR\u003d$(cd $CERT_DIR/..; pwd)"},{"line_number":7,"context_line":"source $TOP_DIR/functions"},{"line_number":8,"context_line":"source $TOP_DIR/stackrc"},{"line_number":9,"context_line":"source $TOP_DIR/localrc"},{"line_number":10,"context_line":"source $TOP_DIR/openrc"},{"line_number":11,"context_line":"TEMPFILE\u003d`mktemp`"},{"line_number":12,"context_line":"RECLONE\u003dTrue"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FOpk%3D","line":9,"in_reply_to":"AAAATn%2F%2Fico%3D","updated":"2013-11-12 15:26:16.000000000","message":"Done","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1d79e2fe26ce8d1db29e8fb3093d1fb90e1465ba","unresolved":false,"context_lines":[{"line_number":10,"context_line":"source $TOP_DIR/openrc"},{"line_number":11,"context_line":"TEMPFILE\u003d`mktemp`"},{"line_number":12,"context_line":"RECLONE\u003dTrue"},{"line_number":13,"context_line":"CINDER_DIR\u003d$DEST/cinder"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"function log_message() {"},{"line_number":16,"context_line":"    MESSAGE\u003d$1"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2Ficw%3D","line":13,"updated":"2013-11-01 18:50:58.000000000","message":"If you source lib/cinder you\u0027ll get CINDER_DIR and all of the other conf vars automagically.","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"97e40dcc999db528c0d4e31705101e0ce226ae79","unresolved":false,"context_lines":[{"line_number":10,"context_line":"source $TOP_DIR/openrc"},{"line_number":11,"context_line":"TEMPFILE\u003d`mktemp`"},{"line_number":12,"context_line":"RECLONE\u003dTrue"},{"line_number":13,"context_line":"CINDER_DIR\u003d$DEST/cinder"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"function log_message() {"},{"line_number":16,"context_line":"    MESSAGE\u003d$1"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FOpc%3D","line":13,"in_reply_to":"AAAATn%2F%2Ficw%3D","updated":"2013-11-12 15:26:16.000000000","message":"Done","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1d79e2fe26ce8d1db29e8fb3093d1fb90e1465ba","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"cd $TOP_DIR"},{"line_number":51,"context_line":"# Verify tempest is installed/enabled"},{"line_number":52,"context_line":"if [[ \"$ENABLED_SERVICES\" \u003d~ \"tempest\" ]]; then"},{"line_number":53,"context_line":"    TEMPEST_DIR\u003d$DEST/tempest"},{"line_number":54,"context_line":"    cd $TEMPEST_DIR"},{"line_number":55,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FicM%3D","line":52,"updated":"2013-11-01 18:50:58.000000000","message":"Use is_service_enabled() for the check.  And I think checking for tempest not enabled and exiting here would be cleaner.","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"97e40dcc999db528c0d4e31705101e0ce226ae79","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"cd $TOP_DIR"},{"line_number":51,"context_line":"# Verify tempest is installed/enabled"},{"line_number":52,"context_line":"if [[ \"$ENABLED_SERVICES\" \u003d~ \"tempest\" ]]; then"},{"line_number":53,"context_line":"    TEMPEST_DIR\u003d$DEST/tempest"},{"line_number":54,"context_line":"    cd $TEMPEST_DIR"},{"line_number":55,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FOpU%3D","line":52,"in_reply_to":"AAAATn%2F%2FicM%3D","updated":"2013-11-12 15:26:16.000000000","message":"Done","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1d79e2fe26ce8d1db29e8fb3093d1fb90e1465ba","unresolved":false,"context_lines":[{"line_number":50,"context_line":"cd $TOP_DIR"},{"line_number":51,"context_line":"# Verify tempest is installed/enabled"},{"line_number":52,"context_line":"if [[ \"$ENABLED_SERVICES\" \u003d~ \"tempest\" ]]; then"},{"line_number":53,"context_line":"    TEMPEST_DIR\u003d$DEST/tempest"},{"line_number":54,"context_line":"    cd $TEMPEST_DIR"},{"line_number":55,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":56,"context_line":"    setup_develop $TEMPEST_DIR"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FicI%3D","line":53,"updated":"2013-11-01 18:50:58.000000000","message":"Again, sourcing lib/tempest will get you TEMPEST_DIR and other vars.","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"97e40dcc999db528c0d4e31705101e0ce226ae79","unresolved":false,"context_lines":[{"line_number":50,"context_line":"cd $TOP_DIR"},{"line_number":51,"context_line":"# Verify tempest is installed/enabled"},{"line_number":52,"context_line":"if [[ \"$ENABLED_SERVICES\" \u003d~ \"tempest\" ]]; then"},{"line_number":53,"context_line":"    TEMPEST_DIR\u003d$DEST/tempest"},{"line_number":54,"context_line":"    cd $TEMPEST_DIR"},{"line_number":55,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":56,"context_line":"    setup_develop $TEMPEST_DIR"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FOpQ%3D","line":53,"in_reply_to":"AAAATn%2F%2FicI%3D","updated":"2013-11-12 15:26:16.000000000","message":"Done","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1d79e2fe26ce8d1db29e8fb3093d1fb90e1465ba","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    TEMPEST_DIR\u003d$DEST/tempest"},{"line_number":54,"context_line":"    cd $TEMPEST_DIR"},{"line_number":55,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":56,"context_line":"    setup_develop $TEMPEST_DIR"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    log_message \"Verify tempest is current....\", True"},{"line_number":59,"context_line":"    git status | tee -a $TEMPFILE"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2Fib4%3D","line":56,"updated":"2013-11-01 18:50:58.000000000","message":"If tempest is enabled isn\u0027t it already installed?  Or is this possibly updating to a different branch?\n\nEither way, I think the tempest functions can be looked at as install_tempest() and friends should be doing this.","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"97e40dcc999db528c0d4e31705101e0ce226ae79","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    TEMPEST_DIR\u003d$DEST/tempest"},{"line_number":54,"context_line":"    cd $TEMPEST_DIR"},{"line_number":55,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":56,"context_line":"    setup_develop $TEMPEST_DIR"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    log_message \"Verify tempest is current....\", True"},{"line_number":59,"context_line":"    git status | tee -a $TEMPFILE"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FOpM%3D","line":56,"in_reply_to":"AAAATn%2F%2Fib4%3D","updated":"2013-11-12 15:26:16.000000000","message":"Done","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1d79e2fe26ce8d1db29e8fb3093d1fb90e1465ba","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    git log --pretty\u003doneline -n 1 | tee -a $TEMPFILE"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    #stop and restart cinder services"},{"line_number":65,"context_line":"    log_message \"Restart Cinder services...\", True"},{"line_number":66,"context_line":"    screen -S stack -p c-api -X stuff $\u0027\\003\u0027"},{"line_number":67,"context_line":"    screen -S stack -p c-api -X stuff \"cd /opt/stack/cinder \u0026\u0026 /opt/stack/cinder/bin/cinder-api --config-file /etc/cinder/cinder.conf"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2Fibk%3D","line":64,"updated":"2013-11-01 18:50:58.000000000","message":"Could start_cinder() and stop_cinder() be used here instead for the service restarts?","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"97e40dcc999db528c0d4e31705101e0ce226ae79","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    git log --pretty\u003doneline -n 1 | tee -a $TEMPFILE"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    #stop and restart cinder services"},{"line_number":65,"context_line":"    log_message \"Restart Cinder services...\", True"},{"line_number":66,"context_line":"    screen -S stack -p c-api -X stuff $\u0027\\003\u0027"},{"line_number":67,"context_line":"    screen -S stack -p c-api -X stuff \"cd /opt/stack/cinder \u0026\u0026 /opt/stack/cinder/bin/cinder-api --config-file /etc/cinder/cinder.conf"}],"source_content_type":"text/x-sh","patch_set":5,"id":"AAAATn%2F%2FOpI%3D","line":64,"in_reply_to":"AAAATn%2F%2Fibk%3D","updated":"2013-11-12 15:26:16.000000000","message":"Done","commit_id":"10284e072f864da97fac3cdf545f445d2e712ace"}]}