Fix the bug that enqueues Computations in ending stage.

[renjiezh]

The PR is to fix issue #1626

[wfa-reviewable]

This change is 

[kungfucraig]

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @emadlivani, @renjiezh, and @SanjayVas)

---

src/main/kotlin/org/wfanet/measurement/duchy/daemon/mill/MillBase.kt line 167 at r1 (raw file):

          "Failing computation due to too many failed ComputationStageAttempts.",
        )
        return

Can you add a test that exercises the bug and shows that this fixes it?

[SanjayVas]

Would you mind splitting all the bug details into a GitHub issue that's referenced by this PR?

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @emadlivani and @renjiezh)

[renjiezh]

Reviewable status: 1 of 3 files reviewed, 1 unresolved discussion (waiting on @emadlivani and @kungfucraig)

---

src/main/kotlin/org/wfanet/measurement/duchy/daemon/mill/MillBase.kt line 167 at r1 (raw file):

Previously, kungfucraig (Craig Wright) wrote…

Can you add a test that exercises the bug and shows that this fixes it?

Test cases updated. The previous test cases acknowledged and allowed the enqueue behavior for failed computation. Now they are corrected.

[SanjayVas]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @emadlivani and @kungfucraig)

[stevenwarejones]

Reviewed 1 of 1 files at r1, 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @emadlivani, @kungfucraig, and @Marco-Premier)

[kungfucraig]

Reviewed 1 of 1 files at r1, 2 of 2 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @emadlivani and @Marco-Premier)