Write access required to Confluence wiki

2018-04-25 Thread Piyush Vijay
I want to open a JIRA and a KIP. Can someone please grant me the necessary permissions? My username is *piyushvijay*. Thank you Piyush Vijay

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-02 Thread Piyush Vijay
ion. Best, Piyush Piyush Vijay On Wed, May 2, 2018 at 1:33 PM, Andy Coates <a...@confluent.io> wrote: > > Perhaps there is a simpler way. It seems like the resource that people > really want to use prefixed ACLs with is topic names. Because topic nam

Re: Write access required to Confluence wiki

2018-04-26 Thread Piyush Vijay
Ping :) Piyush Vijay On Wed, Apr 25, 2018 at 7:45 PM, Piyush Vijay <piyushvij...@gmail.com> wrote: > > I want to open a JIRA and a KIP. Can someone please grant me the necessary > permissions? > My username is *piyushvijay*. > > Thank you > Piyush Vijay > >

[DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-01 Thread Piyush Vijay
There is an open question about the way to add the support in the AdminClient, which I can discuss here in more detail once everyone has taken a first look at the KIP. Looking forward to discuss the change. Best, Piyush Vijay

Permission to open a KIP

2018-04-30 Thread Piyush Vijay
Hi, Can someone please grant me permission to open a KIP? My id is piyushvijay. Thanks Piyush Vijay

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-06 Thread Piyush Vijay
Makes sense. I'll update the KIP. Does anyone have any other comments? :) Thanks Piyush Vijay On Thu, May 3, 2018 at 11:55 AM, Colin McCabe <cmcc...@apache.org> wrote: > Yeah, I guess that's a good point. It probably makes sense to support the > prefix scheme for con

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-14 Thread Piyush Vijay
); public abstract DeleteAclsResult deleteAcls(Collection filters, DeleteAclsOptions options); Thanks Piyush Vijay On Mon, May 14, 2018 at 9:26 AM, Andy Coates <a...@confluent.io> wrote: > +1 > > On 11 May 2018 at 17:14, Colin McCabe <cmcc...@apache.org> wrote: > > &g

Re: Write access required to Confluence wiki

2018-04-27 Thread Piyush Vijay
Hi, Can someone please help/guide me here? I really appreciate it. Thanks Piyush Vijay On Thu, Apr 26, 2018 at 6:37 PM, Piyush Vijay <piyushvij...@gmail.com> wrote: > Ping :) > > > Piyush Vijay > > On Wed, Apr 25, 2018 at 7:45 PM, Piyush Vijay <piyushvij...@gmail.c

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
Hi Andy, I still have some minor changes left to the KIP. I'll make them in the morning. I'm sorry I got caught up in some other things today. But that would still give us 72 hours before the deadline :) Thanks Piyush Vijay On Thu, May 17, 2018 at 1:27 PM, Andy Coates <a...@confluent

[VOTE] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
because resources like consumer groups don't have defined naming convention and can have '*' in their names. Please take a look. Thanks Piyush Vijay

Re: [VOTE] KIP-290: Support for wildcard suffixed ACLs

