Skip to content

[action] [PR:3968] [Marvell-Teralynx][PFCWD]: Fix pfc_detect filename to align with asic_type#4322

Open
mssonicbld wants to merge 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/3968
Open

[action] [PR:3968] [Marvell-Teralynx][PFCWD]: Fix pfc_detect filename to align with asic_type#4322
mssonicbld wants to merge 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/3968

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

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:

  • Renamed pfc_detect_marvell_teralynx.lua to pfc_detect_marvell-teralynx.lua
  • Updated Makefile.am to reference the renamed file

Why I did it:

  • Fix the naming of pfc_detect file to align with asic_type convention
  • Maintain consistency with platform naming standards

How I verified it:
Run the pfc_wd SONiC PTF test cases.

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

  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version):
202505

Signed-off-by: Sonic Build Admin sonicbld@microsoft.com

…_type

**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 sonic-net#3252 and ensures the filename matches the expected asic_type format used throughout the codebase.

**What I did:**
- Renamed pfc_detect_marvell_teralynx.lua to pfc_detect_marvell-teralynx.lua
- Updated Makefile.am to reference the renamed file

**Why I did it:**
- Fix the naming of pfc_detect file to align with asic_type convention
- Maintain consistency with platform naming standards

**How I verified it:**
Run the pfc_wd SONiC PTF test cases.

**Which release branch to backport (provide reason below if selected):**
- [ ] 202405
- [ ] 202411
- [x] 202505
- [x] 202511

**Tested branch (Please provide the tested image version):**
202505

<!--
Please make sure you have read and understood the contribution guildlines:
https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

1. Make sure your commit includes a signature generted with `git commit -s`
2. Make sure your commit title follows the correct format: [component]: description
3. Make sure your commit message contains enough details about the change and related tests
4. Make sure your pull request adds related reviewers, asignees, labels

Please also provide the following information in this pull request:
-->

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #3968

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@radha-danda
Copy link
Copy Markdown

/azpw run Azure.sonic-swss

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/AzurePipelines run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@radha-danda
Copy link
Copy Markdown

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

⚠️ Notice: /azpw run only runs failed jobs now. If you want to trigger a whole pipline run, please rebase your branch or close and reopen the PR.
💡 Tip: You can also use /azpw retry to retry failed jobs directly.

Retrying failed(or canceled) jobs...

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Build not found. Please close and reopen the PR or rebase your branch to trigger a new build.

@radha-danda radha-danda reopened this Mar 30, 2026
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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.

3 participants