It looks like https://github.com/apache/grails-core/pull/16192 was
merged without 2 reviewers approving. I  had also discussed this with
Scott separately, indicating I wanted to review this change given the
build / dev environment changes.   Yet, it was still merged.

I think I'm ultimately ok with the mongo changes, but I'm not ok with
the async changes.  The async changes are fixing a symptom of a
problem - not cleaning up data correctly - and we've now changed the
contract to silently fail.  So are the tests "fixed"?  Yes, but only
because the failure is silent now.  Worse, there's no indication of
the silent failure. We should fix the problem, not the symptom.

I'm a -1 due to this.

On Tue, Aug 25, 2026 at 8:10 AM Gianluca Sartori <[email protected]> wrote:
>
> +1
>
> Gianluca Sartori
> --
> https://dueuno.com
>
>
> On Tue, 25 Aug 2026 at 13:46, Mattias Reichel <[email protected]> wrote:
>
> > Hi Everyone,
> >
> > I am happy to start the VOTE thread on the second attempt to release
> > Apache Grails 8.0.0-M6!
> >
> > Release notes for the release are here:
> > https://github.com/apache/grails-core/releases/tag/v8.0.0-M6
> >
> > The tag for this release is:
> > https://github.com/apache/grails-core/releases/tag/v8.0.0-M6
> > Tag commit id: 70083df4e8bcbbc6e70dbbec4356d5d32ee6cd1d
> >
> > The artifacts to be voted on are located as follows (r86971):
> > Source release:
> > https://dist.apache.org/repos/dist/dev/grails/core/8.0.0-M6/sources
> > Binary distributions:
> > https://dist.apache.org/repos/dist/dev/grails/core/8.0.0-M6/distribution
> >
> > Release artifacts are signed with a key from the following file:
> > https://dist.apache.org/repos/dist/release/grails/KEYS
> >
> > Please vote on releasing this package as: Apache Grails 8.0.0-M6.
> >
> > Reminder on ASF release approval requirements for PMC members:
> > https://www.apache.org/legal/release-policy.html#release-approval
> >
> > Hints on validating checksums/signatures (but replace md5sum with
> > sha512sum):
> > https://www.apache.org/info/verification.html
> >
> > Details of our release process are documented at:
> > https://github.com/apache/grails-core/blob/HEAD/RELEASE.md
> >
> > As a reminder, most of our tools support an env `GRAILS_REPO_URL` so
> > that the tools can work with staged artefacts. Set it to the staging
> > repo like this: `export
> > GRAILS_REPO_URL=https://repository.apache.org/content/groups/staging`
> > <https://repository.apache.org/content/groups/staging>
> >
> > The vote is open for a minimum of 72 hours and passes if a majority of at
> > least
> > three +1 PMC votes are cast.
> >
> > [ ] +1 Release Apache Grails 8.0.0-M6
> > [ ]  0 I don't have a strong opinion about this, but I assume it's ok
> > [ ] -1 Do not release Apache Grails 8.0.0-M6 because...
> >
> > Here is my vote:
> >
> > +1 (binding)
> >
> > /Mattias
> >

Reply via email to