Skip to content

Add disable_pfcwd option in docker_image_ctl.j2 - #75

Open
noaOrMlnx wants to merge 1 commit into
masterfrom
disable_pfc_wd_specific_skus
Open

Add disable_pfcwd option in docker_image_ctl.j2#75
noaOrMlnx wants to merge 1 commit into
masterfrom
disable_pfc_wd_specific_skus

Conversation

@noaOrMlnx

Copy link
Copy Markdown
Owner

Why I did it

There are specific SKUs which require to disable PFCWD.

How I did it

Added the option in docker_image_ctl.j2 - if disable_pfcwd file exists in HwSKU folder, disable it in CONFIG DB.

How to verify it

Add disable_pfcwd to HwSKU folder and make sure PFCWD is disabled in CONFIG DB.

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

noaOrMlnx pushed a commit that referenced this pull request May 5, 2026
…ly (sonic-net#26787)

#### Why I did it
src/sonic-ztp
```
* 036c8ea - (HEAD -> master, origin/master, origin/HEAD) ztp: harden curl command construction in Downloader and DHCP hook (#75) (4 days ago) [xq9mend]
* c84b50a - Use `shutil.move` instead of `os.rename` to move `config_db.json` on place (#66) (6 days ago) [Vladimir Solomatin]
```
#### How I did it
#### How to verify it
#### Description for the changelog
noaOrMlnx pushed a commit that referenced this pull request Jul 27, 2026
…sonic-net#28349)

#### Why I did it
src/dhcpmon
```
* 1f3f4f9 - (HEAD -> master, origin/master, origin/HEAD) [dhcp_devman] Ignore duplicate -i* arguments instead of crashing/leaking (#76) (13 hours ago) [Xichen96]
* 2883fe8 - [dhcpmon] Log Relay-Forward / Relay-Reply counters in v6 status (#75) (13 hours ago) [Xichen96]
* 892925b - [dhcpmon] Cleanup stale v6 counters from COUNTERS_DB (#74) (16 hours ago) [Xichen96]
```
#### 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant