)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"da051afc4ab45e11067da1409dbb1e5559c56391","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Allow specifying devstack branch in redstack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"At present, redstack clones the most recent version of devstack. Sometimes this"},{"line_number":10,"context_line":"is undesirable as devstack can at times become unstable. A solution to this"},{"line_number":11,"context_line":"would be to add an environment variable or option to redstack to specify the"},{"line_number":12,"context_line":"branch of devstack to use (with the default remaining \u0027master\u0027)."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a016987_108bccc4","line":10,"updated":"2015-05-28 18:21:21.000000000","message":"how\u0027s that different from any other project?","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"da051afc4ab45e11067da1409dbb1e5559c56391","unresolved":false,"context_lines":[{"line_number":19,"context_line":"Optionally, for persistence, it can be defined in either options.rc or"},{"line_number":20,"context_line":"~/.redstack.options.rc and it will be picked up from there."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"The branch is switched, even if ithe devstack directory already existed."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"If an invalid branch is specified, redstack stops with an error message."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a016987_f0003036","line":22,"updated":"2015-05-28 18:21:21.000000000","message":"s/ithe/the/","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"}],"scripts/functions":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"c559cf26dde6640b6232db65d79a25098b81c8e6","unresolved":false,"context_lines":[{"line_number":299,"context_line":"            REPO_BRANCH\u003dmaster"},{"line_number":300,"context_line":"        fi"},{"line_number":301,"context_line":"        GIT_STATUS\u003d$(git checkout \"$REPO_BRANCH\" \u0026\u003e /dev/null || echo \"failed\")"},{"line_number":302,"context_line":"        if [[ \"$GIT_STATUS\" \u003d \"failed\" ]]; then"},{"line_number":303,"context_line":"            exclaim \"${COLOR_RED}Could not switch to branch/commit \u0027$REPO_BRANCH\u0027 in $PROJECT, exiting${COLOR_NONE}\""},{"line_number":304,"context_line":"            echo \"Please set \u0027$REPO_BRANCH_VAR_NAME\u0027 to a valid branch/commit and try again.\""},{"line_number":305,"context_line":"            echo \"Note: This error may also mean that there are modified files in $PROJECT.\""}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"5afe65bd_65290be6","line":302,"updated":"2015-06-04 18:17:11.000000000","message":"This is an interesting approach to things; why not just run the command and check the exist status?","commit_id":"5c909549c0ff41d62512f7c721c6ed7a6cdaffaa"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"fe6c31ff655479117490f98781cb0a0a7a4a94e7","unresolved":false,"context_lines":[{"line_number":299,"context_line":"            REPO_BRANCH\u003dmaster"},{"line_number":300,"context_line":"        fi"},{"line_number":301,"context_line":"        GIT_STATUS\u003d$(git checkout \"$REPO_BRANCH\" \u0026\u003e /dev/null || echo \"failed\")"},{"line_number":302,"context_line":"        if [[ \"$GIT_STATUS\" \u003d \"failed\" ]]; then"},{"line_number":303,"context_line":"            exclaim \"${COLOR_RED}Could not switch to branch/commit \u0027$REPO_BRANCH\u0027 in $PROJECT, exiting${COLOR_NONE}\""},{"line_number":304,"context_line":"            echo \"Please set \u0027$REPO_BRANCH_VAR_NAME\u0027 to a valid branch/commit and try again.\""},{"line_number":305,"context_line":"            echo \"Note: This error may also mean that there are modified files in $PROJECT.\""}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3afb71cf_07c5585a","line":302,"in_reply_to":"5afe65bd_65290be6","updated":"2015-06-04 19:33:28.000000000","message":"Because redstack runs with \u0027set -e\u0027 - that\u0027s means any error will cause the script to exit immediately without warning","commit_id":"5c909549c0ff41d62512f7c721c6ed7a6cdaffaa"}],"scripts/redstack":[{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"e9fcf72daeb80eb00c6946acb7356df722fcf449","unresolved":false,"context_lines":[{"line_number":198,"context_line":"                exclaim \"${COLOR_BLUE}Using devstack branch \u0027$DEVSTACK_BRANCH\u0027${COLOR_NONE}\""},{"line_number":199,"context_line":"                git checkout \"$DEVSTACK_BRANCH\""},{"line_number":200,"context_line":"            else"},{"line_number":201,"context_line":"                exclaim \"${COLOR_RED}Branch \u0027$DEVSTACK_BRANCH\u0027 not found in devstack, staying on \u0027master\u0027${COLOR_NONE}\""},{"line_number":202,"context_line":"            fi"},{"line_number":203,"context_line":"            popd"},{"line_number":204,"context_line":"        fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"5ad0253b_b2dda972","line":201,"updated":"2015-04-28 20:09:54.000000000","message":"I think this should fail - stop execution of the redstack script. If the specified branch does not exist the caller will probably want to interrupt the script anyway.","commit_id":"ca97f93ef2ee6dc6c3528959f23daf7110268bee"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"88ff8d8736489039364a920decfcbe5e8d9303ad","unresolved":false,"context_lines":[{"line_number":198,"context_line":"                exclaim \"${COLOR_BLUE}Using devstack branch \u0027$DEVSTACK_BRANCH\u0027${COLOR_NONE}\""},{"line_number":199,"context_line":"                git checkout \"$DEVSTACK_BRANCH\""},{"line_number":200,"context_line":"            else"},{"line_number":201,"context_line":"                exclaim \"${COLOR_RED}Branch \u0027$DEVSTACK_BRANCH\u0027 not found in devstack, staying on \u0027master\u0027${COLOR_NONE}\""},{"line_number":202,"context_line":"            fi"},{"line_number":203,"context_line":"            popd"},{"line_number":204,"context_line":"        fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"5ad0253b_cdb404d3","line":201,"in_reply_to":"5ad0253b_b2dda972","updated":"2015-04-28 20:34:18.000000000","message":"Done","commit_id":"ca97f93ef2ee6dc6c3528959f23daf7110268bee"},{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"75c0b4d1ade05b3fa0772ed18f588c7fa46643d9","unresolved":false,"context_lines":[{"line_number":187,"context_line":"function install_devstack_code() {"},{"line_number":188,"context_line":"    exclaim \"Installing devstack...\""},{"line_number":189,"context_line":"    # Installs devstack (if needed)."},{"line_number":190,"context_line":"    if [ ! -d $PATH_DEVSTACK_SRC ]; then"},{"line_number":191,"context_line":"        echo \"DevStack not in a shared folder, cloning from git.\""},{"line_number":192,"context_line":"        mkdir -p $PATH_DEVSTACK_SRC"},{"line_number":193,"context_line":"        git clone $REPO_DEVSTACK $PATH_DEVSTACK_SRC"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"3acd31a7_297ed5ae","line":190,"range":{"start_line":190,"start_character":15,"end_line":190,"end_character":32},"updated":"2015-04-29 16:11:39.000000000","message":"I think it would be great if we could allow branch switching even if devstack already exists. Right now one has to remove the devstack directory manually.","commit_id":"8f1d11464e02ded37bf59f408100510f3bca1c32"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"d92a19e0d00284ece7e8fa7315ea16f79ba1c828","unresolved":false,"context_lines":[{"line_number":187,"context_line":"function install_devstack_code() {"},{"line_number":188,"context_line":"    exclaim \"Installing devstack...\""},{"line_number":189,"context_line":"    # Installs devstack (if needed)."},{"line_number":190,"context_line":"    if [ ! -d $PATH_DEVSTACK_SRC ]; then"},{"line_number":191,"context_line":"        echo \"DevStack not in a shared folder, cloning from git.\""},{"line_number":192,"context_line":"        mkdir -p $PATH_DEVSTACK_SRC"},{"line_number":193,"context_line":"        git clone $REPO_DEVSTACK $PATH_DEVSTACK_SRC"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"3acd31a7_1bd0fc52","line":190,"range":{"start_line":190,"start_character":15,"end_line":190,"end_character":32},"in_reply_to":"3acd31a7_297ed5ae","updated":"2015-04-29 16:44:37.000000000","message":"Done","commit_id":"8f1d11464e02ded37bf59f408100510f3bca1c32"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"f17c3ec1849d242fbb5d7f40f28031b32ed9ecaa","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                exit 1"},{"line_number":204,"context_line":"            fi"},{"line_number":205,"context_line":"            popd"},{"line_number":206,"context_line":"        fi"},{"line_number":207,"context_line":"    fi"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    exclaim \"Installing devstack projects...\""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5ad0253b_c0e4bd0e","line":206,"updated":"2015-04-29 07:42:54.000000000","message":"Can we simplify this to just test for whether $DEVSTACK_BRANCH is defined and checkout $DEVSTACK_BRANCH in that case?","commit_id":"8f1d11464e02ded37bf59f408100510f3bca1c32"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"d92a19e0d00284ece7e8fa7315ea16f79ba1c828","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                exit 1"},{"line_number":204,"context_line":"            fi"},{"line_number":205,"context_line":"            popd"},{"line_number":206,"context_line":"        fi"},{"line_number":207,"context_line":"    fi"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    exclaim \"Installing devstack projects...\""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"3acd31a7_3be4d839","line":206,"in_reply_to":"5ad0253b_c0e4bd0e","updated":"2015-04-29 16:44:37.000000000","message":"Done","commit_id":"8f1d11464e02ded37bf59f408100510f3bca1c32"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"ce137d3a876a5f451ebc4bf738d6e7a1acd0964c","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    # See if we need to switch devstack to a new branch (i.e. \u0027stable/kilo\u0027)"},{"line_number":196,"context_line":"    if [ -n \"$DEVSTACK_BRANCH\" ]; then"},{"line_number":197,"context_line":"        pushd $PATH_DEVSTACK_SRC"},{"line_number":198,"context_line":"        DEVSTACK_BRANCHES\u003d$(git branch -r | grep -v \"origin/HEAD\")"},{"line_number":199,"context_line":"        if [[ \"$DEVSTACK_BRANCHES\" \u003d~ \"origin/$DEVSTACK_BRANCH\" ]]; then"},{"line_number":200,"context_line":"            exclaim \"${COLOR_BLUE}Using devstack branch \u0027$DEVSTACK_BRANCH\u0027${COLOR_NONE}\""},{"line_number":201,"context_line":"            git checkout \"$DEVSTACK_BRANCH\""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"7a016987_583f78d6","line":198,"updated":"2015-05-27 07:57:02.000000000","message":"This might be more useful if in addition to allowing the branch, we also allow the actual commit hash to checkout.\n\nPerhaps instead of eagerly checking that a branch exists, we can try the checkout and exclaim and fail if it doesn\u0027t succeed? That would work for tags as well...","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"e0edb9158102123e22ad3234e16e615340a50e04","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    # See if we need to switch devstack to a new branch (i.e. \u0027stable/kilo\u0027)"},{"line_number":196,"context_line":"    if [ -n \"$DEVSTACK_BRANCH\" ]; then"},{"line_number":197,"context_line":"        pushd $PATH_DEVSTACK_SRC"},{"line_number":198,"context_line":"        DEVSTACK_BRANCHES\u003d$(git branch -r | grep -v \"origin/HEAD\")"},{"line_number":199,"context_line":"        if [[ \"$DEVSTACK_BRANCHES\" \u003d~ \"origin/$DEVSTACK_BRANCH\" ]]; then"},{"line_number":200,"context_line":"            exclaim \"${COLOR_BLUE}Using devstack branch \u0027$DEVSTACK_BRANCH\u0027${COLOR_NONE}\""},{"line_number":201,"context_line":"            git checkout \"$DEVSTACK_BRANCH\""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"5afe65bd_398a80c0","line":198,"in_reply_to":"7a016987_583f78d6","updated":"2015-06-02 19:48:50.000000000","message":"Done","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"bdb0b02e1cdc95d4d17f7fb3d54b84294d6f810d","unresolved":false,"context_lines":[{"line_number":203,"context_line":"            exclaim \"${COLOR_RED}Branch \u0027$DEVSTACK_BRANCH\u0027 not found in devstack, exiting${COLOR_NONE}\""},{"line_number":204,"context_line":"            echo \"Please set \u0027DEVSTACK_BRANCH\u0027 to a valid branch and try again.\""},{"line_number":205,"context_line":"            exit 1"},{"line_number":206,"context_line":"        fi"},{"line_number":207,"context_line":"        popd"},{"line_number":208,"context_line":"    fi"},{"line_number":209,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3acd31a7_051d3573","line":206,"updated":"2015-04-29 19:16:49.000000000","message":"Nikhil: Ah, I think I misunderstood your comment on patchset 2. I (now) believe you meant that I should take out the check if the branch exists.  Here\u0027s the result (output) of doing that, if you give a bad branch:\n\n  /home/ubuntu/devstack /home/ubuntu/work/dev/ti/trove-integration/scripts\n  *******************************************************************************\n  Using devstack branch \u0027masters\u0027\n  *******************************************************************************\n  error: pathspec \u0027masters\u0027 did not match any file(s) known to git.\n\nAnd here it is with the code as it is now:\n\n  /home/ubuntu/devstack /home/ubuntu/work/dev/ti/trove-integration/scripts\n  *******************************************************************************\n  Branch \u0027masters\u0027 not found in devstack, exiting\n  *******************************************************************************\n  Please set \u0027DEVSTACK_BRANCH\u0027 to a valid branch and try again.\n\nLet me know if you think the check should be taken out.","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"da051afc4ab45e11067da1409dbb1e5559c56391","unresolved":false,"context_lines":[{"line_number":993,"context_line":"function cmd_clone_projects() {"},{"line_number":994,"context_line":"    UPDATE_PROJECTS\u003d$1"},{"line_number":995,"context_line":"    PROJECT_LIST_FILES\u003d${@:2}"},{"line_number":996,"context_line":"    for project in $(cat $PROJECT_LIST_FILES); do"},{"line_number":997,"context_line":"        if [ ! -d $PATH_DEVSTACK_OUTPUT/$project ]; then"},{"line_number":998,"context_line":"            echo \"Creating a new clone of $project...\""},{"line_number":999,"context_line":"            git_clone $OPENSTACK_GITSOURCE/\"$project\".git ${PATH_DEVSTACK_OUTPUT}/$project master"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"7a016987_7037a0ff","line":996,"updated":"2015-05-28 18:21:21.000000000","message":"Wouldn\u0027t these projects also potentially end up being unstable?","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"e0edb9158102123e22ad3234e16e615340a50e04","unresolved":false,"context_lines":[{"line_number":993,"context_line":"function cmd_clone_projects() {"},{"line_number":994,"context_line":"    UPDATE_PROJECTS\u003d$1"},{"line_number":995,"context_line":"    PROJECT_LIST_FILES\u003d${@:2}"},{"line_number":996,"context_line":"    for project in $(cat $PROJECT_LIST_FILES); do"},{"line_number":997,"context_line":"        if [ ! -d $PATH_DEVSTACK_OUTPUT/$project ]; then"},{"line_number":998,"context_line":"            echo \"Creating a new clone of $project...\""},{"line_number":999,"context_line":"            git_clone $OPENSTACK_GITSOURCE/\"$project\".git ${PATH_DEVSTACK_OUTPUT}/$project master"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"5afe65bd_99c174e7","line":996,"in_reply_to":"7a016987_7037a0ff","updated":"2015-06-02 19:48:50.000000000","message":"I\u0027m thinking about this - I\u0027ll try and come up with something in the following patchset (the next one just addresses Nikhil\u0027s concern)","commit_id":"0927be2a83acb051727864206ad79bae7cfffede"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"c559cf26dde6640b6232db65d79a25098b81c8e6","unresolved":false,"context_lines":[{"line_number":996,"context_line":"                git pull"},{"line_number":997,"context_line":"            fi"},{"line_number":998,"context_line":"        fi"},{"line_number":999,"context_line":"        PROJECT_BRANCH_NAME\u003d$(eval echo \"${project}_BRANCH\" | tr \u0027[:lower:]\u0027 \u0027[:upper:]\u0027)"},{"line_number":1000,"context_line":"        DEFAULT_BRANCH\u003d\"$PROJECT_BRANCH\""},{"line_number":1001,"context_line":"        ENV_VARS\u003d\"$PROJECT_BRANCH_NAME\u0027 or \u0027PROJECT_BRANCH\""},{"line_number":1002,"context_line":"        if [[ \"$project\" \u003d \"trove\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"5afe65bd_85d407bd","line":999,"updated":"2015-06-04 18:17:11.000000000","message":"would you put these three lines in an else under the if below?\n\nAlso this logic and the one for devstack above should all be handled in get_project_branch; I think.","commit_id":"5c909549c0ff41d62512f7c721c6ed7a6cdaffaa"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"fe6c31ff655479117490f98781cb0a0a7a4a94e7","unresolved":false,"context_lines":[{"line_number":996,"context_line":"                git pull"},{"line_number":997,"context_line":"            fi"},{"line_number":998,"context_line":"        fi"},{"line_number":999,"context_line":"        PROJECT_BRANCH_NAME\u003d$(eval echo \"${project}_BRANCH\" | tr \u0027[:lower:]\u0027 \u0027[:upper:]\u0027)"},{"line_number":1000,"context_line":"        DEFAULT_BRANCH\u003d\"$PROJECT_BRANCH\""},{"line_number":1001,"context_line":"        ENV_VARS\u003d\"$PROJECT_BRANCH_NAME\u0027 or \u0027PROJECT_BRANCH\""},{"line_number":1002,"context_line":"        if [[ \"$project\" \u003d \"trove\" ]]; then"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"3afb71cf_27dbb41d","line":999,"in_reply_to":"5afe65bd_85d407bd","updated":"2015-06-04 19:33:28.000000000","message":"PROJECT_BRANCH_NAME is the same for both cases, plus it\u0027s needed here as well as in get_project_branch (that\u0027s why I pulled it out of the function, since it was duped).\n\nI can move the other two lines to an \u0027else\u0027 statement (if you feel strongly about it), but it doesn\u0027t change the code - it just puts the exception case before the typical one","commit_id":"5c909549c0ff41d62512f7c721c6ed7a6cdaffaa"}],"scripts/redstack.rc":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"f17c3ec1849d242fbb5d7f40f28031b32ed9ecaa","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# Devstack and OpenStack git repo source info"},{"line_number":43,"context_line":"REPO_DEVSTACK\u003dgit://git.openstack.org/openstack-dev/devstack.git"},{"line_number":44,"context_line":"DEVSTACK_DEFAULT_BRANCH\u003dmaster"},{"line_number":45,"context_line":"# Set this to peg the branch of devstack (i.e. \u0027stable/kilo\u0027)"},{"line_number":46,"context_line":"DEVSTACK_BRANCH\u003d${DEVSTACK_BRANCH:-$DEVSTACK_DEFAULT_BRANCH}"},{"line_number":47,"context_line":"OPENSTACK_GITSOURCE\u003dgit://git.openstack.org/openstack"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5ad0253b_40630db2","line":44,"updated":"2015-04-29 07:42:54.000000000","message":"Is this DEVSTACK_DEFAULT_BRANCH var needed? \nWhy not just default to master if DEVSTACK_BRANCH is not set?","commit_id":"8f1d11464e02ded37bf59f408100510f3bca1c32"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"d92a19e0d00284ece7e8fa7315ea16f79ba1c828","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# Devstack and OpenStack git repo source info"},{"line_number":43,"context_line":"REPO_DEVSTACK\u003dgit://git.openstack.org/openstack-dev/devstack.git"},{"line_number":44,"context_line":"DEVSTACK_DEFAULT_BRANCH\u003dmaster"},{"line_number":45,"context_line":"# Set this to peg the branch of devstack (i.e. \u0027stable/kilo\u0027)"},{"line_number":46,"context_line":"DEVSTACK_BRANCH\u003d${DEVSTACK_BRANCH:-$DEVSTACK_DEFAULT_BRANCH}"},{"line_number":47,"context_line":"OPENSTACK_GITSOURCE\u003dgit://git.openstack.org/openstack"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"3acd31a7_696a8d1c","line":44,"in_reply_to":"5ad0253b_40630db2","updated":"2015-04-29 16:44:37.000000000","message":"I use this to check if we\u0027re on the \u0027default\u0027 branch later, to avoid having to hardcode \u0027master\u0027 twice.  I know it\u0027ll probably never change - I just have the habit of always creating a variable if I\u0027m going to use a value more than once.\n\nBut I can pull it out entirely, based on your comments in the redstack file","commit_id":"8f1d11464e02ded37bf59f408100510f3bca1c32"}]}