[ZMQ] Avoid bind on management ip#25330
Conversation
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@qiluo-msft @prabhataravind Can you please review? |
|
@prsunny , @venkit-nexthop Please review |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
venkit-nexthop
left a comment
venkit-nexthop
left a comment
There was a problem hiding this comment.
Looks good to me.
As discussed in #25279 we need to ensure that binding to mgmt_ip is not necessary any more.
vivekrnv
force-pushed
the
fix_zmq_bind
branch
from
be06259 to
7e93226
Compare
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
Signed-off-by: Vivek Reddy <[email protected]>
Signed-off-by: Vivek Reddy <[email protected]>
vivekrnv
force-pushed
the
fix_zmq_bind
branch
from
7e93226 to
b14fc0c
Compare
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@kperumalbfn, Can you help signoff? |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Cherry-pick PR to 202511: #25654 |
mssonicbld
added
Included in 202511 Branch
and removed
Created PR to 202511 Branch
labels
@rustiqly Can you make a change in sonic-swss to remove the option of '-v mgmt'. Added here sonic-net/sonic-swss#3299 |
[ZMQ] Avoid bind on management ip Signed-off-by: Feng Pan <[email protected]>
[ZMQ] Avoid bind on management ip Signed-off-by: dprital <[email protected]>
7 participants
Why I did it
To avoid starting orchagent on mgmt-ip which serves no purpose and additionally breaks fpmsyncd <-> orchagent communication if enabled
Work item tracking
How I did it
How to verify it
Before this change:
After this change:
Which release branch to backport (provide reason below if selected)
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)