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

[Mellanox] install Mellanox's new platform api on the host #3282

Merged
merged 1 commit into from
Sep 25, 2019

Conversation

stephenxs
Copy link
Collaborator

- What I did
install Mellanox's new platform api on the host

- How I did it

  1. update makefiles, adding the logic of making and installing sonic_platform and sonic_daemon_base on the host.
  2. remove the unnecessary dependency of swss_common on the host by only importing swss_common when connecting to the Redis db.

file changed details:

  • slave.mk: add SONIC_PLATFORM_API_PY2 as dependency of host
  • sonic_debian_extension.j2: install sonic_daemon_base and Mellanox-specific sonic_platform on host
  • mlnx-platform-api.mk: export mlnx_platform_api_py2_wheel_path for sonic_debian_extension.j2
  • sonic-daemon-base.mk: export daemon_base_py2_wheel_path for sonic_debian_extension.j2
  • daemon_base.py: remove unnecessary dependency of swss_common on host

- How to verify it
make the sonic-mellanox.bin and then verity whether sonic_platform has been installed on host in that image.

- Description for the changelog
[Mellanox] install Mellanox's new platform api on the host

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

slave.mk: add SONIC_PLATFORM_API_PY2 as dependency of host
sonic_debian_extension.j2: install sonic_daemon_base and Mellanox-specific sonic_platform on host
mlnx-platform-api.mk: export mlnx_platform_api_py2_wheel_path for sonic_debian_extension.j2
sonic-daemon-base.mk: export daemon_base_py2_wheel_path for sonic_debian_extension.j2
daemon_base.py: hind unnecessary dependency of swss_common on host
@stephenxs
Copy link
Collaborator Author

@andriymoroz-mlnx @nazariig Can we go ahead to get the PR approved if we don't have further question?

@jleveque jleveque merged commit c34a478 into sonic-net:master Sep 25, 2019
@stephenxs stephenxs deleted the pr-install-npapi-on-host branch September 25, 2019 19:26
mssonicbld added a commit that referenced this pull request Sep 21, 2024
…lly (#20310)

#### Why I did it
src/sonic-swss
```
* 9a6a86fe - (HEAD -> master, origin/master, origin/HEAD) Add FEC correctable bit error count to ports_stat_ids (#3291) (28 hours ago) [Prince George]
* 5126f88b - Fix the SAI status check for unsupported API on SAI_SWITCH_ATTR_SUPPO… (#3282) (32 hours ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog
saksarav-nokia pushed a commit to saksarav-nokia/sonic-buildimage that referenced this pull request Oct 9, 2024
…lly (sonic-net#20310)

#### Why I did it
src/sonic-swss
```
* 9a6a86fe - (HEAD -> master, origin/master, origin/HEAD) Add FEC correctable bit error count to ports_stat_ids (sonic-net#3291) (28 hours ago) [Prince George]
* 5126f88b - Fix the SAI status check for unsupported API on SAI_SWITCH_ATTR_SUPPO… (sonic-net#3282) (32 hours ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Nov 11, 2024
…lly (#20729)

#### Why I did it
src/sonic-swss
```
* 4699be7e - (HEAD -> 202405, origin/202405) Fix the SAI status check for unsupported API on SAI_SWITCH_ATTR_SUPPO… (#3282) (4 days ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog
aidan-gallagher pushed a commit to aidan-gallagher/sonic-buildimage that referenced this pull request Nov 16, 2024
…lly (sonic-net#20310)

#### Why I did it
src/sonic-swss
```
* 9a6a86fe - (HEAD -> master, origin/master, origin/HEAD) Add FEC correctable bit error count to ports_stat_ids (sonic-net#3291) (28 hours ago) [Prince George]
* 5126f88b - Fix the SAI status check for unsupported API on SAI_SWITCH_ATTR_SUPPO… (sonic-net#3282) (32 hours ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants