)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b110e5bfce889961e0ff1478401947f39d33b4f1","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Improve both the \u0027--user\u0027 and \u0027--project\u0027 options to allow names as well"},{"line_number":10,"context_line":"as UUIDs. There\u0027s no release note included since this feature hasn\u0027t"},{"line_number":11,"context_line":"been released yet."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7654f3ffc54d38d5cfb03d8d1b2f4dc4fb06fb3d"},{"line_number":14,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"f20f37b1_1812ad76","line":12,"updated":"2021-01-12 15:25:08.000000000","message":"should this not have a depends on then on the patch that adds it?\n\nif this is a client side only feature it should have a release note\nit really should have a release note in either case right?","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"470ee4a5364bba50919cf07a9797710eca3557d9","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Improve both the \u0027--user\u0027 and \u0027--project\u0027 options to allow names as well"},{"line_number":10,"context_line":"as UUIDs. There\u0027s no release note included since this feature hasn\u0027t"},{"line_number":11,"context_line":"been released yet."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7654f3ffc54d38d5cfb03d8d1b2f4dc4fb06fb3d"},{"line_number":14,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"f8f9da1c_cd3d663e","line":12,"in_reply_to":"f20f37b1_1812ad76","updated":"2021-01-12 16:16:46.000000000","message":"Took me a minute to understand what you were getting at 😊 What I was suggesting is that the \u0027server migration list\u0027 command was only added to OSC recently and hasn\u0027t been released yet, which means I don\u0027t need to add a release note saying e.g.\n\n  The \u0027--user\u0027 and \u0027--project\u0027 options of the \u0027server migration list\u0027 command now\n  accept a user and project name, respectively, in addition to an ID.\n\nIf we did add a release note, we\u0027d have one release note saying we\u0027ve added this command and another saying we\u0027ve extended it, which may be in the wrong order and would generally be confusing.\n\nDoes that makes sense?","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"80e6638143062f42c45f6c61d22372fa38294cc8","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Improve both the \u0027--user\u0027 and \u0027--project\u0027 options to allow names as well"},{"line_number":10,"context_line":"as UUIDs. There\u0027s no release note included since this feature hasn\u0027t"},{"line_number":11,"context_line":"been released yet."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7654f3ffc54d38d5cfb03d8d1b2f4dc4fb06fb3d"},{"line_number":14,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"d5528e2d_d1a34071","line":12,"in_reply_to":"f8f9da1c_cd3d663e","updated":"2021-01-12 17:24:23.000000000","message":"yes it does.","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"80e6638143062f42c45f6c61d22372fa38294cc8","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"compute: Improve \u0027server migration list\u0027 options"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Improve both the \u0027--user\u0027 and \u0027--project\u0027 options to allow names as well"},{"line_number":10,"context_line":"as UUIDs. There\u0027s no release note included since this entire command was"},{"line_number":11,"context_line":"added in change I15b4a5aca8d0dee59dd293e7b1c7272cdfbeea20, which hasn\u0027t"},{"line_number":12,"context_line":"been included in a release yet."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I7654f3ffc54d38d5cfb03d8d1b2f4dc4fb06fb3d"},{"line_number":15,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"448b7f7d_9fdb688e","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":31},"updated":"2021-01-12 17:24:23.000000000","message":"+1 this is clearer thanks for updating","commit_id":"dfa869ed1dbd54c76a9d7cd7d520f0d21064918d"}],"openstackclient/compute/v2/server.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b110e5bfce889961e0ff1478401947f39d33b4f1","unresolved":true,"context_lines":[{"line_number":2476,"context_line":"                )"},{"line_number":2477,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":2478,"context_line":""},{"line_number":2479,"context_line":"            search_opts[\u0027project_id\u0027] \u003d identity_common.find_project("},{"line_number":2480,"context_line":"                identity_client,"},{"line_number":2481,"context_line":"                parsed_args.project,"},{"line_number":2482,"context_line":"                parsed_args.project_domain,"}],"source_content_type":"text/x-python","patch_set":8,"id":"8e0539c8_b9a403c6","line":2479,"range":{"start_line":2479,"start_character":56,"end_line":2479,"end_character":68},"updated":"2021-01-12 15:25:08.000000000","message":"so this function will accpet either the name or id and do the translation to an id?","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"470ee4a5364bba50919cf07a9797710eca3557d9","unresolved":false,"context_lines":[{"line_number":2476,"context_line":"                )"},{"line_number":2477,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":2478,"context_line":""},{"line_number":2479,"context_line":"            search_opts[\u0027project_id\u0027] \u003d identity_common.find_project("},{"line_number":2480,"context_line":"                identity_client,"},{"line_number":2481,"context_line":"                parsed_args.project,"},{"line_number":2482,"context_line":"                parsed_args.project_domain,"}],"source_content_type":"text/x-python","patch_set":8,"id":"dc57a9f2_b7b5eb39","line":2479,"range":{"start_line":2479,"start_character":56,"end_line":2479,"end_character":68},"in_reply_to":"8e0539c8_b9a403c6","updated":"2021-01-12 16:16:46.000000000","message":"Yes. If you search through this file you\u0027ll find plenty of references to this function. I was just integrating it here.","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b110e5bfce889961e0ff1478401947f39d33b4f1","unresolved":true,"context_lines":[{"line_number":2490,"context_line":"                )"},{"line_number":2491,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":2492,"context_line":""},{"line_number":2493,"context_line":"            search_opts[\u0027user_id\u0027] \u003d identity_common.find_user("},{"line_number":2494,"context_line":"                identity_client,"},{"line_number":2495,"context_line":"                parsed_args.user,"},{"line_number":2496,"context_line":"                parsed_args.user_domain,"}],"source_content_type":"text/x-python","patch_set":8,"id":"91adebe1_e5da14bf","line":2493,"range":{"start_line":2493,"start_character":52,"end_line":2493,"end_character":62},"updated":"2021-01-12 15:25:08.000000000","message":"same for user","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"470ee4a5364bba50919cf07a9797710eca3557d9","unresolved":false,"context_lines":[{"line_number":2490,"context_line":"                )"},{"line_number":2491,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":2492,"context_line":""},{"line_number":2493,"context_line":"            search_opts[\u0027user_id\u0027] \u003d identity_common.find_user("},{"line_number":2494,"context_line":"                identity_client,"},{"line_number":2495,"context_line":"                parsed_args.user,"},{"line_number":2496,"context_line":"                parsed_args.user_domain,"}],"source_content_type":"text/x-python","patch_set":8,"id":"0da022c9_8d4fe13a","line":2493,"range":{"start_line":2493,"start_character":52,"end_line":2493,"end_character":62},"in_reply_to":"91adebe1_e5da14bf","updated":"2021-01-12 16:16:46.000000000","message":"As above.","commit_id":"c254c2eae68aada0bbe91c7dd27489f0dbf9f19c"}]}
