add paused_actions for INFO Clients

[soloestoy]

Add paused_actions and paused_timeout_milliseconds for INFO Clients to inform users about if clients are paused.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.95%. Comparing base (b3b4bdc) to head (035d8a5).
Report is 31 commits behind head on unstable.

Additional details and impacted files

@@             Coverage Diff              @@
##           unstable    #1519      +/-   ##
============================================
+ Coverage     70.83%   70.95%   +0.11%     
============================================
  Files           120      120              
  Lines         64911    65060     +149     
============================================
+ Hits          45982    46164     +182     
+ Misses        18929    18896      -33     

Files with missing lines	Coverage Δ
src/networking.c	88.51% <100.00%> (+0.26%)	⬆️
src/server.c	87.64% <100.00%> (+0.17%)	⬆️
src/server.h	100.00% <ø> (ø)

... and 53 files with indirect coverage changes

[madolson]

I think this is a good info field to add.

[madolson]

Just fix the format error and it's good for me.

[hwware]

I am confused why this pr is tagged as major-decision-pending?

[madolson]

@hwware New info fields. I thought I posted a message, but maybe I forgot. @valkey-io/core-team I suppose the major decision is if people are okay with the new info fields listed. (Maybe also including the one @artikell suggested).

[zuiderkwast]

This field will be "none" most of the time, right? If we add many fields like this, is the size of the INFO reply a concern?

I remember for the graceful shutdown feature, we added a field that is only present during shutdown:

• shutdown_in_milliseconds: The maximum time remaining for replicas to catch up the replication before completing the shutdown sequence. This field is only present during shutdown.

In the same way, I wanted to add the availability_zone in HELLO reply only if it's non-empty, to avoid bloating the reply with a field that's always empty for many users.

[enjoy-binbin]

this make sense to me, i also mention this in #1131 (comment). I would also want a purpose field if we add the action.

[hwware]

@valkey-io/core-team Although it is approved, status is still major-decision pending, can we just have a vote for it?

[zuiderkwast]

The "stats" section is maybe wrong? Isn't it better to put it in another INFO section?

[soloestoy]

@zuiderkwast which section you think is better?

[zuiderkwast]

Clients section is better? Paused is about paused clients.

Stats is counters of various information. paused_actions is just the current state, so not really stats.

[soloestoy]

Clients makes sense, since it is from CLIENT PAUSE command

[madolson]

I was going to suggest client or server, since it's server side state. I don't feel strongly to change it, I also agreed I thought stats was the wrong place.