+1 (non-binding)

- Downloaded all the resources
- Validated checksum hash
- Build the source with Maven and jdk8
- Build Flink master with new flink-shaded and check that all the tests are
passing

one minor thing that I noticed during releasing: for ci it uses maven 3.8.6
at the same time for release profile there is an enforcement plugin to
check that maven version is less than 3.3
I created a jira issue[1] for that
i made the release with 3.2.5 maven version (I suppose previous version was
also done with 3.2.5 because of same issue)

[1] https://issues.apache.org/jira/browse/FLINK-33703

On Wed, Nov 29, 2023 at 11:41 AM Matthias Pohl <matthias.p...@aiven.io>
wrote:

> +1 (binding)
>
> * Downloaded all resources
> * Extracts sources and compilation on these sources
> * Diff of git tag checkout with downloaded sources
> * Verifies SHA512 checksums & GPG certification
> * Checks that all POMs have the right expected version
> * Generated diffs to compare pom file changes with NOTICE files: Nothing
> suspicious found except for a minor (non-blocking) typo [1]
>
> Thanks for driving this effort, Sergey. :)
>
> [1] https://github.com/apache/flink-shaded/pull/126/files#r1409080162
>
> On Wed, Nov 29, 2023 at 10:25 AM Rui Fan <1996fan...@gmail.com> wrote:
>
>> Sorry, it's non-binding.
>>
>> On Wed, Nov 29, 2023 at 5:19 PM Rui Fan <1996fan...@gmail.com> wrote:
>>
>> > Thanks Matthias for the clarification!
>> >
>> > After I import the latest KEYS, it works fine.
>> >
>> > +1(binding)
>> >
>> > - Validated checksum hash
>> > - Verified signature
>> > - Verified that no binaries exist in the source archive
>> > - Build the source with Maven and jdk8
>> > - Verified licenses
>> > - Verified web PRs, and left a comment
>> >
>> > Best,
>> > Rui
>> >
>> > On Wed, Nov 29, 2023 at 5:05 PM Matthias Pohl
>> > <matthias.p...@aiven.io.invalid> wrote:
>> >
>> >> The key is the last key in the KEYS file. It's just having a different
>> >> format with spaces being added (due to different gpg versions?): F752
>> 9FAE
>> >> 2481 1A5C 0DF3  CA74 1596 BBF0 7268 35D8
>> >>
>> >> On Wed, Nov 29, 2023 at 9:41 AM Rui Fan <1996fan...@gmail.com> wrote:
>> >>
>> >> > Hey Sergey,
>> >> >
>> >> > Thank you for driving this release.
>> >> >
>> >> > I try to check this signature, the whole key is
>> >> > F7529FAE24811A5C0DF3CA741596BBF0726835D8,
>> >> > it matches your 1596BBF0726835D8, but I cannot
>> >> > find it from the Flink KEYS[1].
>> >> >
>> >> > Please correct me if my operation is wrong, thanks~
>> >> >
>> >> > [1] https://dist.apache.org/repos/dist/release/flink/KEYS
>> >> >
>> >> > Best,
>> >> > Rui
>> >> >
>> >> >
>> >> > On Wed, Nov 29, 2023 at 6:09 AM Sergey Nuyanzin <snuyan...@gmail.com
>> >
>> >> > wrote:
>> >> >
>> >> > > Hi everyone,
>> >> > > Please review and vote on the release candidate #1 for the version
>> >> 18.0,
>> >> > as
>> >> > > follows:
>> >> > > [ ] +1, Approve the release
>> >> > > [ ] -1, Do not approve the release (please provide specific
>> comments)
>> >> > >
>> >> > >
>> >> > > The complete staging area is available for your review, which
>> >> includes:
>> >> > > * JIRA release notes [1],
>> >> > > * the official Apache source release to be deployed to
>> >> dist.apache.org
>> >> > > [2],
>> >> > > which are signed with the key with fingerprint 1596BBF0726835D8
>> [3],
>> >> > > * all artifacts to be deployed to the Maven Central Repository [4],
>> >> > > * source code tag "release-18.0-rc1" [5],
>> >> > > * website pull request listing the new release [6].
>> >> > >
>> >> > > The vote will be open for at least 72 hours. It is adopted by
>> majority
>> >> > > approval, with at least 3 PMC affirmative votes.
>> >> > >
>> >> > > Thanks,
>> >> > > Sergey
>> >> > >
>> >> > > [1]
>> https://issues.apache.org/jira/projects/FLINK/versions/12353081
>> >> > > [2]
>> >> https://dist.apache.org/repos/dist/dev/flink/flink-shaded-18.0-rc1
>> >> > > [3] https://dist.apache.org/repos/dist/release/flink/KEYS
>> >> > > [4]
>> >> > >
>> >>
>> https://repository.apache.org/content/repositories/orgapacheflink-1676/
>> >> > > [5]
>> >> https://github.com/apache/flink-shaded/releases/tag/release-18.0-rc1
>> >> > > [6] https://github.com/apache/flink-web/pull/701
>> >> > >
>> >> >
>> >>
>> >
>>
>

-- 
Best regards,
Sergey

Reply via email to