Online DDL: 'cancelled' migration does not update completed_timestamp, leading to uncollected garbage

Similarly to https://github.com/vitessio/vitess/issues/8499, a migration marked as `cancelled` does not mark `completed_timestamp`, keeping it as `NULL`, which leads to the artifact cleanup query not collecting artifact tables for this migration, which means artifacts are not being cleaned up.

