rabbit_quorum_queue: Remove old compatibility code in status/2

The `sys:get_status/1` fallback doesn't work as expected since the
changes in Ra for OTP 27: <rabbitmq/ra@ab280c7>.
This code is meant for mixed-versions compatibility with nodes that
don't have the `rabbit_quorum_queue:key_metrics_rpc/1` function which
was added in v3.13.0.

Quorum queues with a member down could see a failure of the
quorum_status CLI command like so:

    % rabbitmq rabbitmq-queues quorum_status --vhost / "qq-1"
    Status of quorum queue qq-1 on node [email protected] ...
    Error:
    {{:badmatch, false}, [{:rabbit_quorum_queue, :"-status/2-lc$^0/1-0-", 2, [file: ~c"rabbit_quorum_queue.erl", line: 1287]}, {:rabbit_quorum_queue, :status, 2, []}]}

The badmatch comes from this line in `rabbit_quorum_queue:status/2`:

    {_, M} = lists:keyfind(ra_server_state, 1, Sys),

The `Sys` return was `[{data, Sys}]` since the linked change in Ra, so
the `keyfind/3` would always return `false`.

(cherry picked from commit 38f83d7)

Author: the-mikedavis

deps/rabbit/src/rabbit_quorum_queue.erl

@@ -1295,43 +1295,22 @@ status(Vhost, QueueName) ->
                           {<<"Term">>, <<>>},
                           {<<"Machine Version">>, MacVer}
                          ];
-                     {error, _} ->
-                         %% try the old method
-                         case get_sys_status(ServerId) of
-                             {ok, Sys} ->
-                                 {_, M} = lists:keyfind(ra_server_state, 1, Sys),
-                                 {_, RaftState} = lists:keyfind(raft_state, 1, Sys),
-                                 #{commit_index := Commit,
-                                   machine_version := MacVer,
-                                   current_term := Term,
-                                   last_applied := LastApplied,
-                                   log := #{last_index := Last,
-                                            last_written_index_term := {LastWritten, _},
-                                            snapshot_index := SnapIdx}} = M,
-                                 [{<<"Node Name">>, N},
-                                  {<<"Raft State">>, RaftState},
-                                  {<<"Membership">>, voter},
-                                  {<<"Last Log Index">>, Last},
-                                  {<<"Last Written">>, LastWritten},
-                                  {<<"Last Applied">>, LastApplied},
-                                  {<<"Commit Index">>, Commit},
-                                  {<<"Snapshot Index">>, SnapIdx},
-                                  {<<"Term">>, Term},
-                                  {<<"Machine Version">>, MacVer}
-                                 ];
-                             {error, Err} ->
-                                 [{<<"Node Name">>, N},
-                                  {<<"Raft State">>, Err},
-                                  {<<"Membership">>, <<>>},
-                                  {<<"Last Log Index">>, <<>>},
-                                  {<<"Last Written">>, <<>>},
-                                  {<<"Last Applied">>, <<>>},
-                                  {<<"Commit Index">>, <<>>},
-                                  {<<"Snapshot Index">>, <<>>},
-                                  {<<"Term">>, <<>>},
-                                  {<<"Machine Version">>, <<>>}
-                                 ]
-                         end
+                     {error, Reason} ->
+                         State = case is_atom(Reason) of
+                                     true -> Reason;
+                                     false -> unknown
+                                 end,
+                         [{<<"Node Name">>, N},
+                          {<<"Raft State">>, State},
+                          {<<"Membership">>, <<>>},
+                          {<<"Last Log Index">>, <<>>},
+                          {<<"Last Written">>, <<>>},
+                          {<<"Last Applied">>, <<>>},
+                          {<<"Commit Index">>, <<>>},
+                          {<<"Snapshot Index">>, <<>>},
+                          {<<"Term">>, <<>>},
+                          {<<"Machine Version">>, <<>>}
+                         ]
                  end
              end || N <- Nodes];
         {ok, _Q} ->
@@ -1340,17 +1319,6 @@ status(Vhost, QueueName) ->
             E
     end.
 
-get_sys_status(Proc) ->
-    try lists:nth(5, element(4, sys:get_status(Proc))) of
-        Sys -> {ok, Sys}
-    catch
-        _:Err when is_tuple(Err) ->
-            {error, element(1, Err)};
-        _:_ ->
-            {error, other}
-
-    end.
-
 add_member(VHost, Name, Node, Membership, Timeout)
   when is_binary(VHost) andalso
        is_binary(Name) andalso