Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Release/7.0][PERF] Backport urllib fix to release/7.0 #87286

Merged

Conversation

LoopedBard3
Copy link
Member

Backports changes for dotnet/performance#3010 to the release/7.0-staging branch for eventual merge into release/7.0.

…ue to reinstall installing a broken urllib version. (dotnet#85935)" (dotnet#86000)

This reverts commit 4ea93a6 due to the commit seemingly causes major failure in the pipelines.
…rsion (dotnet#86128)

* Move scenario urllib imports to the last install position.

* Remove force-reinstall from all pip installs in the scenarios-job.yml files.
@ghost
Copy link

ghost commented Jun 8, 2023

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Backports changes for dotnet/performance#3010 to the release/7.0-staging branch for eventual merge into release/7.0.

Author: LoopedBard3
Assignees: LoopedBard3
Labels:

area-Infrastructure

Milestone: -

@LoopedBard3 LoopedBard3 changed the title [PERF] Backport urllib fix to release/7.0 [Release/7.0][PERF] Backport urllib fix to release/7.0 Jun 8, 2023
LoopedBard3 added a commit to LoopedBard3/runtime that referenced this pull request Jun 8, 2023
…ing the urllib install based on the release/7.0 backport of the same changes (dotnet#87286).
@LoopedBard3
Copy link
Member Author

@LoopedBard3
Copy link
Member Author

@carlossanlop Can I get this merged? It is a perf infra only change. Thanks!

carlossanlop pushed a commit that referenced this pull request Jun 12, 2023
…ing the urllib install based on the release/7.0 backport of the same changes (#87286). (#87289)
@LoopedBard3
Copy link
Member Author

@carlossanlop Just to make sure there is no confusion, there are 2 PR's doing similar things. This one is for release/7.0 and another for release/6.0 (which has already been merged).

@carlossanlop carlossanlop added the Servicing-approved Approved for servicing release label Jun 12, 2023
@carlossanlop carlossanlop merged commit 27dd02f into dotnet:release/7.0-staging Jun 12, 2023
@LoopedBard3 LoopedBard3 deleted the BackportUrllibFixR7 branch June 12, 2023 21:38
@ghost ghost locked as resolved and limited conversation to collaborators Jul 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants