-
Notifications
You must be signed in to change notification settings - Fork 670
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "state" field in CONFIG_DB a toggle of the fabric port monitor feature #2932
Add "state" field in CONFIG_DB a toggle of the fabric port monitor feature #2932
Conversation
test_config_int_ip_rem_xxx tests failure, not related to this change |
/azpw run Azure.sonic-utilities |
/AzurePipelines run Azure.sonic-utilities |
Azure Pipelines successfully started running 1 pipeline(s). |
test_config_int_ip_rem_xxx tests failure, do not look like related to this change |
/azpw run Azure.sonic-utilities |
/AzurePipelines run Azure.sonic-utilities |
Azure Pipelines successfully started running 1 pipeline(s). |
268e218
to
49427b2
Compare
test_config_int_ip_rem[ip_route_for_int_ip] test_config_int_ip_rem_static[ip_route_for_int_ip] the failures do not look like related to this change |
49427b2
to
91c7615
Compare
test_config_int_ip_rem[ip_route_for_int_ip] it failed with above tests, it does not look like related to this change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as comments
fabric port monitoring feature. The command to set this is "config fabric port monitor state <enable/disable>"
fabric port monitoring feature. The command to set this is "config fabric port monitor state <enable/disable>" Signed-off-by: Jie Feng <[email protected]>
6dbcb81
to
018770e
Compare
fabric port monitoring feature. The command to set this is "config fabric port monitor state <enable/disable>" Signed-off-by: Jie Feng <[email protected]>
018770e
to
cac9fef
Compare
/Azp run Azure.sonic-utilities |
Azure Pipelines successfully started running 1 pipeline(s). |
@arlakshm could you review again |
/Azp run Azure.sonic-utilities |
/Azp run Azure.sonic-utilities |
Azure Pipelines successfully started running 1 pipeline(s). |
…ature (sonic-net#2932) * Add "state" field in CONFIG_DB fabric_monitor table as a toggle of the fabric port monitoring feature. The command to set this is "config fabric port monitor state <enable/disable>" --------- Signed-off-by: Jie Feng <[email protected]>
What I did Add fabric port monitoring toggle check. The command is added at sonic-net/sonic-utilities#2932. This is the orchagent handling part.
fabric port monitoring feature. The command to set this is "config fabric port monitor state <enable/disable>"
What I did
Add "state" field in CONFIG_DB fabric_monitor table as a toggle of the fabric port monitor feature.
The command is "config fabric port monitor state <enable/disable>"
Also add a test for this. The sonic mgmt test will be added shortly.
How I did it
How to verify it
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)