)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"76362b65c64e0031a24804f70234a0dad6eaf256","unresolved":false,"context_lines":[{"line_number":27,"context_line":"$ENVIRON[\u0027project_id\u0027] and $ENVIRON[\u0027user_id\u0027] values that are supplied"},{"line_number":28,"context_line":"in the PUT /allocations/{consumer_uuid} call. I\u0027m still trying to"},{"line_number":29,"context_line":"determine why this is happening."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Change-Id: I6709d068f109f25b25843b70aa77078ece8941ab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_0958222f","line":30,"updated":"2018-06-29 14:09:50.000000000","message":"Don\u0027t forget to add bug tags","commit_id":"80dac563420953b450240209f07069439dd01e57"}],"nova/api/openstack/placement/handlers/allocation.py":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"a275331d1aec339e4713340563fa891851b11f2a","unresolved":false,"context_lines":[{"line_number":161,"context_line":"    want_version \u003d req.environ[microversion.MICROVERSION_ENVIRON]"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"    # confirm existence of consumer so we get a reasonable 404 instead of empty"},{"line_number":164,"context_line":"    # list"},{"line_number":165,"context_line":"    try:"},{"line_number":166,"context_line":"        consumer \u003d consumer_obj.Consumer.get_by_uuid(context, consumer_id)"},{"line_number":167,"context_line":"    except exception.NotFound as exc:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_29fb9e15","line":164,"updated":"2018-06-29 14:06:53.000000000","message":"Will all old allocations have been migrated to have a consumer by the time this can be called? I forget the process there.","commit_id":"80dac563420953b450240209f07069439dd01e57"}],"nova/tests/functional/api/openstack/placement/gabbits/allocations.yaml":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"76362b65c64e0031a24804f70234a0dad6eaf256","unresolved":false,"context_lines":[{"line_number":466,"context_line":"    openstack-api-version: placement 1.28"},{"line_number":467,"context_line":"  response_json_paths:"},{"line_number":468,"context_line":"      $.consumer_generation: 4"},{"line_number":469,"context_line":"      $.project_id: $ENVIRON[\u0027PROJECT_ID\u0027]"},{"line_number":470,"context_line":"      $.user_id: $ENVIRON[\u0027USER_ID\u0027]"},{"line_number":471,"context_line":"      $.allocations: {}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_29cf1e8c","line":470,"range":{"start_line":469,"start_character":0,"end_line":470,"end_character":36},"updated":"2018-06-29 14:09:50.000000000","message":"In the commit message you mentioned a test returning the default proj/user instead of the correct ones - is it here?","commit_id":"80dac563420953b450240209f07069439dd01e57"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"20a8bd70a3c582b36d10a38adbf6deca93d33521","unresolved":false,"context_lines":[{"line_number":466,"context_line":"    openstack-api-version: placement 1.28"},{"line_number":467,"context_line":"  response_json_paths:"},{"line_number":468,"context_line":"      $.consumer_generation: 4"},{"line_number":469,"context_line":"      $.project_id: $ENVIRON[\u0027PROJECT_ID\u0027]"},{"line_number":470,"context_line":"      $.user_id: $ENVIRON[\u0027USER_ID\u0027]"},{"line_number":471,"context_line":"      $.allocations: {}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_9f9304a5","line":470,"range":{"start_line":469,"start_character":0,"end_line":470,"end_character":36},"in_reply_to":"5f7c97a3_29cf1e8c","updated":"2018-06-29 15:13:03.000000000","message":"yup, this is the one that bombs in the functional tests, returning 0000-0000-00000000-00000000000000 for both user and project ID instead of the $ENVIRON[\u0027PROJECT_ID\u0027] that was PUT\u0027d above.\n\nI think I know why now, though. It\u0027s because this particular consumer was created with an old microversion that doesn\u0027t supply a project or user ID on line 134 above. And the consumer record that was created for it used the incomplete project/user...","commit_id":"80dac563420953b450240209f07069439dd01e57"}]}
