Skip to content

[pfcwd] Fix log message logic to indicate correct db based on buffer model#3512

Merged
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:fix_pfcwd_log
May 21, 2021
Merged

[pfcwd] Fix log message logic to indicate correct db based on buffer model#3512
neethajohn merged 1 commit intosonic-net:masterfrom
neethajohn:fix_pfcwd_log

Conversation

@neethajohn
Copy link
Copy Markdown
Contributor

Signed-off-by: Neetha John [email protected]

Description of PR

Log message should indicate APPL_DB when buffer_model is dynamic. In case there is no buffer_model defined or buffer_model is traditional, log message should indicate CONFIG_DB

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

How did you verify/test it?

Ran the test and verified that correct msg is printed

@neethajohn neethajohn requested a review from a team as a code owner May 20, 2021 18:18
@neethajohn neethajohn requested a review from lolyu May 20, 2021 18:19
@neethajohn
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@lolyu lolyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@neethajohn neethajohn merged commit 32044a7 into sonic-net:master May 21, 2021
@neethajohn neethajohn deleted the fix_pfcwd_log branch May 21, 2021 21:22
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
…model (sonic-net#3512)

Log message should indicate APPL_DB when buffer_model is dynamic. In case there is no buffer_model defined or buffer_model is traditional, log message should indicate CONFIG_DB

Signed-off-by: Neetha John <[email protected]>

How did you verify/test it?
Ran the test and verified that correct msg is printed
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (sonic-net#21664)

#### Why I did it
src/sonic-swss
```
* 5031aadc - (HEAD -> 202411, origin/202411) Capability query for MACSEC ACL attribute (sonic-net#3511) (31 hours ago) [mssonicbld]
* 4b357e59 - Fix VRF update handling for loopback interfaces in IntfsOrch (sonic-net#3512) (31 hours ago) [mssonicbld]
* fe98176b - Add a delay between killing teamd processes (sonic-net#3510) (2 days ago) [mssonicbld]
* e967711e - Remove RIF from m_rifsToAdd before deleting it (sonic-net#3499) (6 days ago) [mssonicbld]
* 337c9a10 - Optimize counter polling interval by making it more accurate (sonic-net#3500) (6 days ago) [mssonicbld]
```
#### 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants