[Marvell-Teralynx][PFCWD]: Fix pfc_detect filename to align with asic_type#3968
[Marvell-Teralynx][PFCWD]: Fix pfc_detect filename to align with asic_type#3968prsunny merged 13 commits intosonic-net:masterfrom
Conversation
|
|
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
prsunny
left a comment
prsunny
left a comment
There was a problem hiding this comment.
lgtm, @kperumalbfn , can you check. will this have any compatibility issue?
|
@prsunny the changes are specific to Marvell-Teralynx and there are no compatibility issue. |
jithenderkondam
force-pushed
the
plat_name_fix
branch
from
25f9e35 to
19e3b57
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@kperumalbfn , can you please take a look ? |
orchagent/Makefile.am
Outdated
| pfc_detect_marvell-teralynx.lua \ | ||
| pfc_detect_mellanox.lua \ | ||
| pfc_detect_broadcom.lua \ | ||
| pfc_detect_marvell_prestera.lua \ |
There was a problem hiding this comment.
@jithenderkondam could you update marvell_prestera.lua as well
There was a problem hiding this comment.
@kperumalbfn I have updated the marvell_prestera.lua. Could you please take a look.
Also please help to back port the fix to 202505. Thanks!
There was a problem hiding this comment.
@kperumalbfn Could you review the changes
jithenderkondam
force-pushed
the
plat_name_fix
branch
from
19e3b57 to
ad939cf
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
| pfc_detect_mellanox.lua \ | ||
| pfc_detect_broadcom.lua \ | ||
| pfc_detect_marvell_prestera.lua \ | ||
| pfc_detect_marvell-prestera.lua \ |
There was a problem hiding this comment.
@jithenderkondam how about renaming the actual lua script? Please update
There was a problem hiding this comment.
There was a problem hiding this comment.
@kperumalbfn Thanks for pointing that out! I’ve updated the file name.
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azpw run Azure.sonic-swss |
|
/AzurePipelines run Azure.sonic-swss |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azpw run Azure.sonic-swss |
|
/AzurePipelines run Azure.sonic-swss |
|
Azure Pipelines successfully started running 1 pipeline(s). |
jithenderkondam
force-pushed
the
plat_name_fix
branch
from
3f54de3 to
191ca5c
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azpw run Azure.sonic-swss |
|
/AzurePipelines run Azure.sonic-swss |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Cherry-pick PR to 202511: #4322 |
6 participants
Description:
Change pfc_detect filename from pfc_detect_marvell_teralynx.lua to pfc_detect_marvell-teralynx.lua to align with asic_type naming convention.
This fix addresses the naming inconsistency introduced in PR #3252 and ensures the filename matches the expected asic_type format used throughout the codebase.
What I did:
Why I did it:
How I verified it:
Run the pfc_wd SONiC PTF test cases.
Which release branch to backport (provide reason below if selected):
Tested branch (Please provide the tested image version):
202505