cli: -stale flag on operator debug command is never utilised

[jrasell]

Nomad version

main branch at e16a35167bf1430674e88658a6a31770b11ccbde

Issue

The nomad operator debug command includes a stale flag to allow toggling of the API query parameters, however, it is never used. The command should therefore be updated to take advantage of this flag and the behaviour it allows.

[schmichael]

debug should be stale by default if possible. Since we query each server it's rather silly not to use -stale because otherwise we end up always returning the leader's view of the world.

I think basically all of our APIs should be stale by default, but that ship has sailed.

[tgross]

Preliminary pass at this: it looks like many of the api.Agent package methods don't accept query parameters (which we'd need for the CLI to set the ?stale query parameter). Probably because most of those APIs don't need the stale parameter because they're talking directly to the agent. But it looks like some of those APIs may have grown features over time (and/or were secured with ACLs) that require talking to the server(s), and that's why this is all broke. I'll find the minimum set of api functions that we need to serve operator debug just so that we're not reworking the whole package to fix this.

[tgross]

I've opened #11678 with the bug fix.

[github-actions]

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.