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

changes to support python3 in mgmt-framework #6038

Merged
merged 13 commits into from
Dec 22, 2020

Conversation

faraazbrcm
Copy link
Contributor

@faraazbrcm faraazbrcm commented Nov 25, 2020

- Why I did it

- How I did it

- How to verify it

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

@faraazbrcm
Copy link
Contributor Author

@sachinholla @anand-kumar-subramanian please review the changes

@jleveque
Copy link
Contributor

Retest vsimage please

@lguohan
Copy link
Collaborator

lguohan commented Dec 8, 2020

retest vsimage please

@lguohan
Copy link
Collaborator

lguohan commented Dec 21, 2020

@ben-gale , there is not much activity on this one, can you check?

@faraazbrcm
Copy link
Contributor Author

Retest vsimage please

@@ -10,12 +10,20 @@ RUN apt-get update && \
apt-get install -y g++ python-dev libxml2

RUN pip install connexion==1.1.15 \
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest replacing pip with pip2 to ensure the Python 2 version is used here.

@lguohan lguohan merged commit 9d35fa1 into sonic-net:master Dec 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants