Great. In that case we can fix the license issue retrospectively. I have
created a JIRA for it https://issues.apache.org/jira/browse/KAFKA-16278 and
also updated the release process (which redirects to
https://issues.apache.org/jira/browse/KAFKA-12622) to check for the correct
license in both the kafka binaries.

I am +1 (binding) assuming Mickael's concerns about update notes to 3.7 are
addressed before release.

--
Divij Vaidya



On Mon, Feb 19, 2024 at 6:08 PM Mickael Maison <mickael.mai...@gmail.com>
wrote:

> Hi,
>
> I agree with Josep, I don't think it's worth making a new RC just for this.
>
> Thanks Stanislav for sharing the test results. The last thing holding
> me from casting my vote is the missing upgrade notes for 3.7.0.
>
> Thanks,
> Mickael
>
>
>
> On Mon, Feb 19, 2024 at 4:28 PM Josep Prat <josep.p...@aiven.io.invalid>
> wrote:
> >
> > I think I remember finding a similar problem (NOTICE_binary) and it
> didn't
> > qualify for an extra RC
> >
> > Best,
> >
> > On Mon, Feb 19, 2024 at 3:44 PM Divij Vaidya <divijvaidy...@gmail.com>
> > wrote:
> >
> > > I have performed the following checks. The only thing I would like to
> call
> > > out is the missing licenses before providing a vote. How do we want
> > > to proceed on this? What have we done in the past? (Creating a new RC
> is
> > > overkill IMO for this license issue).
> > >
> > > ## License check
> > >
> > > Test: Validate license of dependencies for both 2.12 & 2.13 binary.
> > > Result: Missing license for some scala* libraries specifically for
> 2.12.
> > > Seems like we have been missing these licenses for quite some version
> now.
> > >
> > > ```
> > > for f in $(ls libs | grep -v "^kafka\|connect\|trogdor"); do if ! grep
> -q
> > > ${f%.*} LICENSE; then echo "${f%.*} is missing in license file"; fi;
> done
> > > scala-collection-compat_2.12-2.10.0 is missing in license file
> > > scala-java8-compat_2.12-1.0.2 is missing in license file
> > > scala-library-2.12.18 is missing in license file
> > > scala-logging_2.12-3.9.4 is missing in license file
> > > scala-reflect-2.12.18 is missing in license file
> > > ```
> > >
> > > ## Long running tests for memory leak (on ARM machine with zstd)
> > >
> > > Test: Run produce/consume for a few hours and verify no gradual
> increase in
> > > heap.
> > > Result: No heap increase observed. The overall CPU utilization is lower
> > > compared to 3.5.1.
> > >
> > > ## Verify system test results
> > >
> > > Test: Spot check the results of system tests.
> > > Result: I have verified that the system tests are passing across
> different
> > > runs.
> > >
> > > --
> > > Divij Vaidya
> > >
> > >
> > >
> > > On Sun, Feb 18, 2024 at 2:50 PM Stanislav Kozlovski
> > > <stanis...@confluent.io.invalid> wrote:
> > >
> > > > The latest system test build completed successfully -
> > > >
> > > >
> > >
> https://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/system-test-kafka-branch-builder--1708250728--apache--3.7--02197edaaa/2024-02-18--001./2024-02-18--001./report.html
> > > >
> > > > *System tests are therefore all good*. We just have some flakes
> > > >
> > > > On Sun, Feb 18, 2024 at 10:45 AM Stanislav Kozlovski <
> > > > stanis...@confluent.io>
> > > > wrote:
> > > >
> > > > > The upgrade test passed ->
> > > > >
> > > >
> > >
> https://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/system-test-kafka-branch-builder--1708103771--apache--3.7--bb6990114b/2024-02-16--001./2024-02-16--001./report.html
> > > > >
> > > > > The replica verification test succeeded in ZK mode, but failed in
> > > > > ISOLATED_KRAFT. It just seems to be very flaky.
> > > > >
> > > >
> > >
> https://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/system-test-kafka-branch-builder--1708100119--apache--3.7--bb6990114b/2024-02-16--001./2024-02-16--001./report.html
> > > > >
> > > > > Scheduling another run in
> > > > >
> > >
> https://jenkins.confluent.io/job/system-test-kafka-branch-builder/6062/
> > > > >
> > > > > On Fri, Feb 16, 2024 at 6:39 PM Stanislav Kozlovski <
> > > > > stanis...@confluent.io> wrote:
> > > > >
> > > > >> Thanks all for the help in verifying.
> > > > >>
> > > > >> I have updated
> > > > >>
> > > >
> > >
> https://gist.github.com/stanislavkozlovski/820976fc7bfb5f4dcdf9742fd96a9982
> > > > >> with the system tests.
> > > > >> There were two builds ran, and across those - the following tests
> > > failed
> > > > >> two times in a row:
> > > > >>
> > > > >>
> > > > >>
> > > >
> > >
> *kafkatest.tests.tools.replica_verification_test.ReplicaVerificationToolTest#test_replica_lagsArguments:{
> > > > >> "metadata_quorum": "ZK"}*Fails with the same error of
> > > > >> *`TimeoutError('Timed out waiting to reach non-zero number of
> replica
> > > > >> lags.')`*
> > > > >> I have scheduled a re-run of this specific test here ->
> > > > >>
> > > https://jenkins.confluent.io/job/system-test-kafka-branch-builder/6057
> > > > >>
> > > > >>
> *kafkatest.tests.core.upgrade_test.TestUpgrade#test_upgradeArguments:{
> > > > >> "compression_types": [ "zstd" ], "from_kafka_version": "2.4.1",
> > > > >> "to_message_format_version": null}*
> > > > >> Fails with the same error of
> > > > >> *`TimeoutError('Producer failed to produce messages for 20s.')`*
> > > > >>
> *kafkatest.tests.core.upgrade_test.TestUpgrade#test_upgradeArguments:{
> > > > >> "compression_types": [ "lz4" ], "from_kafka_version": "3.0.2",
> > > > >> "to_message_format_version": null}*
> > > > >> Fails with the same error of *`TimeoutError('Producer failed to
> > > produce
> > > > >> messages for 20s.')`*
> > > > >>
> > > > >> I have scheduled a re-run of this test here ->
> > > > >>
> > >
> https://jenkins.confluent.io/job/system-test-kafka-branch-builder/6058/
> > > > >>
> > > > >> On Fri, Feb 16, 2024 at 12:15 PM Vedarth Sharma <
> > > > vedarth.sha...@gmail.com>
> > > > >> wrote:
> > > > >>
> > > > >>> Hey Stanislav,
> > > > >>>
> > > > >>> Thanks for the release candidate.
> > > > >>>
> > > > >>> +1 (non-binding)
> > > > >>>
> > > > >>> I tested and verified the docker image artifact
> > > > apache/kafka:3.7.0-rc4:-
> > > > >>> - verified create topic, produce messages and consume messages
> flow
> > > > when
> > > > >>> running the docker image with
> > > > >>>     - default configs
> > > > >>>     - configs provided via env variables
> > > > >>>     - configs provided via file input
> > > > >>> - verified the html documentation for docker image.
> > > > >>> - ran the example docker compose files successfully.
> > > > >>>
> > > > >>> All looks good for the docker image artifact!
> > > > >>>
> > > > >>> Thanks and regards,
> > > > >>> Vedarth
> > > > >>>
> > > > >>>
> > > > >>> On Thu, Feb 15, 2024 at 10:58 PM Mickael Maison <
> > > > >>> mickael.mai...@gmail.com>
> > > > >>> wrote:
> > > > >>>
> > > > >>> > Hi Stanislav,
> > > > >>> >
> > > > >>> > Thanks for running the release.
> > > > >>> >
> > > > >>> > I did the following testing:
> > > > >>> > - verified the check sums and signatures
> > > > >>> > - ran ZooKeeper and KRaft quickstarts with Scala 2.13 binaries
> > > > >>> > - ran a successful migration from ZooKeeper to KRaft
> > > > >>> >
> > > > >>> > We seem to be missing the upgrade notes for 3.7.0 in the docs.
> See
> > > > >>> > https://kafka.apache.org/37/documentation.html#upgrade that
> still
> > > > >>> > points to 3.6.0
> > > > >>> > Before voting I'd like to see results from the system tests
> too.
> > > > >>> >
> > > > >>> > Thanks,
> > > > >>> > Mickael
> > > > >>> >
> > > > >>> > On Thu, Feb 15, 2024 at 6:06 PM Andrew Schofield
> > > > >>> > <andrew_schofield_j...@outlook.com> wrote:
> > > > >>> > >
> > > > >>> > > +1 (non-binding). I used the staged binaries with Scala
> 2.13. I
> > > > tried
> > > > >>> > the new group coordinator
> > > > >>> > > and consumer group protocol which is included with the Early
> > > Access
> > > > >>> > release of KIP-848.
> > > > >>> > > Also verified the availability of the new APIs. All working
> as
> > > > >>> expected.
> > > > >>> > >
> > > > >>> > > Thanks,
> > > > >>> > > Andrew
> > > > >>> > >
> > > > >>> > > > On 15 Feb 2024, at 05:07, Paolo Patierno <
> > > > paolo.patie...@gmail.com
> > > > >>> >
> > > > >>> > wrote:
> > > > >>> > > >
> > > > >>> > > > +1 (non-binding). I used the staged binaries with Scala
> 2.13
> > > and
> > > > >>> mostly
> > > > >>> > > > focused on the ZooKeeper to KRaft migration with multiple
> > > tests.
> > > > >>> > Everything
> > > > >>> > > > works fine.
> > > > >>> > > >
> > > > >>> > > > Thanks
> > > > >>> > > > Paolo
> > > > >>> > > >
> > > > >>> > > > On Mon, 12 Feb 2024, 22:06 Jakub Scholz, <ja...@scholz.cz>
> > > > wrote:
> > > > >>> > > >
> > > > >>> > > >> +1 (non-binding). I used the staged binaries with Scala
> 2.13
> > > and
> > > > >>> the
> > > > >>> > staged
> > > > >>> > > >> Maven artifacts to run my tests. All seems to work fine.
> > > Thanks.
> > > > >>> > > >>
> > > > >>> > > >> Jakub
> > > > >>> > > >>
> > > > >>> > > >> On Fri, Feb 9, 2024 at 4:20 PM Stanislav Kozlovski
> > > > >>> > > >> <stanis...@confluent.io.invalid> wrote:
> > > > >>> > > >>
> > > > >>> > > >>> Hello Kafka users, developers and client-developers,
> > > > >>> > > >>>
> > > > >>> > > >>> This is the second candidate we are considering for
> release
> > > of
> > > > >>> Apache
> > > > >>> > > >> Kafka
> > > > >>> > > >>> 3.7.0.
> > > > >>> > > >>>
> > > > >>> > > >>> Major changes include:
> > > > >>> > > >>> - Early Access to KIP-848 - the next generation of the
> > > consumer
> > > > >>> > rebalance
> > > > >>> > > >>> protocol
> > > > >>> > > >>> - Early Access to KIP-858: Adding JBOD support to KRaft
> > > > >>> > > >>> - KIP-714: Observability into Client metrics via a
> > > standardized
> > > > >>> > interface
> > > > >>> > > >>>
> > > > >>> > > >>> Release notes for the 3.7.0 release:
> > > > >>> > > >>>
> > > > >>> > > >>>
> > > > >>> > > >>
> > > > >>> >
> > > > >>>
> > > >
> > >
> https://home.apache.org/~stanislavkozlovski/kafka-3.7.0-rc4/RELEASE_NOTES.html
> > > > >>> > > >>>
> > > > >>> > > >>> *** Please download, test and vote by Thursday, February
> > > 15th,
> > > > >>> 9AM
> > > > >>> > PST
> > > > >>> > > >> ***
> > > > >>> > > >>>
> > > > >>> > > >>> Kafka's KEYS file containing PGP keys we use to sign the
> > > > release:
> > > > >>> > > >>> https://kafka.apache.org/KEYS
> > > > >>> > > >>>
> > > > >>> > > >>> * Release artifacts to be voted upon (source and binary):
> > > > >>> > > >>>
> https://home.apache.org/~stanislavkozlovski/kafka-3.7.0-rc4/
> > > > >>> > > >>>
> > > > >>> > > >>> * Docker release artifact to be voted upon:
> > > > >>> > > >>> apache/kafka:3.7.0-rc4
> > > > >>> > > >>>
> > > > >>> > > >>> * Maven artifacts to be voted upon:
> > > > >>> > > >>>
> > > > >>> >
> > > >
> https://repository.apache.org/content/groups/staging/org/apache/kafka/
> > > > >>> > > >>>
> > > > >>> > > >>> * Javadoc:
> > > > >>> > > >>>
> > > > >>>
> https://home.apache.org/~stanislavkozlovski/kafka-3.7.0-rc4/javadoc/
> > > > >>> > > >>>
> > > > >>> > > >>> * Tag to be voted upon (off 3.7 branch) is the 3.7.0 tag:
> > > > >>> > > >>> https://github.com/apache/kafka/releases/tag/3.7.0-rc4
> > > > >>> > > >>>
> > > > >>> > > >>> * Documentation:
> > > > >>> > > >>> https://kafka.apache.org/37/documentation.html
> > > > >>> > > >>>
> > > > >>> > > >>> * Protocol:
> > > > >>> > > >>> https://kafka.apache.org/37/protocol.html
> > > > >>> > > >>>
> > > > >>> > > >>> * Successful Jenkins builds for the 3.7 branch:
> > > > >>> > > >>>
> > > > >>> > > >>> Unit/integration tests: I am in the process of running
> and
> > > > >>> analyzing
> > > > >>> > > >> these.
> > > > >>> > > >>> System tests: I am in the process of running these.
> > > > >>> > > >>>
> > > > >>> > > >>> Expect a follow-up over the weekend
> > > > >>> > > >>>
> > > > >>> > > >>> * Successful Docker Image Github Actions Pipeline for 3.7
> > > > branch:
> > > > >>> > > >>> Docker Build Test Pipeline:
> > > > >>> > > >>> https://github.com/apache/kafka/actions/runs/7845614846
> > > > >>> > > >>>
> > > > >>> > > >>> /**************************************
> > > > >>> > > >>>
> > > > >>> > > >>> Best,
> > > > >>> > > >>> Stanislav
> > > > >>> > > >>>
> > > > >>> > > >>
> > > > >>> > >
> > > > >>> >
> > > > >>>
> > > > >>
> > > > >>
> > > > >> --
> > > > >> Best,
> > > > >> Stanislav
> > > > >>
> > > > >
> > > > >
> > > > > --
> > > > > Best,
> > > > > Stanislav
> > > > >
> > > >
> > > >
> > > > --
> > > > Best,
> > > > Stanislav
> > > >
> > >
> >
> >
> > --
> > [image: Aiven] <https://www.aiven.io>
> >
> > *Josep Prat*
> > Open Source Engineering Director, *Aiven*
> > josep.p...@aiven.io   |   +491715557497
> > aiven.io <https://www.aiven.io>   |   <
> https://www.facebook.com/aivencloud>
> >   <https://www.linkedin.com/company/aiven/>   <
> https://twitter.com/aiven_io>
> > *Aiven Deutschland GmbH*
> > Alexanderufer 3-7, 10117 Berlin
> > Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen
> > Amtsgericht Charlottenburg, HRB 209739 B
>

Reply via email to