Review use of amqqueue:get_pid and node/1

[lukebakken]

Describe the bug

Related:

• rabbit_amqqueue:list/0 should distinguish between queue types during filtering #8564
• Closes #8564 #8595
• VESC-1076

In VESC-1076, we see this stack trace:

Making sure data directory 'c:/Users/username/AppData/Roaming/RabbitMQ/db/rabbit@rabbitmq3-mnesia/msg_stores/vhosts/628WB79CIFDYO9LJI6DKMI09L' for vhost '/' exists
     supervisor: {<0.7418.7>,rabbit_vhost_sup_wrapper}
     errorContext: start_error
     reason: {badarg,
                 [{erlang,node,
                      [{'%2F_Queue','rabbit@rabbitmq3'}],
                      [{error_info,#{module => erl_erts_errors}}]},
                  {rabbit_classic_queue,is_recoverable,1,
                      [{file,"rabbit_classic_queue.erl"},{line,120}]},
                  {rabbit_amqqueue,'-find_local_durable_queues/1-fun-10-',7,
                      [{file,"rabbit_amqqueue.erl"},{line,182}]},
                  {qlc,collect,1,[{file,"qlc.erl"},{line,1373}]},
                  {qlc,eval,2,[{file,"qlc.erl"},{line,301}]},
                  {mnesia_tm,non_transaction,5,
                      [{file,"mnesia_tm.erl"},{line,746}]},
                  {rabbit_amqqueue,mark_local_durable_queues_stopped,1,
                      [{file,"rabbit_amqqueue.erl"},{line,167}]},
                  {rabbit_vhost_process,init,1,
                      [{file,"rabbit_vhost_process.erl"},{line,49}]}]}
     offender: [{pid,undefined},
                {id,rabbit_vhost_process},
                {mfargs,{rabbit_vhost_process,start_link,[<<"/">>]}},
                {restart_type,permanent},
                {significant,false},
                {shutdown,300000},
                {child_type,worker}]

Note the badarg. This is due to assuming you can call node/1 on the return value from amqqueue:get_pid/1

Reproduction steps

N/A

Expected behavior

N/A

Additional context

N/A