[Mellanox][202012] Enhance Python3 support for platform API (back-porting #7410)#72
Closed
[Mellanox][202012] Enhance Python3 support for platform API (back-porting #7410)#72
Conversation
keboliu
approved these changes
Jun 17, 2021
stephenxs
changed the title
- Why I did it Enhance the Python3 support for platform API. Originally, some platform APIs call SDK API which didn't support Python 3. Now the Python 3 APIs have been supported in SDK 4.4.3XXX, Python3 is completely supported by platform API - How I did it Start all platform daemons from python3 1. Remove #/usr/bin/env python at the beginning of each platform API file as the platform API won't be started as daemons but be imported from other daemons. 2. Adjust SDK API calls accordingly - How to verify it Manually test and run regression platform test Signed-off-by: Stephen Sun <[email protected]>
stephenxs
force-pushed
the
cherr-pick-python3
branch
from
7f91625 to
c725a8c
Compare
stephenxs
pushed a commit
that referenced
this pull request
Jun 5, 2022
[sonic-linkmgrd][202012] submodule updates 489cf3 Jing Zhang Wed May 18 09:59:02 2022 -0700 Avoid switching active when LinkState == Down (#77) a6c9713 Jing Zhang Tue May 24 11:03:54 2022 -0700 [202012] Add option to enable or disable default route related feature (#72) dbb607d Jing Zhang Thu May 12 08:19:20 2022 -0700 [ci]: uplift diff coverage threshold to 80% (#71) sign-off: Jing Zhang [email protected]
stephenxs
pushed a commit
that referenced
this pull request
May 16, 2023
a24412c25 (HEAD, origin/master, origin/HEAD, master) [mka]: Fix unexpected cleanup (#73) 26d1da0bc [mka]: Fix re-establishment by reset MI (#72) Signed-off-by: Ze Gan <[email protected]>
stephenxs
pushed a commit
that referenced
this pull request
Dec 19, 2024
…ically (sonic-net#21040) #### Why I did it src/sonic-dbsyncd ``` * 7ad34f6 - (HEAD -> master, origin/master, origin/HEAD) [ci] Fix pipeline issue caused by dist upgrade (#72) (53 minutes ago) [Liu Shilong] ``` #### How I did it #### How to verify it #### Description for the changelog
stephenxs
pushed a commit
that referenced
this pull request
Jul 23, 2025
…ically (sonic-net#23157) #### Why I did it src/sonic-dash-ha ``` * b8ca7c1 - (HEAD -> master, origin/master, origin/HEAD) Change field_values deserializer to support serde(default) (#71) (4 hours ago) [yue-fred-gao] * 1b6f04f - Test utils to test hamgrd in local workspace (#72) (4 hours ago) [yue-fred-gao] ``` #### How I did it #### How to verify it #### Description for the changelog
stephenxs
pushed a commit
that referenced
this pull request
Sep 16, 2025
…ly (sonic-net#23898) #### Why I did it src/sonic-stp ``` * e3626a4 - (HEAD -> master, origin/master, origin/HEAD) [MSTP] Fixing buffer OOB Read/Write issues (#72) (20 hours ago) [Shivashankar C R] ``` #### How I did it #### How to verify it #### Description for the changelog
stephenxs
pushed a commit
that referenced
this pull request
Oct 20, 2025
…ly (sonic-net#24123) #### Why I did it src/dhcprelay ``` * 5b078ef - (HEAD -> master, origin/master, origin/HEAD) Move DHCPv6 Relay code to dhcp6relay folder (#72) (7 minutes ago) [Shivashankar C R] ``` #### How I did it #### How to verify it #### Description for the changelog
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why I did it
This is to back-port Azure 7410 to 202012 branch.
Enhance the Python3 support for platform API. Originally, some platform APIs call SDK API which didn't support Python 3. Now the Python 3 APIs have been supported in SDK 4.4.3XXX, Python3 is completely supported by platform API
How I did it
Start all platform daemons from python3
Manually test and run regression platform test
Signed-off-by: Stephen Sun [email protected]
Why I did it
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)