Build succeeded.

• containerd-build-arm64 : SUCCESS in 4m 46s (non-voting)
• containerd-test-arm64 : SUCCESS in 5m 35s (non-voting)
• containerd-integration-test-arm64 : SUCCESS in 22m 07s (non-voting)

if newReference {
        if abortErr := s.store.Abort(ctx, config.Reference); abortErr != nil {
                log.G(ctx).WithError(abortErr).WithField("ref", config.Reference).Warnf("failed to delete diff upload")
        }
}

Since calling Close on a closed writer will not error, we can set the errOpen after a failed Commit.

if err := cw.Commit(ctx, 0, dgst, commitopts...); err != nil {
        if !errdefs.IsAlreadyExists(err) {
                errOpen = err
                return errors.Wrap(err, "failed to commit")
        }
}

Does it mean that we need to perform roll-back after a Commit failure?

I think in this case calling Abort is the best thing to do. Not aborting on Commit failure allows the caller to try again, but that is not relevant here. I updated the code to just use errOpen on Commit

Build succeeded.

• containerd-build-arm64 : NODE_FAILURE in 0s (non-voting)
• containerd-test-arm64 : SUCCESS in 5m 55s (non-voting)
• containerd-integration-test-arm64 : SUCCESS in 24m 00s (non-voting)