Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Joey Frazee
I have https://github.com/apache/nifi/pull/4632 which fixes an OOME in PutAzureBlobStorage reported in https://lists.apache.org/x/thread.html/rdef82be24828277b85bdc94dc57a8fb9df6f73552daeda289c941a51%40%3Cusers.nifi.apache.org%3E It’s a pretty small change. -joey > On Jan 5, 2021, at 3:14 PM,

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Matt Burgess
I am reviewing a PR at the moment but intend to review the graph stuff right afterwards. > On Jan 5, 2021, at 5:35 PM, Mike Thomsen wrote: > > I have a PR for graph that we need to close out because part of the graph > bundle will be broken without it. > >> On Tue, Jan 5, 2021, 11:50 Mark

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Mike Thomsen
I have a PR for graph that we need to close out because part of the graph bundle will be broken without it. On Tue, Jan 5, 2021, 11:50 Mark Bean wrote: > I'd like to see this PR completed as well > https://github.com/apache/nifi/pull/4225 > > There's been discussion on it, and I don't see any

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Peter Turcsanyi
There is a bug in PropertyValue::isExpressionLanguagePresent we should fix for the next release: PropertyValue.isExpressionLanguagePresent always returns true for non-null values https://issues.apache.org/jira/browse/NIFI-8034 On Tue, Jan 5, 2021 at 5:49 PM Mark Bean wrote: > I'd like to see

Re: NiFi Error on SSL Setup for cluster UI not loading - Insufficient Permissions Untrusted proxy

2021-01-05 Thread Sushil Kumar
Hello Karthikeyan As Chris has pointed out, you would need to add node identities in authorizations.xml This is required to establish node identity with NiFi cluster. Thanks Sushil Kumar On Tue, Jan 5, 2021 at 7:52 AM MUTHUKRISHNAN, KARTHIKEYAN wrote: > Hi Team, > > Can you help us to

Re: [EXT] Using github to merge PRs and just generally easier tracking of commits..

2021-01-05 Thread Joe Witt
I dont know if you can tweak the message in github before hitting the button as a reviewer. Now that I see how to tell who merged though in any case Im less concerned personally. On Tue, Jan 5, 2021 at 10:56 AM Peter Wicks (pwicks) wrote: > Should mergers be putting the “signed-off by: …” in

Re: [EXT] Using github to merge PRs and just generally easier tracking of commits..

2021-01-05 Thread Peter Wicks (pwicks)
Should mergers be putting the “signed-off by: …” in when they Merge PR’s through Github? I haven’t merged anything in a while, but I was putting this in the comment box when I merged, and it showed up on the commit the same way it did when I did it through the CLI:

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Mark Bean
I'd like to see this PR completed as well https://github.com/apache/nifi/pull/4225 There's been discussion on it, and I don't see any further objections. Thanks, Mark On Thu, Nov 26, 2020 at 4:55 AM Pierre Villard wrote: > Hi community, > > Starting a discussion thread around the release of

Re: request to merge PRs to main

2021-01-05 Thread Nissim Shiman
Hello Nifi Devs, Could someone move https://issues.apache.org/jira/browse/NIFI-6242 into the coming 1.13.0 release.  This is a bug fix I have reviewed and tested so it is ready for a merge. It handles a case where the SEND provenance event sometimes has the incorrect value for its transitUri.

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Joe Witt
Thanks Nathan - lets do that. On Tue, Jan 5, 2021 at 9:03 AM Nathan Gough wrote: > If possible I would like to get > https://issues.apache.org/jira/browse/NIFI-7356 in for this release as > well. This enables TLS + embedded Zookeeper. Right now, the PR for TLS + > external Zookeeper is merged,

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Nathan Gough
If possible I would like to get https://issues.apache.org/jira/browse/NIFI-7356 in for this release as well. This enables TLS + embedded Zookeeper. Right now, the PR for TLS + external Zookeeper is merged, and there is an open PR for TLS + embedded Zookeeper here:

Re: NiFi Error on SSL Setup for cluster UI not loading - Insufficient Permissions Untrusted proxy

2021-01-05 Thread MUTHUKRISHNAN, KARTHIKEYAN
Hi Team, Can you help us to troubleshoot this issue ? Thanks & Regards, M.Karthikeyan. From: MUTHUKRISHNAN, KARTHIKEYAN Sent: Tuesday, December 22, 2020 7:08 PM To: dev@nifi.apache.org Cc: RANJAN, RAJIV ; RAJU, JOSEPH Subject: NiFi Error on SSL Setup for

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Joe Witt
Thanks Pierre - definitely a good time to pull together a release. Were you looking to do the RM work or do we need someone? I'm happy to do it if needed. Thanks On Tue, Jan 5, 2021 at 8:04 AM Pierre Villard wrote: > Hi all, > > Happy new year to everyone, hope you all had a great time. I'm

Re: [DISCUSS] Release of Apache NiFi 1.13.0

2021-01-05 Thread Pierre Villard
Hi all, Happy new year to everyone, hope you all had a great time. I'm bumping this thread up as I think it'd make a lot of sense to release NiFi 1.13.0. We have a ton of bug fixes and improvements that would be very useful for our community. Let's try to see if there are any blockers for a