)]}'
{"nova/api/openstack/compute/migrate_server.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"783c30af2fe652fc5b017c61f2d19880d22bbd16","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            msg \u003d _(\"The os-migrateLive action on a server with ports having \""},{"line_number":138,"context_line":"                    \"resource requests, like a port with a QoS minimum \""},{"line_number":139,"context_line":"                    \"bandwidth policy, is not supported until every \""},{"line_number":140,"context_line":"                    \"nova-compute is upgraded to Ussuri\")"},{"line_number":141,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1fa4df85_4a3585d9","line":140,"range":{"start_line":140,"start_character":49,"end_line":140,"end_character":55},"updated":"2020-03-18 16:01:40.000000000","message":"So, I probably didn\u0027t catch this in your earlier patch, but.. isn\u0027t it a bad idea to be be saying things like this out of the API? Backports would make this invalid or confusing, and it also maybe exposes not only some information about the deployment, but un-actionable-by-the-api-user details.\n\nSeems like logging this would be useful, but can\u0027t/shouldn\u0027t we keep the API wording to \"this cluster is unable to support this operation right now\" ?","commit_id":"390d796759ace6d914ffef1091f8825b782126d9"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"c810c7dd5475149cc2bb038a34e823d9dae2a7f7","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            msg \u003d _(\"The os-migrateLive action on a server with ports having \""},{"line_number":138,"context_line":"                    \"resource requests, like a port with a QoS minimum \""},{"line_number":139,"context_line":"                    \"bandwidth policy, is not supported until every \""},{"line_number":140,"context_line":"                    \"nova-compute is upgraded to Ussuri\")"},{"line_number":141,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1fa4df85_304af854","line":140,"range":{"start_line":140,"start_character":49,"end_line":140,"end_character":55},"in_reply_to":"1fa4df85_4a3585d9","updated":"2020-03-18 18:14:14.000000000","message":"Make sense. Done in all three places (evacuate, live migrate, unshelve)","commit_id":"390d796759ace6d914ffef1091f8825b782126d9"}]}