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

[SAI-PTF][mlnx]Enable saiserver test container on mlnx platforms #13311

Merged
merged 7 commits into from
Feb 16, 2023

Conversation

richardyu-ms
Copy link
Contributor

@richardyu-ms richardyu-ms commented Jan 10, 2023

Why I did it

Enable Test sai api on mlnx container with a lightweight container(saiserver).

How I did it
enable saiserver container on mlnx platform.

How I did it

add docker-saiserver-mlnx.mk for building saiserver container in platform/mellanox/docker-saiserver-mlnx, add necessary files that needs in saiserver container

How to verify it

Test with build command

NOSTRETCH=y NOJESSIE=y ENABLE_SYNCD_RPC=y SAITHRIFT_V2=y make target/docker-saiserverv2-mlnx.gz

Also build with test PR
#13312
image
image

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@richardyu-ms richardyu-ms changed the title Why I did it [SAI-PTF][mlnx]Enable saiserver test container on mlnx container Jan 10, 2023
@richardyu-ms richardyu-ms marked this pull request as ready for review February 7, 2023 10:15
@richardyu-ms
Copy link
Contributor Author

@alexrallen could you help review this PR? thanks.

@richardyu-ms richardyu-ms changed the title [SAI-PTF][mlnx]Enable saiserver test container on mlnx container [SAI-PTF][mlnx]Enable saiserver test container on mlnx platforms Feb 8, 2023
@yxieca
Copy link
Contributor

yxieca commented Feb 10, 2023

@richardyu-ms if this change is still needed. please resolve the merge conflict.

Enable Test sai api on bfn container with a lightweight container(saiserver).
[SAI-PTF][mlnx]Enable saiserver test container on mlnx container

How I did it
enable saiserver container on mlnx platform.

add docker-saiserver-mlnx.mk for building saiserver container
in platform/barefoot/docker-saiserver-mlnx, add necessary files that needs in saiserver container
How to verify it

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
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.

5 participants