cli/command/system/runInfo: set client API defaults, and negotiated API version when available

[thaJeztah]

• relates to cmd/docker: areFlagsSupported: don't Ping if not needed #4506
• relates to Docker cli slows bash init #4505

cli/command/system/runInfo: set negotiated API version when available

The current logic was ignoring (e.g.) --format=json formats, and was
only setting the negotiated API version if no format was specified.

While we do want to avoid calling the API if possible, we do already
check if the given template requires a server connection, so let's
move updating the API version to that block.

cli/command/system/newClientVersion: initialize with default API version

Set the APIVersion and DefaultAPIVersion fields to the default version,
as that's the version the client assumes without making a API connection
to do version negotiation.

⚠️ One change worth mentioning is that this means that the API version will
differ, depending on the format:

If no server information is fetched:

docker info --format='{{ json .ClientInfo }}' | jq .ApiVersion
"1.44"

If server information is fetched:

docker info --format='{{ json .}}' | jq .ClientInfo.ApiVersion
"1.43"

An alternative could be to leave the ApiVersion field empty if no
negotiation took place.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 59.05%. Comparing base (fc99fe2) to head (c0482fd).

Additional details and impacted files

@@ Coverage Diff @@## master #4507 +/- ##
==========================================
- Coverage 59.05% 59.05% -0.01% 
==========================================
Files 357 357 Lines 29843 29846 +3 ==========================================
+ Hits 17624 17625 +1 - Misses 11241 11243 +2 
Partials 978 978 

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[vvoland]

If no server information is fetched:

docker info --format='{{ json .ClientInfo }}' | jq .ApiVersion
"1.44"

If server information is fetched:

docker info --format='{{ json .}}' | jq .ClientInfo.ApiVersion
"1.43"

An alternative could be to leave the ApiVersion field empty if no negotiation took place.

Hmm not sure about this one.

On one side it's weird for the version to be different depending on the format, on the other side side being empty is also not great.

I think leaving it empty if no negotiation took place would be slightly more useful:

• It's already that way so there's no existing expectation that it's always non-empty
• Empty value is better than misleading value as it's easier to notice
• The CLI APIVersion that's not impacted by the server negotiation is already there in DefaultAPIVersion and docker version --format='{{ json .Client }}' | jq .ApiVersion