)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"c2675e101e1c1d1b3334746cbe1725fbf1e1badf","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Updating devstack.localrc.example for accuracy"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The devstack.localrx.example file is used by users trying"},{"line_number":10,"context_line":"to figure out how to setup Grenade to run their upgrade testing"},{"line_number":11,"context_line":"but the file seems to be very outdated since it contains some"},{"line_number":12,"context_line":"references to juno and kilo. Also it considers as if Heat is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a3905b3_65cd415d","line":9,"updated":"2017-01-24 22:05:34.000000000","message":"should be localrc not localrx","commit_id":"9cdb18ad23cc14658398d4b243a9841f36a1ee0f"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"ee71a5d3e4678f397904bb106fab68e3e16e5fa5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Updating devstack.localrc.example for accuracy"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The devstack.localrx.example file is used by users trying"},{"line_number":10,"context_line":"to figure out how to setup Grenade to run their upgrade testing"},{"line_number":11,"context_line":"but the file seems to be very outdated since it contains some"},{"line_number":12,"context_line":"references to juno and kilo. Also it considers as if Heat is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a3905b3_60360fe9","line":9,"in_reply_to":"5a3905b3_65cd415d","updated":"2017-01-24 22:12:47.000000000","message":"Done","commit_id":"9cdb18ad23cc14658398d4b243a9841f36a1ee0f"},{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"c2675e101e1c1d1b3334746cbe1725fbf1e1badf","unresolved":false,"context_lines":[{"line_number":12,"context_line":"references to juno and kilo. Also it considers as if Heat is"},{"line_number":13,"context_line":"on by default and Neutron is not, which is no longer true."},{"line_number":14,"context_line":"This commit attempts to update the example file so it reflects"},{"line_number":15,"context_line":"more accurately the wasy devstack works currently, and at"},{"line_number":16,"context_line":"the same time tries to be a little bit more explanatory for"},{"line_number":17,"context_line":"new users trying to discover Grenade."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a3905b3_45f7658c","line":15,"updated":"2017-01-24 22:05:34.000000000","message":"should be way not wasy","commit_id":"9cdb18ad23cc14658398d4b243a9841f36a1ee0f"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"ee71a5d3e4678f397904bb106fab68e3e16e5fa5","unresolved":false,"context_lines":[{"line_number":12,"context_line":"references to juno and kilo. Also it considers as if Heat is"},{"line_number":13,"context_line":"on by default and Neutron is not, which is no longer true."},{"line_number":14,"context_line":"This commit attempts to update the example file so it reflects"},{"line_number":15,"context_line":"more accurately the wasy devstack works currently, and at"},{"line_number":16,"context_line":"the same time tries to be a little bit more explanatory for"},{"line_number":17,"context_line":"new users trying to discover Grenade."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a3905b3_c04b435e","line":15,"in_reply_to":"5a3905b3_45f7658c","updated":"2017-01-24 22:12:47.000000000","message":"Done","commit_id":"9cdb18ad23cc14658398d4b243a9841f36a1ee0f"}],"devstack.localrc.example":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"ae6e5f1caacacf65a86f01db34ab4f2ea14cdff2","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# if you already have MYSQL and RABBIT setup on this system"},{"line_number":7,"context_line":"# from previous runs, uncomment the following lines with the"},{"line_number":8,"context_line":"# appropriate passwords for override."},{"line_number":9,"context_line":"#MYSQL_PASSWORD\u003dpass"},{"line_number":10,"context_line":"#RABBIT_PASSWORD\u003dpass"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"# Replication of gate configuration. Below helps you get to a"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5a3905b3_1b021e70","line":9,"range":{"start_line":9,"start_character":1,"end_line":9,"end_character":20},"updated":"2017-01-24 22:53:57.000000000","message":"Rename mysql_password to database_password since it was updated on Change-Id: Iafb295a0b7707e08a81e6528620db8543d40f7ae","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"4348e3c2cb8c7e5f31b9a8f8a9178d53c34810a1","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# if you already have MYSQL and RABBIT setup on this system"},{"line_number":7,"context_line":"# from previous runs, uncomment the following lines with the"},{"line_number":8,"context_line":"# appropriate passwords for override."},{"line_number":9,"context_line":"#MYSQL_PASSWORD\u003dpass"},{"line_number":10,"context_line":"#RABBIT_PASSWORD\u003dpass"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"# Replication of gate configuration. Below helps you get to a"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"3a461143_3e88a7e7","line":9,"range":{"start_line":9,"start_character":1,"end_line":9,"end_character":20},"in_reply_to":"5a3905b3_1b021e70","updated":"2017-01-25 18:39:13.000000000","message":"Done","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"ae6e5f1caacacf65a86f01db34ab4f2ea14cdff2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# configuration that looks a lot like what we use in the gate"},{"line_number":14,"context_line":"USE_SCREEN\u003dFalse"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# If you need specific interfaces for nova network, specify them here"},{"line_number":17,"context_line":"#FLAT_INTERFACE\u003deth1"},{"line_number":18,"context_line":"#VLAN_INTERFACE\u003deth1"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# Neutron needs a lot of manual hand holding to get rolling, we"},{"line_number":21,"context_line":"# specify a lot of config in the gate in neutron jobs related to"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"5a3905b3_fb6b92ef","line":18,"range":{"start_line":16,"start_character":0,"end_line":18,"end_character":20},"updated":"2017-01-24 22:53:57.000000000","message":"Not sure about these keys, if removed, README.rst should be updated too.","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"4348e3c2cb8c7e5f31b9a8f8a9178d53c34810a1","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# configuration that looks a lot like what we use in the gate"},{"line_number":14,"context_line":"USE_SCREEN\u003dFalse"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# If you need specific interfaces for nova network, specify them here"},{"line_number":17,"context_line":"#FLAT_INTERFACE\u003deth1"},{"line_number":18,"context_line":"#VLAN_INTERFACE\u003deth1"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# Neutron needs a lot of manual hand holding to get rolling, we"},{"line_number":21,"context_line":"# specify a lot of config in the gate in neutron jobs related to"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"3a461143_fec0dfe2","line":18,"range":{"start_line":16,"start_character":0,"end_line":18,"end_character":20},"in_reply_to":"5a3905b3_fb6b92ef","updated":"2017-01-25 18:39:13.000000000","message":"I couldn\u0027t find references to FLAT_INTERFACE in devstack anymore, but there are references to VLAN_INTERFACE, so I will only remove FLAT_INTERFACE.","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"a3bc5b8d05cb4c56bbce0cadf7c2eb406379e842","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Keystone in stable/juno defaults to eventlet, you need to also do"},{"line_number":20,"context_line":"# eventlet in kilo otherwise the upgrade does not detect the switch."},{"line_number":21,"context_line":"KEYSTONE_USE_MOD_WSGI\u003dFalse"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# Speed things up by tuning down screen sleep time"},{"line_number":24,"context_line":"SCREEN_SLEEP\u003d0.5"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3a461143_2a7113e1","side":"PARENT","line":21,"updated":"2017-01-26 16:20:34.000000000","message":"variable is used on https://github.com/openstack-dev/grenade/blob/b69f4851950d2e6aa66617cf5b751fb8aa1c1324/projects/10_keystone/upgrade.sh#L71\nbefore deleting variable it should be deprecated (1 cycle backwards compatible).","commit_id":"24a2dccc71b68b47d75ede54b253b5061894d0bb"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"2c3ac9b2732b765f759472ecbbd0f5bd20f7d8df","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Keystone in stable/juno defaults to eventlet, you need to also do"},{"line_number":20,"context_line":"# eventlet in kilo otherwise the upgrade does not detect the switch."},{"line_number":21,"context_line":"KEYSTONE_USE_MOD_WSGI\u003dFalse"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# Speed things up by tuning down screen sleep time"},{"line_number":24,"context_line":"SCREEN_SLEEP\u003d0.5"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3a461143_6baf9ae3","side":"PARENT","line":21,"in_reply_to":"3a461143_2a7113e1","updated":"2017-02-02 22:39:36.000000000","message":"I\u0027m not saying this variable is not valid anymore, but since it is not the default behavior I don\u0027t think it belongs to the example file, since users tend to grab the example as a base for their own configuration. As devstack works now, if the user just leaves this variable, it will cause Grenade to fail (unless the user adds more configuration)","commit_id":"24a2dccc71b68b47d75ede54b253b5061894d0bb"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e48e7d9476cfbf02f4aa934080b100bd16f00455","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Keystone in stable/juno defaults to eventlet, you need to also do"},{"line_number":20,"context_line":"# eventlet in kilo otherwise the upgrade does not detect the switch."},{"line_number":21,"context_line":"KEYSTONE_USE_MOD_WSGI\u003dFalse"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# Speed things up by tuning down screen sleep time"},{"line_number":24,"context_line":"SCREEN_SLEEP\u003d0.5"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9a30ddce_857adaef","side":"PARENT","line":21,"in_reply_to":"3a461143_6baf9ae3","updated":"2017-03-08 19:45:03.000000000","message":"From Grenade\u0027s perspective, I\u0027d support this being removed immediately. Grenade users are still free to use it as long as devstack supports it.","commit_id":"24a2dccc71b68b47d75ede54b253b5061894d0bb"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e48e7d9476cfbf02f4aa934080b100bd16f00455","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# NOTE: Copy this file to the root Grenade directory and "},{"line_number":2,"context_line":"# rename it devstack.localrc for it to work properly."},{"line_number":3,"context_line":"# All the values set up in this file will be appended at the"},{"line_number":4,"context_line":"# bottom of the devstack.localrc.base and devstack.localrc.target files"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9a30ddce_e5b476d2","line":1,"updated":"2017-03-08 19:45:03.000000000","message":"nit: trailing whitespace","commit_id":"d5425b1a04ec16fe4fef969ed7b2d9f0453f9317"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"a3bc5b8d05cb4c56bbce0cadf7c2eb406379e842","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# if you already have a DATABASE and RABBIT setup on this system"},{"line_number":7,"context_line":"# from previous runs, uncomment the following lines with the"},{"line_number":8,"context_line":"# appropriate passwords for override."},{"line_number":9,"context_line":"#DATABASE_PASSWORD\u003dpass"},{"line_number":10,"context_line":"#RABBIT_PASSWORD\u003dpass"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"# Replication of gate configuration. Below helps you get to a"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3a461143_2a867313","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":23},"updated":"2017-01-26 16:20:34.000000000","message":"I noticed MYSQL_PASSWORD is being used on grenade on https://github.com/openstack-dev/grenade/blob/master/inc/upgrade#L35","commit_id":"d5425b1a04ec16fe4fef969ed7b2d9f0453f9317"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"2c3ac9b2732b765f759472ecbbd0f5bd20f7d8df","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# if you already have a DATABASE and RABBIT setup on this system"},{"line_number":7,"context_line":"# from previous runs, uncomment the following lines with the"},{"line_number":8,"context_line":"# appropriate passwords for override."},{"line_number":9,"context_line":"#DATABASE_PASSWORD\u003dpass"},{"line_number":10,"context_line":"#RABBIT_PASSWORD\u003dpass"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"# Replication of gate configuration. Below helps you get to a"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3a461143_2bcaf2a1","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":23},"in_reply_to":"3a461143_2a867313","updated":"2017-02-02 22:39:36.000000000","message":"Same answer, this is just an example file, and the base and target files already include this variable anyway. I think leaving it here is misleading for the user.","commit_id":"d5425b1a04ec16fe4fef969ed7b2d9f0453f9317"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e48e7d9476cfbf02f4aa934080b100bd16f00455","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"# Heat services are not installed by default, you can enable them"},{"line_number":33,"context_line":"# by uncommenting the following line"},{"line_number":34,"context_line":"#enable_service heat h-api h-api-cfn h-api-cw h-eng"},{"line_number":35,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9a30ddce_058b4ac4","line":34,"updated":"2017-03-08 19:45:03.000000000","message":"I\u0027m not sure I see any value in these last two blocks for general grenade users, as they\u0027re both rather specific configurations. If anything, perhaps there should just be a note and reference to devstack docs on what else can be configured here?","commit_id":"d5425b1a04ec16fe4fef969ed7b2d9f0453f9317"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e48e7d9476cfbf02f4aa934080b100bd16f00455","unresolved":false,"context_lines":[{"line_number":32,"context_line":"# Heat services are not installed by default, you can enable them"},{"line_number":33,"context_line":"# by uncommenting the following line"},{"line_number":34,"context_line":"#enable_service heat h-api h-api-cfn h-api-cw h-eng"},{"line_number":35,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"9a30ddce_4550e25d","line":35,"updated":"2017-03-08 19:45:03.000000000","message":"nit: trailing empty line","commit_id":"d5425b1a04ec16fe4fef969ed7b2d9f0453f9317"}],"inc/bootstrap":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"0ccb41ddae3213366ee1016daf874e36bab99c04","unresolved":false,"context_lines":[{"line_number":261,"context_line":"        rsync -a ${STACK_ROOT}/images/* $BASE_DEVSTACK_DIR/files"},{"line_number":262,"context_line":"    fi"},{"line_number":263,"context_line":""},{"line_number":264,"context_line":"    # Set up base localrc"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    # if local.conf exists and local.conf.orig does not exist, save local.conf  to local.conf.orig"},{"line_number":267,"context_line":"    if [[ -r $BASE_DEVSTACK_DIR/local.conf \u0026\u0026 ! -r $BASE_DEVSTACK_DIR/local.conf.orig ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5a3905b3_3b80b946","line":264,"range":{"start_line":264,"start_character":18,"end_line":264,"end_character":25},"updated":"2017-01-24 22:56:47.000000000","message":"change to local.conf","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"4348e3c2cb8c7e5f31b9a8f8a9178d53c34810a1","unresolved":false,"context_lines":[{"line_number":261,"context_line":"        rsync -a ${STACK_ROOT}/images/* $BASE_DEVSTACK_DIR/files"},{"line_number":262,"context_line":"    fi"},{"line_number":263,"context_line":""},{"line_number":264,"context_line":"    # Set up base localrc"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    # if local.conf exists and local.conf.orig does not exist, save local.conf  to local.conf.orig"},{"line_number":267,"context_line":"    if [[ -r $BASE_DEVSTACK_DIR/local.conf \u0026\u0026 ! -r $BASE_DEVSTACK_DIR/local.conf.orig ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3a461143_0130ee55","line":264,"range":{"start_line":264,"start_character":18,"end_line":264,"end_character":25},"in_reply_to":"5a3905b3_3b80b946","updated":"2017-01-25 18:39:13.000000000","message":"Actually I decided to remove the change I had done to this file as part of this commit. I think it would be better to tackle this as part of a different patch, the changes are not related whatsoever. Specially if there are more changes that need to be made to this file.","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"ae6e5f1caacacf65a86f01db34ab4f2ea14cdff2","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        cat $BASE_DEVSTACK_DIR/localrc.orig \u003e\u003e$BASE_DEVSTACK_DIR/local.conf"},{"line_number":287,"context_line":"    fi"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    # if devstack.local.conf exists append it to locarc"},{"line_number":290,"context_line":"    if [[ -r $GRENADE_DIR/devstack.local.conf ]]; then"},{"line_number":291,"context_line":"        echo \"#vvvvvvvvvv devstack.local.conf vvvvvvvvvv\" \u003e\u003e$BASE_DEVSTACK_DIR/local.conf"},{"line_number":292,"context_line":"        cat $GRENADE_DIR/devstack.local.conf \u003e\u003e$BASE_DEVSTACK_DIR/local.conf"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5a3905b3_9b2b6e94","line":289,"range":{"start_line":289,"start_character":49,"end_line":289,"end_character":55},"updated":"2017-01-24 22:53:57.000000000","message":"change to local.conf","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"ae6e5f1caacacf65a86f01db34ab4f2ea14cdff2","unresolved":false,"context_lines":[{"line_number":325,"context_line":"        cat $TARGET_DEVSTACK_DIR/localrc.orig \u003e\u003e$TARGET_DEVSTACK_DIR/local.conf"},{"line_number":326,"context_line":"    fi"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    # if devstack.local.conf exists append it to locarc"},{"line_number":329,"context_line":"    if [[ -r $GRENADE_DIR/devstack.local.conf ]]; then"},{"line_number":330,"context_line":"        echo \"#vvvvvvvvvv devstack.local.conf vvvvvvvvvv\" \u003e\u003e$TARGET_DEVSTACK_DIR/local.conf"},{"line_number":331,"context_line":"        cat $GRENADE_DIR/devstack.local.conf \u003e\u003e$TARGET_DEVSTACK_DIR/local.conf"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5a3905b3_7b05e212","line":328,"range":{"start_line":328,"start_character":49,"end_line":328,"end_character":55},"updated":"2017-01-24 22:53:57.000000000","message":"change to local.conf","commit_id":"c9eb665158da2bb913ec983cf6b12a82f59bc1ba"}]}