Add support for snat, dnat and ipmc crm resources#1258
Add support for snat, dnat and ipmc crm resources#1258arlakshm merged 2 commits intosonic-net:masterfrom
Conversation
|
please add unit tests accordingly. |
Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom.com> Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom>
3b44b87 to
fca8737
Compare
I have updated the PR with sonic-utilities test cases for the newly added resources. |
lguohan
left a comment
There was a problem hiding this comment.
i am wondering if there is no data in the db, what is the output, will the command throw exception?
When no data on CONFIG_DB, crm show threshold CLI command will not have these resources listed.
No exception will be observed on both cases. |
|
retest this please |
|
All, @arlakshm - this has been quiet for 6 days - can we get some review input so we can move forwards please? The clock is ticking ... |
|
@ben-gale - fyi on a test failure which might be related to this PR - https://github.com/Azure/sonic-buildimage/issues/6302 |
What I did Added support for snat, dnat and ipmc resources under CRM module. Why I did it New feature NAT adds new resources snat_enty and dnat_entry that needs to be monitored. ipmc_entry tracks IP multicast resources used by switch. Associated PRs sonic-net/sonic-utilities/pull/1258 sonic-net/sonic-buildimage#6012 Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom.com>
Signed-off-by: Prabhu Sreenivasan prabhu.sreenivasan@broadcom.com - What I did Added support for snat, dnat and ipmc resources under CRM module. - How I did it New feature NAT adds new resources snat_enty and dnat_entry that needs to be monitored. ipmc_entry tracks IP multicast resources used by switch. - How to verify it sonic-utilities tests and crm spytest
What I did Added support for snat, dnat and ipmc resources under CRM module. Why I did it New feature NAT adds new resources snat_enty and dnat_entry that needs to be monitored. ipmc_entry tracks IP multicast resources used by switch. Associated PRs sonic-net/sonic-utilities/pull/1258 sonic-net/sonic-buildimage#6012 Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom.com>
Signed-off-by: Prabhu Sreenivasan prabhu.sreenivasan@broadcom.com
- What I did
Added support for snat, dnat and ipmc resources under CRM module.
- How I did it
New feature NAT adds new resources snat_enty and dnat_entry that needs to be monitored. ipmc_entry tracks IP multicast resources used by switch.
- How to verify it
sonic-utilities tests and crm spytest
- Associated PRs
sonic-net/sonic-swss#1511
sonic-net/sonic-buildimage#6012
- Previous command output (if the output of a command-line utility has changed)
- New command output (if the output of a command-line utility has changed)