Your message dated Mon, 12 Apr 2021 20:11:53 +0000
with message-id <[email protected]>
and subject line unblock rabbitmq-server
has caused the Debian Bug report #986846,
regarding unblock: rabbitmq-server/3.8.9-3
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
986846: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=986846
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: [email protected]
Usertags: unblock

Please unblock package rabbitmq-server

There's a nasty bug when rabbit parses its config file.

If rabbit missreads it, it may incorrectly size its memory
footpring, leading to a (maybe) non-functional service that
doesn't allocate enough memory.

The proposed new release includes a patch from upstream
to fix the situation.

Please unblock rabbitmq-server/3.8.9-3

Cheers,

Thomas Goirand (zigo)
diff -Nru rabbitmq-server-3.8.9/debian/changelog 
rabbitmq-server-3.8.9/debian/changelog
--- rabbitmq-server-3.8.9/debian/changelog      2021-03-26 14:24:32.000000000 
+0100
+++ rabbitmq-server-3.8.9/debian/changelog      2021-04-10 22:59:57.000000000 
+0200
@@ -1,3 +1,10 @@
+rabbitmq-server (3.8.9-3) unstable; urgency=medium
+
+  [ Adam Cecile ]
+  * Merge upstream PR #2965 fixing rabbitmqctl parsing bug. (Closes: #986734)
+
+ -- Thomas Goirand <[email protected]>  Sat, 10 Apr 2021 22:59:57 +0200
+
 rabbitmq-server (3.8.9-2) unstable; urgency=medium
 
   * Do not use the /usr/sbin/rabbitmq-server wrapper in systemd service file
diff -Nru rabbitmq-server-3.8.9/debian/patches/series 
rabbitmq-server-3.8.9/debian/patches/series
--- rabbitmq-server-3.8.9/debian/patches/series 2021-03-26 14:24:32.000000000 
+0100
+++ rabbitmq-server-3.8.9/debian/patches/series 2021-04-10 22:59:57.000000000 
+0200
@@ -1,2 +1,3 @@
 lets-use-python3-not-python-binary.patch
 rabbitmq-dist.mk.patch
+Upstream_PR2965_fixing_rabbitmqctl_parsing
diff -Nru 
rabbitmq-server-3.8.9/debian/patches/Upstream_PR2965_fixing_rabbitmqctl_parsing 
rabbitmq-server-3.8.9/debian/patches/Upstream_PR2965_fixing_rabbitmqctl_parsing
--- 
rabbitmq-server-3.8.9/debian/patches/Upstream_PR2965_fixing_rabbitmqctl_parsing 
    1970-01-01 01:00:00.000000000 +0100
+++ 
rabbitmq-server-3.8.9/debian/patches/Upstream_PR2965_fixing_rabbitmqctl_parsing 
    2021-04-10 22:59:57.000000000 +0200
@@ -0,0 +1,44 @@
+Description: Fix incorrect parsing of vm_memory_high_watermark
+ Upstream bug:
+ https://github.com/rabbitmq/rabbitmq-server/issues/2964
+Author: Michael Klishin <[email protected]>
+Origin: upstream, https://github.com/rabbitmq/rabbitmq-server/pull/2965
+Bug-Debian: https://bugs.debian.org/986734
+Reviewed-By: Adam Cecile <[email protected]>
+Last-Update: 2021-04-10
+
+--- rabbitmq-server-3.8.9.orig/deps/rabbit/src/rabbit_disk_monitor.erl
++++ rabbitmq-server-3.8.9/deps/rabbit/src/rabbit_disk_monitor.erl
+@@ -66,7 +66,7 @@
+ 
+ %%----------------------------------------------------------------------------
+ 
+--type disk_free_limit() :: (integer() | string() | {'mem_relative', float() | 
integer()}).
++-type disk_free_limit() :: integer() | {'absolute', integer()} | string() | 
{'mem_relative', float() | integer()}.
+ 
+ %%----------------------------------------------------------------------------
+ %% Public API
+@@ -272,6 +272,8 @@ parse_free_win32(CommandResult) ->
+ interpret_limit({mem_relative, Relative})
+     when is_number(Relative) ->
+     round(Relative * vm_memory_monitor:get_total_memory());
++interpret_limit({absolute, Absolute}) ->
++    interpret_limit(Absolute);
+ interpret_limit(Absolute) ->
+     case rabbit_resource_monitor_misc:parse_information_unit(Absolute) of
+         {ok, ParsedAbsolute} -> ParsedAbsolute;
+--- 
rabbitmq-server-3.8.9.orig/deps/rabbitmq_cli/lib/rabbitmq/cli/core/memory.ex
++++ rabbitmq-server-3.8.9/deps/rabbitmq_cli/lib/rabbitmq/cli/core/memory.ex
+@@ -56,6 +56,12 @@ defmodule RabbitMQ.CLI.Core.Memory do
+   def formatted_watermark(val) when is_float(val) do
+     %{relative: val}
+   end
++  def formatted_watermark({:relative, val}) when is_float(val) do
++    %{relative: val}
++  end
++  def formatted_watermark(:infinity) do
++    %{relative: 1.0}
++  end
+   def formatted_watermark({:absolute, val}) do
+     %{absolute: parse_watermark(val)}
+   end

--- End Message ---
--- Begin Message ---
Unblocked.

--- End Message ---

Reply via email to