Skip to content

[202012] [Celestica/Seastone2] Install sonic_platform#9770

Closed
bluecmd wants to merge 2 commits intosonic-net:202012from
bluecmd:patch-6
Closed

[202012] [Celestica/Seastone2] Install sonic_platform#9770
bluecmd wants to merge 2 commits intosonic-net:202012from
bluecmd:patch-6

Conversation

@bluecmd
Copy link
Contributor

@bluecmd bluecmd commented Jan 16, 2022

Why I did it

This is part of fixing #8212.
This changes fixes installation of sonic_platform to be installed for both python2 and python3.
Right now neither is installed.

How I did it

Looked at the error messages when booting a clean Seastone2 as well as the packaging files.

How to verify it

Verify that sonic_platform is installed.

(vrf:mgmt)bluecmd@ixp-kg-sw1:~$ python2 -c 'import sonic_platform'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: No module named sonic_platform
(vrf:mgmt)bluecmd@ixp-kg-sw1:~$ python3 -c 'import sonic_platform'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'sonic_platform'

After:

(vrf:mgmt)bluecmd@ixp-kg-sw1:~$ python2 -c 'import sonic_platform'
(vrf:mgmt)bluecmd@ixp-kg-sw1:~$ python3 -c 'import sonic_platform'

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

cel/seastone2: install sonic_platform

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

image

@bluecmd bluecmd changed the title [Celestica/Seastone2] Install sonic_platform [202012] [Celestica/Seastone2] Install sonic_platform Jan 16, 2022
@bluecmd
Copy link
Contributor Author

bluecmd commented Feb 1, 2022

@sujinmkang can you maybe review this?

@sujinmkang
Copy link
Collaborator

@bluecmd Can you please rebase this PR again?

@bluecmd
Copy link
Contributor Author

bluecmd commented May 5, 2022

@sujinmkang Rebase was done :-)

bluecmd added 2 commits May 5, 2022 09:03
`pip` refers to `pip3` in at least 202012 and newer.

Signed-off-by: Christian Svensson <[email protected]>
@bluecmd
Copy link
Contributor Author

bluecmd commented Jul 27, 2022

@lguohan OK to merge?

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.

2 participants