I opened https://github.com/apache/grails-core/pull/16223 to address these concerns. I also made a discovery about tests not running so I've combined that change in that PR. If people want me to put that in a separate PR, I'm ok with that.
On Tue, Aug 25, 2026 at 10:04 AM James Daugherty <[email protected]> wrote: > > 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 > > >
