Skip to content

[restore_neighbors] python3 support for restore_neighbors.py#1542

Merged
lguohan merged 1 commit intosonic-net:masterfrom
zhenggen-xu:python3-neighbor-restore
Dec 20, 2020
Merged

[restore_neighbors] python3 support for restore_neighbors.py#1542
lguohan merged 1 commit intosonic-net:masterfrom
zhenggen-xu:python3-neighbor-restore

Conversation

@zhenggen-xu
Copy link
Copy Markdown
Collaborator

@zhenggen-xu zhenggen-xu commented Dec 13, 2020

[restore_neighbors] python3 support for restore_neighbors.py

Signed-off-by: Zhenggen Xu zxu@linkedin.com

This is dependent on sonic-net/sonic-buildimage#6207

What I did
Change the restore_neighbors.py to support python3

Why I did it
python2 EOL.

How I verified it
swss vs tests

Details if related

@zhenggen-xu zhenggen-xu marked this pull request as ready for review December 15, 2020 06:50
@zhenggen-xu zhenggen-xu requested a review from jleveque December 15, 2020 06:51
lguohan pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Dec 15, 2020
…restore_neighbors.py (#6207)

Install the necessary python3 dependent packages to convert restore_neighbor.py 
to support python3 as python2 is EOL. See: sonic-net/sonic-swss#1542

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
@jleveque
Copy link
Copy Markdown
Contributor

I believe the vs test will fail until the next vs image is built later today with the changes from sonic-net/sonic-buildimage#6207. Will wait to retest until tomorrow.

@zhenggen-xu
Copy link
Copy Markdown
Collaborator Author

retest this please

3 similar comments
@zhenggen-xu
Copy link
Copy Markdown
Collaborator Author

retest this please

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Dec 17, 2020

retest this please

@zhenggen-xu
Copy link
Copy Markdown
Collaborator Author

retest this please

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
@zhenggen-xu zhenggen-xu force-pushed the python3-neighbor-restore branch from f666c7a to a20dd85 Compare December 19, 2020 01:34
@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Dec 19, 2020

retest vs please

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Dec 19, 2020

retest this please

1 similar comment
@zhenggen-xu
Copy link
Copy Markdown
Collaborator Author

retest this please

@lguohan lguohan merged commit f11063f into sonic-net:master Dec 20, 2020
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
bug introduced in sonic-net#1393

Signed-off-by: Guohan Lu <lguohan@gmail.com>
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
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