)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5a2341b3c1c712c1ce38e99fb347c9a8e63da82a","unresolved":false,"context_lines":[{"line_number":10,"context_line":"client with the current request context and uses the client to query"},{"line_number":11,"context_line":"neutron ports. Neutron does not return the resource_request of the"},{"line_number":12,"context_line":"neutron port if it is queried by a non-admin. So if the request context"},{"line_number":13,"context_line":"was a non admin context nova thought that non of the ports has resource"},{"line_number":14,"context_line":"request."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch ensures that an admin client is used to query the ports."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_65857207","line":13,"range":{"start_line":13,"start_character":42,"end_line":13,"end_character":45},"updated":"2019-11-06 20:35:45.000000000","message":"none","commit_id":"0108117d6170929cc4dbc67862514eedfeaabbdc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5a2341b3c1c712c1ce38e99fb347c9a8e63da82a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"request."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch ensures that an admin client is used to query the ports."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I1178fb77a74010c3b9f51eea22c7e7576b600015"},{"line_number":19,"context_line":"Closes-Bug: #1849695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_25777a3e","line":17,"updated":"2019-11-06 20:35:45.000000000","message":"Similar to my question in https://review.opendev.org/#/c/690999/ - would it be possible to add a resize wrinkle to the nova-next post_test_script server that is created with a port that has a resource request on it to show this bug and then that it\u0027s fixed with this patch?","commit_id":"0108117d6170929cc4dbc67862514eedfeaabbdc"}],"nova/network/neutronv2/api.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5a2341b3c1c712c1ce38e99fb347c9a8e63da82a","unresolved":false,"context_lines":[{"line_number":2223,"context_line":"                        raise exception.NetworkNotFound(network_id\u003did_str)"},{"line_number":2224,"context_line":"        return ports_needed_per_instance"},{"line_number":2225,"context_line":""},{"line_number":2226,"context_line":"    def get_requested_resource_for_instance(self, context, instance_uuid):"},{"line_number":2227,"context_line":"        \"\"\"Collect resource requests from the ports associated to the instance"},{"line_number":2228,"context_line":""},{"line_number":2229,"context_line":"        :param context: nova request context"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_659a5263","line":2226,"updated":"2019-11-06 20:35:45.000000000","message":"This is now also used during an evacuate flow (starting in Ussuri) but evacuate is admin-only by default.","commit_id":"0108117d6170929cc4dbc67862514eedfeaabbdc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5a2341b3c1c712c1ce38e99fb347c9a8e63da82a","unresolved":false,"context_lines":[{"line_number":2232,"context_line":"        \"\"\""},{"line_number":2233,"context_line":""},{"line_number":2234,"context_line":"        # NOTE(gibi): We need to use an admin client as otherwise a non admin"},{"line_number":2235,"context_line":"        # initiated resize causes that neutron does fill the resource_request"},{"line_number":2236,"context_line":"        # field of the port and this will lead to resource allocation issues."},{"line_number":2237,"context_line":"        # See bug 1849695"},{"line_number":2238,"context_line":"        neutron \u003d get_client(context, admin\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_e5708243","line":2235,"range":{"start_line":2235,"start_character":47,"end_line":2235,"end_character":51},"updated":"2019-11-06 20:35:45.000000000","message":"does not","commit_id":"0108117d6170929cc4dbc67862514eedfeaabbdc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6f91f7c8be9c8fadc1399027cd0585c8602d5354","unresolved":false,"context_lines":[{"line_number":2232,"context_line":"        \"\"\""},{"line_number":2233,"context_line":""},{"line_number":2234,"context_line":"        # NOTE(gibi): We need to use an admin client as otherwise a non admin"},{"line_number":2235,"context_line":"        # initiated resize causes that neutron does fill the resource_request"},{"line_number":2236,"context_line":"        # field of the port and this will lead to resource allocation issues."},{"line_number":2237,"context_line":"        # See bug 1849695"},{"line_number":2238,"context_line":"        neutron \u003d get_client(context, admin\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_45e396d9","line":2235,"range":{"start_line":2235,"start_character":47,"end_line":2235,"end_character":51},"in_reply_to":"3fa7e38b_e5708243","updated":"2019-11-06 20:39:48.000000000","message":"Done","commit_id":"0108117d6170929cc4dbc67862514eedfeaabbdc"}]}