Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
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 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 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 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
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 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 > > > > > wrote: > > > > > > > +1 (binding) > > > > Thanks Colin > > > > > > > > On Tue, May 19, 2020 at 10:57 AM Manikumar > > > > wrote: > > > > > > > > > > +1 (binding) > > > > > > > > > > Thanks for the KIP > > > > > > > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot > > > > wrote: > > > > > > > > > > > +1 (non-binding). > > > > > > > > > > > > Thanks for the KIP. > > > > > > > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang > > > > > > wrote: > > > > > > > > > > > > > +1. > > > > > > > > > > > > > > Thanks Colin! > > > > > > > > > > > > > > Guozhang > > > > > > > > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe > > > > > > 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 > > > > > > > > > > > > > > > > > > > > > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
Sounds good. +1 from me. On Tue, May 19, 2020 at 5:41 PM Colin McCabe 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 > > > wrote: > > > > > +1 (binding) > > > Thanks Colin > > > > > > On Tue, May 19, 2020 at 10:57 AM Manikumar > > > wrote: > > > > > > > > +1 (binding) > > > > > > > > Thanks for the KIP > > > > > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot > > > wrote: > > > > > > > > > +1 (non-binding). > > > > > > > > > > Thanks for the KIP. > > > > > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang > > > > wrote: > > > > > > > > > > > +1. > > > > > > > > > > > > Thanks Colin! > > > > > > > > > > > > Guozhang > > > > > > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe > > > > 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 > > > > > > > > > > > > > > > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
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 > wrote: > > > +1 (binding) > > Thanks Colin > > > > On Tue, May 19, 2020 at 10:57 AM Manikumar > > wrote: > > > > > > +1 (binding) > > > > > > Thanks for the KIP > > > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot > > wrote: > > > > > > > +1 (non-binding). > > > > > > > > Thanks for the KIP. > > > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang > > wrote: > > > > > > > > > +1. > > > > > > > > > > Thanks Colin! > > > > > > > > > > Guozhang > > > > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe > > 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 > > > > > > > > > > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
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 On Tue, May 19, 2020 at 5:22 AM Mickael Maison wrote: > +1 (binding) > Thanks Colin > > On Tue, May 19, 2020 at 10:57 AM Manikumar > wrote: > > > > +1 (binding) > > > > Thanks for the KIP > > > > On Tue, May 19, 2020 at 12:29 PM David Jacot > wrote: > > > > > +1 (non-binding). > > > > > > Thanks for the KIP. > > > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang > wrote: > > > > > > > +1. > > > > > > > > Thanks Colin! > > > > > > > > Guozhang > > > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe > 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 > > > > > > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
+1 (binding) Thanks Colin On Tue, May 19, 2020 at 10:57 AM Manikumar wrote: > > +1 (binding) > > Thanks for the KIP > > On Tue, May 19, 2020 at 12:29 PM David Jacot wrote: > > > +1 (non-binding). > > > > Thanks for the KIP. > > > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang wrote: > > > > > +1. > > > > > > Thanks Colin! > > > > > > Guozhang > > > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe 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 > > > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
+1 (binding) Thanks for the KIP On Tue, May 19, 2020 at 12:29 PM David Jacot wrote: > +1 (non-binding). > > Thanks for the KIP. > > On Fri, May 15, 2020 at 12:41 AM Guozhang Wang wrote: > > > +1. > > > > Thanks Colin! > > > > Guozhang > > > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe 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 > > >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
+1 (non-binding). Thanks for the KIP. On Fri, May 15, 2020 at 12:41 AM Guozhang Wang wrote: > +1. > > Thanks Colin! > > Guozhang > > On Tue, May 12, 2020 at 3:45 PM Colin McCabe 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 >
Re: [VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
+1. Thanks Colin! Guozhang On Tue, May 12, 2020 at 3:45 PM Colin McCabe 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
[VOTE]: KIP-604: Remove ZooKeeper Flags from the Administrative Tools
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