Github user joewitt commented on the issue:
https://github.com/apache/nifi/pull/1422
@trixpan thanks for the contrib! @jfrazee thanks for the review. Merging
based on your findings. Running a quick build. Thanks
---
If your project is set up for it, you can reply to this email an
Github user jfrazee commented on the issue:
https://github.com/apache/nifi/pull/1422
LGTM +1
Verified builds on Amazon Linux w/ and w/o -Prpm and checked rpm vs. tar.gz
and .zip installs to verify correct bootstrap.conf.
---
If your project is set up for it, you can reply to
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1422
@jfrazee should be ready to review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled a
Github user jfrazee commented on the issue:
https://github.com/apache/nifi/pull/1422
@trixpan Starting review of this right now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1422
@fidget83 and @jfrazee mind having a look and let me know if it addresses
the issue?
Cheers
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1422
Reviewers:
Key here is that bootstrap.conf with RPM profile should result on
`run.as=nifi` while vanilla build should result in `run.as=`
(naturally assuming nothing else breaks)
--
