Fixed Docker <unknown> image_id returned from parser using recent BuildKit on Windows (#21000)

The regex we used seems to be outdated, as `writing image sha256:...` now has a trailing duration before `done`.
Using the same fix as #20693 by removing the `done` from the regex.

Author: sureshjoshi

docs/notes/2.23.x.md

@@ -51,6 +51,9 @@ Nodejs processes configured with `extra_env_vars`, e.g.
 now supports extending the `PATH` variable of such processes. Passing `extra_env_vars=["PATH=/usr/bin"]` was previously
 silently ignored.
 
+#### Docker
+
+Fixed a bug where the internal Docker BuildKit parser would return `<unknown> image_id` if the BuildKit output used step durations.
 
 ### Plugin API changes
 

src/python/pants/backend/docker/goals/package_image.py

@@ -523,7 +523,7 @@ def parse_image_id_from_docker_build_output(docker: DockerBinary, *outputs: byte
             "|".join(
                 (
                     # BuildKit output.
-                    r"(writing image (?P<digest>sha256:\S+) done)",
+                    r"(writing image (?P<digest>sha256:\S+))",
                     # BuildKit with containerd-snapshotter output.
                     r"(exporting manifest list (?P<manifest_list>sha256:\S+))",
                     # Docker output.

src/python/pants/backend/docker/goals/package_image_test.py

@@ -1625,7 +1625,7 @@ def test_get_context_root(
                 """
             ),
         ),
-        # Buildkit
+        # Buildkit without step duration
         (
             DockerBinary("/bin/docker", "1234", is_podman=False),
             "sha256:7805a7da5f45a70bb9e47e8de09b1f5acd8f479dda06fb144c5590b9d2b86dd7",
@@ -1648,6 +1648,25 @@ def test_get_context_root(
             ),
             "",
         ),
+        # Buildkit with step duration
+        (
+            DockerBinary("/bin/docker", "1234", is_podman=False),
+            "sha256:7805a7da5f45a70bb9e47e8de09b1f5acd8f479dda06fb144c5590b9d2b86dd7",
+            dedent(
+                """\
+                #5 [2/2] RUN sleep 1
+                #5 DONE 1.1s
+
+                #6 exporting to image
+                #6 exporting layers
+                #6 exporting layers 0.7s done
+                #6 writing image sha256:7805a7da5f45a70bb9e47e8de09b1f5acd8f479dda06fb144c5590b9d2b86dd7 0.0s done
+                #6 naming to docker.io/library/my-docker-image:latest 0.1s done
+                #6 DONE 1.1s
+                """
+            ),
+            "",
+        ),
         # Buildkit with containerd-snapshotter 0.12.1
         (
             DockerBinary("/bin/docker", "1234", is_podman=False),