2018-05-19 Thread Piyush Vijay
Hi Ron, I've incorporated both of your suggestions. Thanks Piyush Vijay On Sat, May 19, 2018 at 4:07 AM, Ron Dagostino <rndg...@gmail.com> wrote: > Hi Piyush. I think it would be better to match the flag in kafka-acls.sh > to the new enum. Instead of stating “–wildcard-suffi

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-17 Thread Piyush Vijay
Hi Rajini/Colin, I will remove the wildcard principals from the scope for now, updating KIP right now and will open it for vote. Thanks Piyush Vijay On Thu, May 17, 2018 at 6:59 AM, Rajini Sivaram <rajinisiva...@gmail.com> wrote: > Hi Piyush, > > I have added a PR (https://gi

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-15 Thread Piyush Vijay
this change? This will be a one-time breaking change in APIs marked as "Evolving", but makes sense in the long term? Am I missing something here? Piyush Vijay On Tue, May 15, 2018 at 9:01 AM, Colin McCabe <cmcc...@apache.org> wrote: > Hi Piyush, > > I think AclBinding sh

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-17 Thread Piyush Vijay
> I think that's the only change we need to admin client > > regards, > Colin > > > On Thu, May 17, 2018, at 08:55, Piyush Vijay wrote: > > Hi Rajini/Colin, > > > > I will remove the wildcard principals from the scope for now, updating > KIP > > righ

Re: [VOTE] KIP-290: Support for wildcard suffixed ACLs

2018-05-21 Thread Piyush Vijay
Thanks for the +1s. I'll send out a PR shortly. Thanks Piyush Vijay On Mon, May 21, 2018 at 8:59 AM, Guozhang Wang <wangg...@gmail.com> wrote: > Thanks for the KIP, +1 from me (binding). > > > Guozhang > > On Mon, May 21, 2018 at 5:22 AM, Damian Guy <damian

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
Ready to review. Let me know if something looks missing or not clear. Thanks Piyush Vijay On Fri, May 18, 2018 at 12:54 PM, Piyush Vijay <piyushvij...@gmail.com> wrote: > Andy, > > 1. Updated the KIP about need for getMatchingAcls(). Basically, it's > required to add an

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
s cut off for KIPs. > This doesn't leave much room for resolving any issues any committers find. > Also, we now require at least three committers to review this KIP today > _and_ find no issues if we're to get this KIP accepted. > > Thanks, > > Andy > > On 18 May 2018 at 01:21, Piyush V

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
about moving everything to a v2 path. We might still have to support snowflakes but I will this better. I'm giving it a final read. I'll update here once I think it's ready. Thanks Piyush Vijay On Fri, May 18, 2018 at 12:18 PM, Piyush Vijay <piyushvij...@gmail.com> wrote: > On

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-17 Thread Piyush Vijay
It's still not complete. I'll drop a message here when I'm done with the updates. Thanks Piyush Vijay On Thu, May 17, 2018 at 12:04 PM, Andy Coates <a...@confluent.io> wrote: > Thanks for the update to the KIP Piyush! > > Reading it through again, I've a couple of questi

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
Piyush Vijay On Fri, May 18, 2018 at 1:55 PM, Andy Coates <a...@confluent.io> wrote: > Hey Piyush, > > Thanks for getting this in! :D > > About to read now. But just quickly... > > 1. I'll read up on the need for getMatchingAcls - but just playing devils > advoca

Re: [DISCUSS] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
, will start voting thread now. Thanks everyone for the valuable feedback. Piyush Vijay Piyush Vijay On Fri, May 18, 2018 at 6:07 PM, Andy Coates <a...@confluent.io> wrote: > Hi Piyush, > > We're fast approaching the KIP deadline. Are you actively working on this? > If you're n

[VOTE] KIP-290: Support for wildcard suffixed ACLs

2018-05-18 Thread Piyush Vijay
because resources like consumer groups don't have defined naming convention and can have '*' in their names. Please take a look. Thanks Piyush Vijay

[jira] [Created] (KAFKA-4185) Abstract out password verifier in SaslServer as an injectable dependency

2016-09-16 Thread Piyush Vijay (JIRA)
Piyush Vijay created KAFKA-4185: --- Summary: Abstract out password verifier in SaslServer as an injectable dependency Key: KAFKA-4185 URL: https://issues.apache.org/jira/browse/KAFKA-4185 Project: Kafka

[jira] [Commented] (KAFKA-4185) Abstract out password verifier in SaslServer as an injectable dependency

2016-09-16 Thread Piyush Vijay (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-4185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15497494#comment-15497494 ] Piyush Vijay commented on KAFKA-4185: - [~junrao], [~gwenshap], [~ijuma] Any thoughts? > Abstract

[jira] [Commented] (KAFKA-4185) Abstract out password verifier in SaslServer as an injectable dependency

2016-09-28 Thread Piyush Vijay (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-4185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15530580#comment-15530580 ] Piyush Vijay commented on KAFKA-4185: - [~ecomar], even though I agree that procedure to add new

[jira] [Commented] (KAFKA-4185) Abstract out password verifier in SaslServer as an injectable dependency

2016-10-12 Thread Piyush Vijay (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-4185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570070#comment-15570070 ] Piyush Vijay commented on KAFKA-4185: - comments [~ecomar], [~ijuma]? > Abstract out password verif

[jira] [Created] (KAFKA-6841) Add support for wildcard suffixed ACLs

2018-05-01 Thread Piyush Vijay (JIRA)
Piyush Vijay created KAFKA-6841: --- Summary: Add support for wildcard suffixed ACLs Key: KAFKA-6841 URL: https://issues.apache.org/jira/browse/KAFKA-6841 Project: Kafka Issue Type: New Feature