)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"372ccd629517746c0c0b5e995447451e28bf932c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ffe62b97_a2fefeab","line":14,"updated":"2017-03-23 19:51:25.000000000","message":"May you add:\n\nDepends-On: Id660240c3e9c90b2845f62015d46b474cb00d146\n\nIt basically means that this change depends on the commit in project-config getting merged before this does. That\u0027s because that commit in project-config is needed before we can make sure that this change works.","commit_id":"af83dceb69b91f21a47ddecb4944af02cb6723f2"}],"contrib/post_test_hook.sh":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"372ccd629517746c0c0b5e995447451e28bf932c","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"# Import devstack function \u0027iniset\u0027."},{"line_number":36,"context_line":"source $BASE/new/devstack/functions"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"# Use uuid tokens for faster test runs"},{"line_number":39,"context_line":"KEYSTONE_CONF\u003d/etc/keystone/keystone.conf"},{"line_number":40,"context_line":"iniset $KEYSTONE_CONF token provider uuid"},{"line_number":41,"context_line":"sudo service apache2 restart"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"# First argument is expected to contain value equal either to \u0027admin\u0027 or"},{"line_number":44,"context_line":"# \u0027member\u0027 (both lower-case)."}],"source_content_type":"text/x-sh","patch_set":2,"id":"ffe62b97_e26f6690","side":"PARENT","line":41,"range":{"start_line":37,"start_character":0,"end_line":41,"end_character":28},"updated":"2017-03-23 19:51:25.000000000","message":"This shouldn\u0027t have been removed. This is needed.","commit_id":"c0c31404ba001ebaad6afed81045d28dcb7d5410"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"372ccd629517746c0c0b5e995447451e28bf932c","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    RBAC_ROLE\u003d\"Member\""},{"line_number":45,"context_line":"fi"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"# Second argument is expected to contain value to indicate whether it is "},{"line_number":48,"context_line":"# a \"slow test\" gate"},{"line_number":49,"context_line":"TYPE\u003d$2"},{"line_number":50,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"ffe62b97_a2695e8c","line":47,"range":{"start_line":47,"start_character":72,"end_line":47,"end_character":73},"updated":"2017-03-23 19:51:25.000000000","message":"nit: remove extra space.","commit_id":"af83dceb69b91f21a47ddecb4944af02cb6723f2"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"4dccb72dfdf39503c3c2b9762f2e90fdebde739c","unresolved":false,"context_lines":[{"line_number":75,"context_line":"# Note that this change is dependent on Id660240c3e9c90b2845f62015d46b474cb00d146"},{"line_number":76,"context_line":"# The commit in project-config is needed for this change to work"},{"line_number":77,"context_line":"if [[ \"$TYPE\" \u003d\u003d \"fast\" ]]; then"},{"line_number":78,"context_line":"    $TEMPEST_COMMAND -eall-plugin -- $DEVSTACK_GATE_FAST_TEMPEST_REGEX --concurrency\u003d$TEMPEST_CONCURRENCY"},{"line_number":79,"context_line":"else"},{"line_number":80,"context_line":"    $TEMPEST_COMMAND -eall-plugin -- $DEVSTACK_GATE_SLOW_TEMPEST_REGEX --concurrency\u003d$TEMPEST_CONCURRENCY"},{"line_number":81,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":6,"id":"ffe62b97_7359c658","line":78,"range":{"start_line":78,"start_character":38,"end_line":78,"end_character":70},"updated":"2017-03-24 15:08:33.000000000","message":"Should be DEVSTACK_FAST_GATE_TEMPEST_REGEX","commit_id":"d8e77f3dcec621456cb10acd8833692e47f6c792"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"4dccb72dfdf39503c3c2b9762f2e90fdebde739c","unresolved":false,"context_lines":[{"line_number":77,"context_line":"if [[ \"$TYPE\" \u003d\u003d \"fast\" ]]; then"},{"line_number":78,"context_line":"    $TEMPEST_COMMAND -eall-plugin -- $DEVSTACK_GATE_FAST_TEMPEST_REGEX --concurrency\u003d$TEMPEST_CONCURRENCY"},{"line_number":79,"context_line":"else"},{"line_number":80,"context_line":"    $TEMPEST_COMMAND -eall-plugin -- $DEVSTACK_GATE_SLOW_TEMPEST_REGEX --concurrency\u003d$TEMPEST_CONCURRENCY"},{"line_number":81,"context_line":"fi"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"sudo -H -u tempest .tox/all-plugin/bin/tempest list-plugins"}],"source_content_type":"text/x-sh","patch_set":6,"id":"ffe62b97_f33a76e5","line":80,"range":{"start_line":80,"start_character":38,"end_line":80,"end_character":70},"updated":"2017-03-24 15:08:33.000000000","message":"Should be DEVSTACK_SLOW_GATE_TEMPEST_REGEX","commit_id":"d8e77f3dcec621456cb10acd8833692e47f6c792"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"592ef896b9855f8f6870dd104e69b899eabf195c","unresolved":false,"context_lines":[{"line_number":72,"context_line":"cd $BASE/new/tempest"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"# Select Fast Gate if Type is set to \u0027fast\u0027, else use \u0027slow\u0027 gate"},{"line_number":75,"context_line":"# Note that this change is dependent on Id660240c3e9c90b2845f62015d46b474cb00d146"},{"line_number":76,"context_line":"# The commit in project-config is needed for this change to work"},{"line_number":77,"context_line":"if [[ \"$TYPE\" \u003d\u003d \"fast\" ]]; then"},{"line_number":78,"context_line":"    $TEMPEST_COMMAND -eall-plugin -- $DEVSTACK_FAST_GATE_TEMPEST_REGEX --concurrency\u003d$TEMPEST_CONCURRENCY"},{"line_number":79,"context_line":"else"}],"source_content_type":"text/x-sh","patch_set":7,"id":"ffe62b97_3bc5543d","line":76,"range":{"start_line":75,"start_character":0,"end_line":76,"end_character":64},"updated":"2017-03-24 16:05:54.000000000","message":"This comment should be placed in the commit message, not here. However, there is a correct way to do this in the commit message:\n\nAt the bottom of the commit message, add:\n\nDepends-On: Id660240c3e9c90b2845f62015d46b474cb00d146\n\nHowever, since the infra commit has already been merged, this is no longer needed. Just remove this comment.","commit_id":"e7398136d46cebfd74a2f184d1435496e6526495"}]}