-
Notifications
You must be signed in to change notification settings - Fork 285
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
CA-341155: Fix console refresh when starting management server + other improvements #4176
Conversation
The intention of the if-statement in `start` is to only refresh the consoles on the master if management is enabled, which is the case when the server is running on all IP addresses. The `listening_all` ref, however, is updated only _after_ the call to the `start` function. Instead refer to the `addr` parameter. Signed-off-by: Rob Hoes <rob.hoes@citrix.com>
The only user is `Xapi_pif.reconfigure_ipv6`, which does not actually need it anymore, because we always listen on all IPv4 and IPv6 addresses by default. Signed-off-by: Rob Hoes <rob.hoes@citrix.com>
Reviewers: I suggest reviewing patch-by-patch, and with whitespace changes suppressed for the "restructure" patch. |
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.
Just a small comment - I'm not familiar with the overall logic
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.
Random small comments
Combined the server state refs into a single one with a clear type, and a matching state transformation function. Signed-off-by: Rob Hoes <rob.hoes@citrix.com>
The only way to update the HIMN address is through the `enable_himn` function. Signed-off-by: Rob Hoes <rob.hoes@citrix.com>
No description provided.