Hello @oranagra , the command of sentinel info-cache basicallly returning the cached info for all the specified masters their slaves, and if no master name provided, it simply returning all cached master and its slave info and its time for how long it was received.that's why it is ok to have 0, 1 or more additional arguments. I think we need to mention this in a better way to avoid people get confused. I will do a fix shortly.
Also I think you bring a good topic here, I also feel there are lots of documentation missing good explation for Sentinel part. Most of the sentinel code was written around 6 years ago (including the info-cache one) so there are missing lots of things. I would suggest we put the sentinel command also into the commands page (https://redis.io/commands). But that requires a lot of documentation change and not sure whether we worth to do it. How do you think?

I rather not mix Sentinel commands with Redis commands in the same table / index.
We can maybe create a similar section lile redis.io/sentinal/commands/ which will behave the same (using the same infrastructure). but i'm not sure it worth the effort.
@itamarhaber any opinion on that?

For now, let's just try to improve the existing page.
But i just wonder if some of these commands were not documented on purpose.
If one of these commands exist from day one (before the documentation page was created), then it must mean that someone considered that an internal debug command and didn't bother to document it.
much like the Redis REPLCONF and (most of) DEBUG commands aren't mentioned.

@oranagra I see, and that is a very good point.. I checked this when at the very beginning Salvatore's commit 78c6f7e#diff-4578785ae9106bc325486759f3f1eae0de1bf7c18af224a19d7c3146cc10b872R161 he mentioned the is-master-down-by-addr command in the doc, but later he removed it. Maybe the is-master-down-by-addr api is used for debugging purpose and Salvatore deliberatey removed it in the doc.

looks like at some point Salvatore added a new sentinel doc that didn't include is-master-down-by-addr in the list of commands, and much later he removed the old doc that did have it.
173155f
03c4ad5
however, even the new doc does refer to is-master-down-by-addr (it is mentioned a few times, just not listed), so i think we can add it back.

i see that:
pending-scripts exists since 2012 (3f194a9)
info-cache exists since 2014 (f8c73e3)
simulate-failure exists since 2015 (fb3af75)
so it seems to me that the later two (being more recent addition) might have just been overlooked. but i'd guess that the first one isn't listed on purpose.

@itamarhaber maybe you can think of some old-timer that can shed more light on this?

Regrettably, I can't think of someone like that.

ok. so let's do what we think is right.
@hwware wanna refresh this PR and decide which of the unlisted commands should be listed and which ones can be skipped?

@hwware do you want to give this a final push? I promise to merge 🤞

@itamarhaber give me some time to refresh my memory and push it, Thanks

👷 Deploy request for redis-doc pending review.

Visit the deploys page to approve it

@itamarhaber Updated, please take a look, Thanks

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}