Skip to content

[ppi]: Enable bulk API#1171

Merged
saiarcot895 merged 1 commit intosonic-net:masterfrom
nazariig:master-ppi-rel
Jan 4, 2023
Merged

[ppi]: Enable bulk API#1171
saiarcot895 merged 1 commit intosonic-net:masterfrom
nazariig:master-ppi-rel

Conversation

@nazariig
Copy link
Collaborator

@nazariig nazariig commented Dec 9, 2022

Signed-off-by: Nazarii Hnydyn [email protected]

HLD: sonic-net/SONiC#1084

  • fix vslib to handle bulk requests properly
  • enable bulk API for both VS and NVIDIA

@liat-grozovik
Copy link
Collaborator

@kcudnik , @saiarcot895 could you please help to review?
@nazariig please check coverage report and have it increased.

@nazariig
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig
Copy link
Collaborator Author

FYI. vstest is unstable: each run is a totally different story.

@nazariig
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

saiarcot895
saiarcot895 previously approved these changes Dec 20, 2022
Signed-off-by: Nazarii Hnydyn <[email protected]>
@nazariig
Copy link
Collaborator Author

nazariig commented Jan 3, 2023

@kcudnik can you please have a look?

@saiarcot895 saiarcot895 merged commit 402eb14 into sonic-net:master Jan 4, 2023
StormLiangMS pushed a commit that referenced this pull request Feb 10, 2023
Signed-off-by: Nazarii Hnydyn <[email protected]>

Signed-off-by: Nazarii Hnydyn <[email protected]>
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