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

Add python-is-python3 package for broadcom syncd docker container #11807

Closed
wants to merge 1 commit into from

Conversation

ZhaohuiS
Copy link
Contributor

Signed-off-by: Zhaohui Sun zhaohuisun@microsoft.com

Why I did it

In latest syncd container, it is installed bullseye, can't find command '/usr/bin/python'.
Some scripts such as test_copp still calls /usr/bin/python in syncd.

How I did it

Install python-is-python3 package to resolve this issue, whatever run python or python3, it will both run /usr/bin/python3, will not cause the error of can't find command '/usr/bin/python'

How to verify it

run python in syncd container.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
@ZhaohuiS ZhaohuiS requested a review from lguohan as a code owner August 22, 2022 05:49
@lguohan
Copy link
Collaborator

lguohan commented Aug 29, 2022

i suggest to install this on the bullseye base docker.

@ZhaohuiS
Copy link
Contributor Author

i suggest to install this on the bullseye base docker.

@lguohan Agree, resubmitted it for bullseye base docker in PR #11895. Close this one.

@ZhaohuiS
Copy link
Contributor Author

@wangxin close this PR as I resubmitted in #11895.

@ZhaohuiS ZhaohuiS closed this Aug 30, 2022
ZhaohuiS added a commit that referenced this pull request Sep 1, 2022
Why I did it
In latest syncd container, it is installed bullseye, can't find command '/usr/bin/python'.
Some scripts such as test_copp still calls /usr/bin/python in syncd.
Submitted the change in #11807 for syncd docker, but it's better to add it in bullseye base docker.

How I did it
Install python-is-python3 package in bullseye base docker to resolve this issue, whatever run python or python3, it will run /usr/bin/python3, will not cause the error of can't find command '/usr/bin/python'

How to verify it
run python in syncd container.

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants