)]}'
{"lib/keystone":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"c4c098334bad5aa0f957c4f260920bc8abdebb03","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# - ``S3_SERVICE_PORT`` (template backend only)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# ``stack.sh`` calls the entry points in this order:"},{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"# - install_keystone"},{"line_number":18,"context_line":"# - configure_keystone"},{"line_number":19,"context_line":"# - _config_keystone_apache_wsgi"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"1ae5cdf2_66ebbba4","line":16,"updated":"2014-06-24 20:07:19.000000000","message":"is the new function supposed to be listed here? (looks like install_keystoneclient is missing, too)","commit_id":"9dd90281756b1a7e818bf4ef69c17806aec5d2c2"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"77e641636a7d57ed648dcc1f8c8f1fd5b01e5df3","unresolved":false,"context_lines":[{"line_number":442,"context_line":"# install_keystonemiddleware() - Collect source and prepare"},{"line_number":443,"context_line":"function install_keystonemiddleware {"},{"line_number":444,"context_line":"    git_clone $KEYSTONEMIDDLEWARE_REPO $KEYSTONEMIDDLEWARE_DIR $KEYSTONEMIDDLEWARE_BRANCH"},{"line_number":445,"context_line":"    setup_develop $KEYSTONECLIENT_DIR"},{"line_number":446,"context_line":"}"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"# install_keystone() - Collect source and prepare"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"dab895d6_b3cd5889","line":445,"updated":"2014-06-29 12:15:56.000000000","message":"Should we be doing setup_install instead? If we aren\u0027t starting a service with this, I think setup_install should be used otherwise things get a little goofy because of editable vs. not editable installs.","commit_id":"9dd90281756b1a7e818bf4ef69c17806aec5d2c2"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"e964c765dfcc27c34cd1c579a5328f5ac01252ce","unresolved":false,"context_lines":[{"line_number":442,"context_line":"# install_keystonemiddleware() - Collect source and prepare"},{"line_number":443,"context_line":"function install_keystonemiddleware {"},{"line_number":444,"context_line":"    git_clone $KEYSTONEMIDDLEWARE_REPO $KEYSTONEMIDDLEWARE_DIR $KEYSTONEMIDDLEWARE_BRANCH"},{"line_number":445,"context_line":"    setup_develop $KEYSTONECLIENT_DIR"},{"line_number":446,"context_line":"}"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"# install_keystone() - Collect source and prepare"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"dab895d6_ee4d23c6","line":445,"updated":"2014-06-29 15:38:18.000000000","message":"also, this should be KEYSTONEMIDDLEWARE_DIR\n\nSince I was able to run this just fine, not sure what this does.","commit_id":"9dd90281756b1a7e818bf4ef69c17806aec5d2c2"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"819808f7480191bc21c4a7a7a60dfc9ccd8fdf5e","unresolved":false,"context_lines":[{"line_number":442,"context_line":"# install_keystonemiddleware() - Collect source and prepare"},{"line_number":443,"context_line":"function install_keystonemiddleware {"},{"line_number":444,"context_line":"    git_clone $KEYSTONEMIDDLEWARE_REPO $KEYSTONEMIDDLEWARE_DIR $KEYSTONEMIDDLEWARE_BRANCH"},{"line_number":445,"context_line":"    setup_develop $KEYSTONECLIENT_DIR"},{"line_number":446,"context_line":"}"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"# install_keystone() - Collect source and prepare"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"dab895d6_c2a2f399","line":445,"in_reply_to":"dab895d6_ee4d23c6","updated":"2014-06-29 17:04:39.000000000","message":"++ will fix in next patch (both comments)","commit_id":"9dd90281756b1a7e818bf4ef69c17806aec5d2c2"}],"stack.sh":[{"author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"change_message_id":"de08d81a5f58e84289abd7116209733bb28cd405","unresolved":false,"context_lines":[{"line_number":723,"context_line":"fi"},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"# Install middleware"},{"line_number":726,"context_line":"install_keystonemiddleware"},{"line_number":727,"context_line":""},{"line_number":728,"context_line":"git_clone $OPENSTACKCLIENT_REPO $OPENSTACKCLIENT_DIR $OPENSTACKCLIENT_BRANCH"},{"line_number":729,"context_line":"setup_develop $OPENSTACKCLIENT_DIR"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baada198_e5e7dde2","line":726,"updated":"2014-07-01 14:48:58.000000000","message":"shouldn\u0027t be only when keystone installed? (on line 731 we do make that difference)","commit_id":"58936fdbcda70b28e14dfe9a5fe344fc1f22c40e"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"17999168615bb6a5c50d2d5545000ecc66f961d3","unresolved":false,"context_lines":[{"line_number":723,"context_line":"fi"},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"# Install middleware"},{"line_number":726,"context_line":"install_keystonemiddleware"},{"line_number":727,"context_line":""},{"line_number":728,"context_line":"git_clone $OPENSTACKCLIENT_REPO $OPENSTACKCLIENT_DIR $OPENSTACKCLIENT_BRANCH"},{"line_number":729,"context_line":"setup_develop $OPENSTACKCLIENT_DIR"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baada198_45d889ef","line":726,"in_reply_to":"baada198_e5e7dde2","updated":"2014-07-01 14:53:21.000000000","message":"I thought about that, but this was previously installed with keystoneclient so always installing it here is not a change in behaviour.  In a multi-node config we would need to install this on all nodes anyway so it would need a separate \u0027service\u0027 name...","commit_id":"58936fdbcda70b28e14dfe9a5fe344fc1f22c40e"}]}