)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"df4caf65fbc5ffc005524a3347a89fa8857acc36","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I7ac4744927395e300205e0f2283b7f2828f431d6"},{"line_number":10,"context_line":"Depends-On: I1b257cfe7c6aec7f999824a1510b4edc63167e80"},{"line_number":11,"context_line":"Depends-On: I1adeb20f261004a24d3f28723ccbf84699c53a19"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"ba5201f7_a9f540cc","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":53},"updated":"2017-01-09 17:41:09.000000000","message":"Why you need this?","commit_id":"bb08d6299aadc828fd970cf2e110117f9c788920"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"0e8a0b8f296bfedce71cccdb1adf1e76f17bbd33","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I7ac4744927395e300205e0f2283b7f2828f431d6"},{"line_number":10,"context_line":"Depends-On: I1b257cfe7c6aec7f999824a1510b4edc63167e80"},{"line_number":11,"context_line":"Depends-On: I1adeb20f261004a24d3f28723ccbf84699c53a19"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"ba5201f7_7f406636","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":53},"in_reply_to":"ba5201f7_a9f540cc","updated":"2017-01-09 18:28:13.000000000","message":"This is needed to skip flavor creation when only n-api is enabled.  Without this there would be failure because there is no keystone, mysql, rabbit on api node and it tries to create flavors.","commit_id":"bb08d6299aadc828fd970cf2e110117f9c788920"}],"nova/tests/zero_downtime/hooks/run_tests.sh":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5e3589924642f4016fe673f3d2f4a72845635272","unresolved":false,"context_lines":[{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"},{"line_number":7,"context_line":"cd $BASE/new/tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"source $BASE/new/nova/nova/tests/live_migration/hooks/utils.sh"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"primary_node\u003d$(cat /etc/nodepool/primary_node_private)"}],"source_content_type":"text/x-sh","patch_set":4,"id":"3a71b18c_c1cedc82","line":8,"updated":"2016-12-01 17:43:47.000000000","message":"I think you need to add:\n\nsource $BASE/new/devstack/functions\nsource $BASE/new/devstack/functions-common\nsource $WORKSPACE/devstack-gate/functions.sh","commit_id":"5cc62665431ec957a865133b2d47e7288b3302be"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"367df38b920ffd2c9c3d5d12b64c7caa83779376","unresolved":false,"context_lines":[{"line_number":22,"context_line":"update_endpoint compute admin $old_url $new_url"},{"line_number":23,"context_line":"update_endpoint compute internal $old_url $new_url"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"echo_summary \"Testing compute api endpoint\""},{"line_number":26,"context_line":"openstack catalog list"},{"line_number":27,"context_line":"nova list"},{"line_number":28,"context_line":"nova host-list"}],"source_content_type":"text/x-sh","patch_set":19,"id":"ba5201f7_00ec248e","line":25,"updated":"2017-01-03 19:14:29.000000000","message":"nit: Technically, L26 only tests keystone.","commit_id":"f3739269798587b9f780fb89f22e387cf3f83341"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"2042ef76786c78bda71f59b02201cd4bbefc8ea1","unresolved":false,"context_lines":[{"line_number":22,"context_line":"update_endpoint compute admin $old_url $new_url"},{"line_number":23,"context_line":"update_endpoint compute internal $old_url $new_url"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"echo_summary \"Testing compute api endpoint\""},{"line_number":26,"context_line":"openstack catalog list"},{"line_number":27,"context_line":"nova list"},{"line_number":28,"context_line":"nova host-list"}],"source_content_type":"text/x-sh","patch_set":19,"id":"ba5201f7_89d03f55","line":25,"in_reply_to":"ba5201f7_00ec248e","updated":"2017-01-03 20:27:15.000000000","message":"Dolph.. I added these just for my testing. Removing them.","commit_id":"f3739269798587b9f780fb89f22e387cf3f83341"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"367df38b920ffd2c9c3d5d12b64c7caa83779376","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"echo_summary \"Running all tempest tests with old api\""},{"line_number":34,"context_line":"cd $BASE/new/tempest"},{"line_number":35,"context_line":"tox -esmoke -- --concurrency\u003d$TEMPEST_CONCURRENCY"}],"source_content_type":"text/x-sh","patch_set":19,"id":"ba5201f7_60fdf039","line":35,"updated":"2017-01-03 19:14:29.000000000","message":"Missing newline at EOF","commit_id":"f3739269798587b9f780fb89f22e387cf3f83341"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"b7b6e9fd50a86cb9db95bc522bc80db7c82e7b8c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Change nova api endpoint to old api and run"},{"line_number":3,"context_line":"# tempest tests in multi api environment."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"}],"source_content_type":"text/x-sh","patch_set":28,"id":"7a3c09a3_d48157c4","line":3,"range":{"start_line":2,"start_character":2,"end_line":3,"end_character":41},"updated":"2017-01-18 18:00:37.000000000","message":"Maybe little more description here? ITs just running tests on old api, how that suffice zero-downtime testing?","commit_id":"37753a514b607494d8fd1587c5e7b788556d0505"},{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"367c94f68321f3735aee5b0b57278dd1960d8aaf","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Change nova api endpoint to old api and run"},{"line_number":3,"context_line":"# tempest tests in multi api environment."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a461143_aa48c6cd","line":3,"in_reply_to":"3a461143_81708c0c","updated":"2017-02-02 04:35:14.000000000","message":"Here\u0027s the first stab at it: https://review.openstack.org/#/c/427529/","commit_id":"37753a514b607494d8fd1587c5e7b788556d0505"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"1edc8021c5eae5e993fb4614b97947666df8b472","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Change nova api endpoint to old api and run"},{"line_number":3,"context_line":"# tempest tests in multi api environment."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a461143_81708c0c","line":3,"in_reply_to":"3a461143_a59894c1","updated":"2017-01-31 19:07:10.000000000","message":"Steve,\nWe should also come up with a process that is similar to what you have mentioned. Do you mind linking me to your patches/changes.\nThanks.","commit_id":"37753a514b607494d8fd1587c5e7b788556d0505"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"03e3e794b668c3c12db97c623767f32ba2a3d8b9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Change nova api endpoint to old api and run"},{"line_number":3,"context_line":"# tempest tests in multi api environment."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a461143_3efd356e","line":3,"in_reply_to":"3a461143_fc2f9985","updated":"2017-01-31 18:39:59.000000000","message":"Dolph,\nThis script is not enough to assert zero-downtime. Sorry for the confusion here. I will update the commit message on what this script does exactly.","commit_id":"37753a514b607494d8fd1587c5e7b788556d0505"},{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"97fc35290b02c679b9e20f6bc12e12051729c7b0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Change nova api endpoint to old api and run"},{"line_number":3,"context_line":"# tempest tests in multi api environment."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a461143_a59894c1","line":3,"in_reply_to":"3a461143_fc2f9985","updated":"2017-01-27 22:52:42.000000000","message":"We are looking at something similar for grenade to test zero downtime - basically looping curl calls to an API throughout the upgrade process to ensure that the API remains functional (and eventually performant as well).","commit_id":"37753a514b607494d8fd1587c5e7b788556d0505"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"63d4972fdc11ae5d8731d585eb0efcde2b9b5a98","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Change nova api endpoint to old api and run"},{"line_number":3,"context_line":"# tempest tests in multi api environment."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"set -xe"},{"line_number":6,"context_line":"BASE\u003d/opt/stack"}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a461143_fc2f9985","line":3,"in_reply_to":"7a3c09a3_d48157c4","updated":"2017-01-27 20:55:13.000000000","message":"In order to test zero downtime, you basically need to constantly smoke test (or at the very least, ping) the service *while* it\u0027s being upgraded, for the entire duration of the upgrade process.\n\nIf I\u0027m reading this correctly, then it\u0027s explicitly testing that exactly one of the endpoints is operating correctly during some midpoint  during the upgrade? If so, that\u0027s probably sufficient to test rolling upgrades (assuming the other endpoint is equally exercised already), but not zero-downtime.","commit_id":"37753a514b607494d8fd1587c5e7b788556d0505"}],"requirements.txt":[{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"df4caf65fbc5ffc005524a3347a89fa8857acc36","unresolved":false,"context_lines":[{"line_number":57,"context_line":"os-win\u003e\u003d1.1.0 # Apache-2.0"},{"line_number":58,"context_line":"castellan\u003e\u003d0.4.0 # Apache-2.0"},{"line_number":59,"context_line":"microversion-parse\u003e\u003d0.1.2 # Apache-2.0"},{"line_number":60,"context_line":"#test"}],"source_content_type":"text/plain","patch_set":24,"id":"ba5201f7_692a782e","line":60,"range":{"start_line":60,"start_character":0,"end_line":60,"end_character":5},"updated":"2017-01-09 17:41:09.000000000","message":"Drop this?","commit_id":"bb08d6299aadc828fd970cf2e110117f9c788920"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"0e8a0b8f296bfedce71cccdb1adf1e76f17bbd33","unresolved":false,"context_lines":[{"line_number":57,"context_line":"os-win\u003e\u003d1.1.0 # Apache-2.0"},{"line_number":58,"context_line":"castellan\u003e\u003d0.4.0 # Apache-2.0"},{"line_number":59,"context_line":"microversion-parse\u003e\u003d0.1.2 # Apache-2.0"},{"line_number":60,"context_line":"#test"}],"source_content_type":"text/plain","patch_set":24,"id":"ba5201f7_df5eb211","line":60,"range":{"start_line":60,"start_character":0,"end_line":60,"end_character":5},"in_reply_to":"ba5201f7_692a782e","updated":"2017-01-09 18:28:13.000000000","message":"Added this just to run the newly introduced experimental gate job.  Changing only nova/tests/zero_downtime/hooks/run_tests.sh doesn\u0027t run the gate job.","commit_id":"bb08d6299aadc828fd970cf2e110117f9c788920"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"71a0646869255eca31b23da2c20294d45f1f48ec","unresolved":false,"context_lines":[{"line_number":57,"context_line":"os-win\u003e\u003d1.1.0 # Apache-2.0"},{"line_number":58,"context_line":"castellan\u003e\u003d0.4.0 # Apache-2.0"},{"line_number":59,"context_line":"microversion-parse\u003e\u003d0.1.2 # Apache-2.0"},{"line_number":60,"context_line":"#test"}],"source_content_type":"text/plain","patch_set":24,"id":"ba5201f7_f2a9e425","line":60,"range":{"start_line":60,"start_character":0,"end_line":60,"end_character":5},"in_reply_to":"ba5201f7_aeb8de2b","updated":"2017-01-09 22:21:52.000000000","message":"We are testing experimental gate job. So we need to comment \u0027check experimental\u0027 to run the new gate. But you are right, this change will not be accepted upstreams. I will remove this change here and test the new gate job in a separate patch.","commit_id":"bb08d6299aadc828fd970cf2e110117f9c788920"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"7a012d235e9f7ffd8dae6026a5e0abdfe215ac10","unresolved":false,"context_lines":[{"line_number":57,"context_line":"os-win\u003e\u003d1.1.0 # Apache-2.0"},{"line_number":58,"context_line":"castellan\u003e\u003d0.4.0 # Apache-2.0"},{"line_number":59,"context_line":"microversion-parse\u003e\u003d0.1.2 # Apache-2.0"},{"line_number":60,"context_line":"#test"}],"source_content_type":"text/plain","patch_set":24,"id":"ba5201f7_aeb8de2b","line":60,"range":{"start_line":60,"start_character":0,"end_line":60,"end_character":5},"in_reply_to":"ba5201f7_df5eb211","updated":"2017-01-09 20:33:30.000000000","message":"So this will not be accepted upstreams, can\u0027t you run gate jobs manually just by \"recheck\"?","commit_id":"bb08d6299aadc828fd970cf2e110117f9c788920"}]}