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

[build] Fix dependency issue between responses and urllib3 package. #2928

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

liushilongbuaa
Copy link
Contributor

@liushilongbuaa liushilongbuaa commented Jul 27, 2023

What I did

Update urllib3 to V2.

How I did it

Some package depends on urllib3V2.
But urllib3 is pinned on V1 for docker 4.4.1 not support urllibV2.
Now docker is upgraded to V6. The issue didn't exist.

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@liushilongbuaa liushilongbuaa merged commit 4b547ef into sonic-net:master Jul 27, 2023
4 checks passed
liushilongbuaa added a commit to liushilongbuaa/sonic-utilities that referenced this pull request Jul 27, 2023
…onic-net#2928)

* [build] Fix dependency issue between responses and urllib3 package.
liushilongbuaa added a commit to liushilongbuaa/sonic-utilities that referenced this pull request Jul 27, 2023
…onic-net#2928)

* [build] Fix dependency issue between responses and urllib3 package.
StormLiangMS pushed a commit that referenced this pull request Jul 27, 2023
…2928) (#2930)

* [build] Fix dependency issue between responses and urllib3 package.
StormLiangMS pushed a commit that referenced this pull request Jul 27, 2023
…2928) (#2929)

* [build] Fix dependency issue between responses and urllib3 package.
pdhruv-marvell pushed a commit to pdhruv-marvell/sonic-utilities that referenced this pull request Aug 23, 2023
…onic-net#2928)

* [build] Fix dependency issue between responses and urllib3 package.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants