-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Support kernel.net_ticktime in Cuttlefish configuration (master) #1524
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
Support kernel.net_ticktime in Cuttlefish configuration (master) #1524
Conversation
RABBITMQ_NAME_TYPE should be correctly set in Windows batch files
net_ticktime, if present in the .conf file, is set up with net_kernel:set_net_ticktime/1 before the configuration of the .conf is applied to the other applications. The kernel is already running when the configuration is applied, so net_ticktime is set in a specific way. [#155393098] Fixes #1522
net_ticktime, if present in the .conf file, is set up with net_kernel:set_net_ticktime/1 before the configuration of the .conf is applied to the other applications. The kernel is already running when the configuration is applied, so net_ticktime is set in a specific way. [#155393098] Fixes #1522 (cherry picked from commit fdb3e75)
… into rabbitmq-server-1522-net-ticktime-in-new-config-format-master
I discovered the following during manual testing: since the transition period by default takes 60 seconds, should a CLI tool contact a running node with
because CLI tools temporarily tweak node's net_ticktime for the two values to be in sync. We can try a transition period of 0 but I don't know what complications can stem out of that. I don't think the change is worth the risk since it's a minor convenience improvement. |
A correction: it's not just CLI tools, it's any code that uses rabbit_misc:rpc_call/5 but it should not be used on any hot code path. Updating that function to support transition periods should be sufficient. It can still potentially be a breaking change for the environment where CLI tools use a significantly higher tick time value vs. the server: the server would then detect inactivity on an idle connection much earlier than the CLI tool. We don't have a lot of reports about that, though, possibly because CLI tool connections do go inactive for long enough periods of time. |
According to rabbitmq/rabbitmq-common#60, that function is used by management and possibly other plugins. |
…t_ticktime/2 See rabbitmq/rabbitmq-server#1524 and rabbitmq/rabbitmq-server#1522 for details.
…t_ticktime/2 See rabbitmq/rabbitmq-server#1524 and rabbitmq/rabbitmq-server#1522 for details. (cherry picked from commit cf78ab8)
net_ticktime
, if present in the.conf
file, is set up withnet_kernel:set_net_ticktime/1
before the configuration of the.conf
file is applied to the other applications. Thekernel
application is already running when the configuration is applied, so net_ticktime is set in a specific way.Fixes #1522