)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"41a31ceb671f7480536b3a4288c529e5e71ba2ee","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Additionally, we do an override in the gate to reduce this value from"},{"line_number":13,"context_line":"the default (600 seconds) to 300."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I3b2a833a3e96a3b5deac76052eed480ddcb6175d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a68dd71_0f75c062","line":14,"updated":"2016-01-22 15:15:16.000000000","message":"In the absence of a release note, please add \u0027UpgradeImpact\u0027 as a tag.","commit_id":"ca5b765bb7bd442d9569a13a6ba4cbf4b684bea7"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"1f6d34ea10fa16df9a30950d0bff1872d4a4d84f","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For some additional information (as provided by dolphm), it looks like"},{"line_number":27,"context_line":"endpoint updates invalidate portions of the dogpile cache but not all."},{"line_number":28,"context_line":"It\u0027s unclear at this point in time if there will be an upstream fix for"},{"line_number":29,"context_line":"this or if the recommended solution is to simply keep the"},{"line_number":30,"context_line":"expiration_time low."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"[1] https://review.openstack.org/#/c/215212/"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9a68dd71_721f4108","line":30,"range":{"start_line":28,"start_character":0,"end_line":30,"end_character":20},"updated":"2016-01-25 09:19:39.000000000","message":"This is inaccurate, there will be an upstream fix, either we will revert the caching change or apply the fix associated to the bug #1537617","commit_id":"611d7066d0637e1fafc6a86b37dd6770714f748d"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"301a262d4e49b1c5a580cd07c3b1c81a15e5e779","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For some additional information (as provided by dolphm), it looks like"},{"line_number":27,"context_line":"endpoint updates invalidate portions of the dogpile cache but not all."},{"line_number":28,"context_line":"It\u0027s unclear at this point in time if there will be an upstream fix for"},{"line_number":29,"context_line":"this or if the recommended solution is to simply keep the"},{"line_number":30,"context_line":"expiration_time low."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"[1] https://review.openstack.org/#/c/215212/"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9a68dd71_72f7418b","line":30,"range":{"start_line":28,"start_character":0,"end_line":30,"end_character":20},"in_reply_to":"9a68dd71_721f4108","updated":"2016-01-25 09:24:59.000000000","message":"Apologies -- that was from my initial commit which was written before your bug was submitted.","commit_id":"611d7066d0637e1fafc6a86b37dd6770714f748d"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"1f6d34ea10fa16df9a30950d0bff1872d4a4d84f","unresolved":false,"context_lines":[{"line_number":32,"context_line":"[1] https://review.openstack.org/#/c/215212/"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"UpgradeImpact"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Change-Id: I3b2a833a3e96a3b5deac76052eed480ddcb6175d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9a68dd71_321549e4","line":35,"updated":"2016-01-25 09:19:39.000000000","message":"please add\n\n\"Related-bug: #1537617\"","commit_id":"611d7066d0637e1fafc6a86b37dd6770714f748d"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"301a262d4e49b1c5a580cd07c3b1c81a15e5e779","unresolved":false,"context_lines":[{"line_number":32,"context_line":"[1] https://review.openstack.org/#/c/215212/"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"UpgradeImpact"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Change-Id: I3b2a833a3e96a3b5deac76052eed480ddcb6175d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9a68dd71_f231f15a","line":35,"in_reply_to":"9a68dd71_321549e4","updated":"2016-01-25 09:24:59.000000000","message":"Done","commit_id":"611d7066d0637e1fafc6a86b37dd6770714f748d"}],"scripts/gate-check-commit.sh":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"41a31ceb671f7480536b3a4288c529e5e71ba2ee","unresolved":false,"context_lines":[{"line_number":102,"context_line":"  export BOOTSTRAP_OPTS\u003d\"${BOOTSTRAP_OPTS} bootstrap_host_libvirt_config\u003dyes\""},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"fi"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"export BOOTSTRAP_OPTS\u003d\"${BOOTSTRAP_OPTS} {\u0027openstack_user_config_overrides\u0027: {\u0027keystone_keystone_conf_overrides\u0027: {\u0027cache\u0027: {\u0027expiration_time\u0027: \u0027300\u0027}}}}\""},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"# Bootstrap an AIO"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a68dd71_aff034ff","line":105,"updated":"2016-01-22 15:15:16.000000000","message":"Please add a comment about why this is being done.","commit_id":"ca5b765bb7bd442d9569a13a6ba4cbf4b684bea7"}],"tests/roles/bootstrap-host/templates/user_variables.aio.yml.j2":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"b1b657b1598ccc1257ed2e5bb96904ca1a53829c","unresolved":false,"context_lines":[{"line_number":91,"context_line":"lxc_container_template_security_apt_repo: {{ bootstrap_host_ubuntu_security_repo }}"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"# We need to drop this from the default (10 minutes) as we add endpoints and"},{"line_number":94,"context_line":"# query right after and run the risk of getting stale data from the cache store"},{"line_number":95,"context_line":"keystone_keystone_conf_overrides:"},{"line_number":96,"context_line":"  cache:"},{"line_number":97,"context_line":"    expiration_time: 60"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"9a68dd71_1923a5a8","line":94,"updated":"2016-01-23 18:07:04.000000000","message":"+1 for adding the comment!","commit_id":"98f7276d86aa8f1ac4746fd39112f4d79fdc94e5"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"b1b657b1598ccc1257ed2e5bb96904ca1a53829c","unresolved":false,"context_lines":[{"line_number":94,"context_line":"# query right after and run the risk of getting stale data from the cache store"},{"line_number":95,"context_line":"keystone_keystone_conf_overrides:"},{"line_number":96,"context_line":"  cache:"},{"line_number":97,"context_line":"    expiration_time: 60"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"9a68dd71_7932e1f3","line":97,"updated":"2016-01-23 18:07:04.000000000","message":"I\u0027m inclined to perhaps even reduce this to 10 or 15 seconds for a better guarantee. However it does appear that perhaps this value is achieving the goal (the gate succeeded for the first time in ages).\n\nNote that the failure in the -nv gate check was unrelated - it was a SSH failure. This is OK in the context of the current issue.","commit_id":"98f7276d86aa8f1ac4746fd39112f4d79fdc94e5"}]}
