Adding precautionary check for DockerHub registry availability in dev container cli

[Kaniska244]

Ref: https://github.com/github/codespaces/issues/22031

Description of changes:

• The above codespace availability incident was created to address the multiple outages noticed in DockerHub registry which caused downtime for codepaces as devcontainer up and build commands injects syntax directive # syntax=docker/dockerfile:1.4 which forces devcontainer cli to download docker/dockerfile:1.4 parser image from DockerHub registry. This change was added in PR with #syntax directives + buildkit + user namespace remapping by auto-injecting the #syntax directive in dev container build. This is not required for docker engine version v23 onwards as the default moby buildkit version includes docker/dockerfile:1.4 or higher version of the parser. As part of this PR adding precautionary check to call DockerHub registry URL and confirm the availability before auto-injecting the #syntax directive.

Changelog:

• Changed src/spec-node/utils.ts by changing the faulty error logging in retry function.
• Added a new precautionary check as described above, in src/spec-node/containerFeatures.ts file.

Checklist:

• All checks are passed.

[sam-byng]

LGTM - I'm happy with this, if we've done sufficient testing

We could have made a general utility function in utils.ts to check the dockerhub auth/ registry access status, and placed the image name/version as variables, but i'm happy with this, can extend this in future if we need.

question: What testing have you done here? Can you update the PR description to state this?

[sam-byng]

Thanks - looks better now ✅