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] update python package docker in host image to 6.1.1 #14993

Merged
merged 1 commit into from
May 10, 2023

Conversation

k-v1
Copy link
Contributor

@k-v1 k-v1 commented May 9, 2023

@k-v1 k-v1 requested a review from lguohan as a code owner May 9, 2023 19:51
@k-v1
Copy link
Contributor Author

k-v1 commented May 9, 2023

@xumia
Probably you can also review this.
New version of python package docker is enough to fix build failure.
Another solution is to pin urllib3<2 and requests<2.29, but I think better solution is to use new version of docker package.

@xumia xumia requested a review from liushilongbuaa May 9, 2023 22:33
@xumia
Copy link
Collaborator

xumia commented May 9, 2023

LGTM, @liushilongbuaa , could you please help review it? Thanks.

Copy link
Contributor

@liushilongbuaa liushilongbuaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. both 6.1.0 and 6.1.1 are good.

@lguohan lguohan merged commit bba4fb8 into sonic-net:master May 10, 2023
19 checks passed
@aparkhomenko-xsight
Copy link

@lguohan could you also cherry-pick it to 202211 branch?
Seems like issue was with 202211 branch

@mlok-nokia
Copy link
Contributor

mlok-nokia commented May 11, 2023

@judyjoseph @lguohan @liushilongbuaa This issue also shows up on 202205 branch. Could you please cherry-pick to the 202205 branch?

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202106: #15044

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202111: #15045

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #15050

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #15989

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants