Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools

[Jason Gustafson <ja...@confluent.io.INVALID>]

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 <cm...@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.maison@gmail.com
> > > >
> > > > wrote:
> > > >
> > > > > +1 (binding)
> > > > > Thanks Colin
> > > > >
> > > > > On Tue, May 19, 2020 at 10:57 AM Manikumar <
> manikumar.reddy@gmail.com>
> > > > > wrote:
> > > > > >
> > > > > > +1 (binding)
> > > > > >
> > > > > > Thanks for the KIP
> > > > > >
> > > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot <
> djacot@confluent.io>
> > > > > wrote:
> > > > > >
> > > > > > > +1 (non-binding).
> > > > > > >
> > > > > > > Thanks for the KIP.
> > > > > > >
> > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang <
> wangguoz@gmail.com
> > > >
> > > > > wrote:
> > > > > > >
> > > > > > > > +1.
> > > > > > > >
> > > > > > > > Thanks Colin!
> > > > > > > >
> > > > > > > > Guozhang
> > > > > > > >
> > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe <
> cmccabe@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
> > > > > > > >
> > > > > > >
> > > > >
> > > >
> > >
> >
>