Hi All, I have a small addendum to this KIP. We realized that we also needed to drop support in kafka-acls.sh for modifications through Zookeeper. This was supported through the --authorizer, --authorizer-properties, and --zk-tls-config-file arguments. I have updated the KIP to include removal of these arguments. Please let me know if you have any concerns. The JIRA tracking this is https://issues.apache.org/jira/browse/KAFKA-13071.
Best, Jason On Thu, May 21, 2020 at 8:02 AM Colin McCabe <co...@cmccabe.xyz> wrote: > Hi all, > > With 4 binding +1 votes from Guozhang Wang, Manikumar, Mickael Miason, and > Jason Gustafson, and 1 non-binding vote from David Jacot, the vote passes. > > thanks, all. > Colin > > > On Wed, May 20, 2020, at 18:16, Jason Gustafson wrote: > > Sounds good. +1 from me. > > > > On Tue, May 19, 2020 at 5:41 PM Colin McCabe <cmcc...@apache.org> wrote: > > > > > On Tue, May 19, 2020, at 09:31, Jason Gustafson wrote: > > > > Hi Colin, > > > > > > > > Looks good. I just had one question. It sounds like your intent is to > > > > change kafka-configs.sh so that the --zookeeper flag is only > supported > > > for > > > > bootstrapping. I assume in the case of SCRAM that we will only make > this > > > > change after the broker API is available? > > > > > > > > Thanks, > > > > Jason > > > > > > Hi Jason, > > > > > > Yes, that's correct. We will have the SCRAM API ready by the Kafka 3.0 > > > release. > > > > > > best, > > > Colin > > > > > > > > > > > > > > On Tue, May 19, 2020 at 5:22 AM Mickael Maison < > mickael.mai...@gmail.com > > > > > > > > wrote: > > > > > > > > > +1 (binding) > > > > > Thanks Colin > > > > > > > > > > On Tue, May 19, 2020 at 10:57 AM Manikumar < > manikumar.re...@gmail.com> > > > > > wrote: > > > > > > > > > > > > +1 (binding) > > > > > > > > > > > > Thanks for the KIP > > > > > > > > > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot < > dja...@confluent.io> > > > > > wrote: > > > > > > > > > > > > > +1 (non-binding). > > > > > > > > > > > > > > Thanks for the KIP. > > > > > > > > > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang < > wangg...@gmail.com > > > > > > > > > wrote: > > > > > > > > > > > > > > > +1. > > > > > > > > > > > > > > > > Thanks Colin! > > > > > > > > > > > > > > > > Guozhang > > > > > > > > > > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe < > cmcc...@apache.org > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > Hi all, > > > > > > > > > > > > > > > > > > I'd like to start a vote on KIP-604: Remove ZooKeeper Flags > > > from > > > > > the > > > > > > > > > Administrative Tools. > > > > > > > > > > > > > > > > > > As a reminder, this KIP is for the next major release of > Kafka, > > > > > the 3.0 > > > > > > > > > release. So it won't go into the upcoming 2.6 release. > It's > > > a > > > > > pretty > > > > > > > > > small change that just removes the --zookeeper flags from > some > > > > > tools > > > > > > > and > > > > > > > > > removes a deprecated tool. We haven't decided exactly when > > > we'll > > > > > do > > > > > > > 3.0 > > > > > > > > > but I believe we will certainly want this change in that > > > release. > > > > > > > > > > > > > > > > > > The KIP does contain one small change relevant to Kafka > 2.6: > > > adding > > > > > > > > > support for --if-exists and --if-not-exists in combination > > > with the > > > > > > > > > --bootstrap-server flag. > > > > > > > > > > > > > > > > > > best, > > > > > > > > > Colin > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > -- Guozhang > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >