[GitHub] nifi issue #2827: NIFI-5354 Changing ranger.version to 1.0.0 and ranger.hado...

2018-06-29 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2827 @bbende thanks Bryan I will review shortly ---

[GitHub] nifi issue #2628: NIFI-4942 Add capability for encrypt-config tool to use se...

2018-04-13 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2628 Spoke with @alopresto offline. He highlighted that is it important to maintain control of location of secure-hash.key file in order to prevent calling application from piping into a file

[GitHub] nifi issue #2628: NIFI-4942 Add capability for encrypt-config tool to use se...

2018-04-12 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2628 @alopresto ran through test cases and this works as expected. One question for you is there a way to designate an output location for the secure-hash.key file? Such as if I want to pipe

[GitHub] nifi issue #2628: NIFI-4942 Add capability for encrypt-config tool to use se...

2018-04-11 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2628 @alopresto thanks for addressing this, happy to review. It does look like Travis is failing on a ratcheck related error in nifi-toolkit-encrypt-config ---

[GitHub] nifi pull request #1997: NIFI-4164 Adding a realistic time simulator process...

2018-02-07 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1997#discussion_r166648706 --- Diff: nifi-nar-bundles/nifi-simulator-bundle/nifi-simulator-processors/src/main/java/com/apache/nifi/processors/simulator

[GitHub] nifi issue #2107: NIFI-4310 - added changes to support detection of reportin...

2017-08-25 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2107 @markap14 thank you! --- 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

[GitHub] nifi issue #2065: NIFI-4255 - added flag to allow migration of existing (sou...

2017-08-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2065 thank you @jtstorck ! --- 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

[GitHub] nifi pull request #2107: NIFI-4310 - added changes to support detection of r...

2017-08-23 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/2107 NIFI-4310 - added changes to support detection of reporting tasks and… … controller services during isEmpty flow check. Added testing scenarios Recommend the following tests

[GitHub] nifi issue #2065: NIFI-4255 - added flag to allow migration of existing (sou...

2017-08-22 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2065 spoke with @jtstorck offline and we discussed refactoring a bit to ensure that the logic for determining acls are externalized and staged within the CompletableFuture chain. Committing those

[GitHub] nifi pull request #2065: NIFI-4255 - added flag to allow migration of existi...

2017-08-14 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/2065#discussion_r132969031 --- Diff: nifi-toolkit/nifi-toolkit-zookeeper-migrator/src/main/java/org/apache/nifi/toolkit/zkmigrator/ZooKeeperMigrator.java --- @@ -269,15 +269,23

[GitHub] nifi pull request #2065: NIFI-4255 - added flag to allow migration of existi...

2017-08-12 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/2065#discussion_r132821753 --- Diff: nifi-toolkit/nifi-toolkit-zookeeper-migrator/src/main/java/org/apache/nifi/toolkit/zkmigrator/ZooKeeperMigrator.java --- @@ -269,15 +269,23

[GitHub] nifi pull request #2065: NIFI-4255 - added flag to allow migration of existi...

2017-08-07 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/2065 NIFI-4255 - added flag to allow migration of existing (source) acls Testing Recommendations: Using a secured Zookeeper instance with predefined paths/znodes use the ZK Migrator

[GitHub] nifi issue #2046: NIFI-4022 - Enabled SASL auth scheme/ACL support for Curat...

2017-08-04 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2046 @bbende thanks! --- 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 and wishes so

[GitHub] nifi issue #2019: NIFI-4032: Managed Ranger Authorizer

2017-07-24 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2019 @mcgilman with the latest commit I was able to add a new node and see the users.xml file populated. The node started successfully and I was able to access the cluster without issue

[GitHub] nifi issue #2019: NIFI-4032: Managed Ranger Authorizer

2017-07-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2019 I've worked through 3 Ranger configuration scenarios that leveraged the ldap user group provider, or the composite configurable user group provider (pairing the ldap provider with the file

[GitHub] nifi issue #2019: NIFI-4032: Managed Ranger Authorizer

2017-07-18 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/2019 @mcgilman will take a look soon --- 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

[GitHub] nifi issue #1938: NIFI-1763: Initial implementation of ConfluentSchemaRegist...

2017-07-06 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1938 @markap14 that now looks fixed. I was able to use the Confluence schema write strategy without issue. All is working as expected. +1 will merge in shortly --- If your

[GitHub] nifi issue #1938: NIFI-1763: Initial implementation of ConfluentSchemaRegist...

2017-07-06 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1938 @markap14 thanks will check it out --- 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

[GitHub] nifi issue #1938: NIFI-1763: Initial implementation of ConfluentSchemaRegist...

2017-07-06 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1938 @markap14 I’m not quite sure if I ran into a configuration problem on my part or if this is truly an issue. I’m testing the ConfluentSchemaRegistry and do have it set up in NiFi with two

[GitHub] nifi issue #1956: NIFI-4135 - added hadoop-client and enhanced Authorizers e...

2017-07-05 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1956 @bbende did add the change to order (couldn't recreate the issue). --- 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

[GitHub] nifi issue #1938: NIFI-1763: Initial implementation of ConfluentSchemaRegist...

2017-07-05 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1938 @markap14 I can try this out as well --- 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

[GitHub] nifi issue #1956: NIFI-4135 - added hadoop-client and enhanced Authorizers e...

2017-07-05 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1956 @bbende sorry for the delay (was away on vacation). Not a picky request at all; the weirdness I ran into was that the xsd could not resolve the property value if classpath was just before

[GitHub] nifi issue #1957: NIFI-4137: Add support for UTF-8 in GetFTP and PutFTP

2017-06-28 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1957 Ran tests using FileZilla on Windows 10 Server with flow containing GetFTP -> UpdateAttributes (for filename append) -> PutFTP. Worked as expected with the below use cases:

[GitHub] nifi issue #1957: NIFI-4137: Add support for UTF-8 in GetFTP and PutFTP

2017-06-28 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1957 @mattyb149 will 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

[GitHub] nifi issue #1956: NIFI-4135 - added hadoop-client and enhanced Authorizers e...

2017-06-28 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1956 @bbende would appreciate if you could review. Thanks in advance! --- 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

[GitHub] nifi pull request #1956: NIFI-4135 - added hadoop-client and enhanced Author...

2017-06-28 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1956 NIFI-4135 - added hadoop-client and enhanced Authorizers entity to su… …pport classpath for resources entry Thank you for submitting a contribution to Apache NiFi

[GitHub] nifi issue #1889: NIFI-3696 - Initial commit of ConfigMigration and FileMana...

2017-06-07 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1889 @brosander I've added updates to documentation as well as additional verbose information as recommended. please let me know if you see any additional issues --- If your project is set up

[GitHub] nifi pull request #1889: NIFI-3696 - Initial commit of ConfigMigration and F...

2017-06-07 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1889#discussion_r120722963 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1317,6 +1318,82 @@ cluster if restarted and the flow for the cluster has

[GitHub] nifi pull request #1889: NIFI-3696 - initial commit of ConfigMigration and F...

2017-06-05 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1889 NIFI-3696 - initial commit of ConfigMigration and FileManager tools Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask

[GitHub] nifi issue #1866: NIFI-3984 - upgraded version to 0.2.1

2017-05-26 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1866 @bbende can you please 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

[GitHub] nifi pull request #1866: NIFI-3984 - upgraded version to 0.2.1

2017-05-26 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1866 NIFI-3984 - upgraded version to 0.2.1 Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure

[GitHub] nifi issue #1786: NIFI-2199 & NIFI-3112 - Handle propagation of stderr and a...

2017-05-16 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1786 @apiri validated ssh command on remote host with with start/stop/restart. All worked as expected, with NiFi starting appropriately and the IO stream for ssh was closed. Also validated

[GitHub] nifi issue #1748: NIFI-3793 - switch to use project.version

2017-05-03 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1748 @joewitt thanks switched to refer to 1.2.0-SNAPSHOT --- 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

[GitHub] nifi pull request #1748: NIFI-3793 - switch to use project.version

2017-05-03 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1748 NIFI-3793 - switch to use project.version Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure

[GitHub] nifi issue #1736: Nifi 3774

2017-05-03 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1736 @thesolson did the merge but corrected one small thing, tick marks on controller were still missing. hope is was ok to apply. --- If your project is set up for it, you can reply to this email

[GitHub] nifi issue #1736: Nifi 3774

2017-05-03 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1736 Thanks Sarah for the updates +1 Will squash and merge this shortly. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114452389 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,146 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114451504 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,146 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114451288 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,146 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114450875 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,146 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114450829 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,146 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114433000 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,148 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114432596 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,148 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi pull request #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1736#discussion_r114432248 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -1176,6 +1176,148 @@ Secure Prompt If you prefer not to provide

[GitHub] nifi issue #1736: Nifi 3774

2017-05-02 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1736 @thesolson thanks will 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

[GitHub] nifi issue #1686: NIFI-3594 Encrypted provenance repository implementation

2017-05-01 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1686 @alopresto actually Andy just noticed there's a conflict. If you could resolve, squash and push I'll be happy to continue with merging into master. --- If your project is set up for it, you

[GitHub] nifi issue #1686: NIFI-3594 Encrypted provenance repository implementation

2017-05-01 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1686 @alopresto ran tests with the following corner cases: -flow running with normal shutdown invoked. NiFi shutdown and started without issue -flow running with kill -9 on process invoked

[GitHub] nifi issue #1686: NIFI-3594 Encrypted provenance repository implementation

2017-05-01 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1686 @alopresto moving to a separate ticket makes sense. I have one more test to run (run out of disk space per @markap14 suggestion) and will provide any feedback shortly. --- If your project

[GitHub] nifi pull request #1726: NIFI-3765 - Added a 'status' operation to Node Mana...

2017-05-01 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1726 NIFI-3765 - Added a 'status' operation to Node Manager The command for status is the following: ./node-manager.sh -d {nifi-installation-dir} -b {nifi-bootstrap-file} -o status

[GitHub] nifi issue #1686: NIFI-3594 Encrypted provenance repository implementation

2017-04-28 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1686 @alopresto Ran a test where I had an existing unencrypted provenance repo and switched to the encrypted provider. The issues [you've documented](https://github.com/alopresto/nifi/blob

[GitHub] nifi issue #1686: NIFI-3594 Encrypted provenance repository implementation

2017-04-28 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1686 @alopresto happy to do some functional testing of this as well. --- 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

[GitHub] nifi issue #1697: NIFI-3695 - added proxy dn flag to support providing autho...

2017-04-27 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1697 Looks good to me, thanks @mcgilman ! --- 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

[GitHub] nifi issue #1697: NIFI-3695 - added proxy dn flag to support providing autho...

2017-04-26 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1697 @mcgilman I addressed your comments above in the latest commit. Please let me know if you have any additional questions or feedback. --- If your project is set up for it, you can reply

[GitHub] nifi issue #1697: NIFI-3695 - added proxy dn flag to support providing autho...

2017-04-26 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1697 @mcgilman understood. It does makes sense to have more polite messaging given this new case. I'll work on those changes and update. --- If your project is set up for it, you can reply

[GitHub] nifi issue #1697: NIFI-3695 - added proxy dn flag to support providing autho...

2017-04-26 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1697 @mcgilman and @brosander appreciate if you could take a look at this enhancement. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi pull request #1697: NIFI-3695 - added proxy dn flag to support providin...

2017-04-26 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1697 NIFI-3695 - added proxy dn flag to support providing authorized usern… …ame for secured environments An updated version of documentation, with usage of the proxy flag is attached

[GitHub] nifi issue #1669: NIFI-3695 - created the nifi admin toolkit which includes ...

2017-04-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1669 @alopresto I addressed the comments to apply LdapName as well as other minor corrections suggested --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1669: NIFI-3695 - created the nifi admin toolkit which includes ...

2017-04-20 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1669 @alopresto thanks so much for your feedback. I've made changes for your review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi issue #1669: NIFI-3695 - created the nifi admin toolkit which includes ...

2017-04-20 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1669 @brosander I've chatted with @thesolson about documentation and she was willing to help there. She will create a separate request in Jira for that effort to add admin-toolkit info to the admin

[GitHub] nifi issue #1669: NIFI-3695 - created the nifi admin toolkit which includes ...

2017-04-19 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1669 @brosander concerning the binaries they have a very small footprint so I wasn't too concerned about including them (32K, 27K and 1K). I will work to incorporate the cert generation test

[GitHub] nifi pull request #1669: NIFI-3695 - created the nifi admin toolkit which in...

2017-04-19 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1669#discussion_r112242565 --- Diff: nifi-toolkit/nifi-toolkit-admin/src/main/groovy/org/apache/nifi/toolkit/admin/client/NiFiClientUtil.groovy --- @@ -0,0 +1,156

[GitHub] nifi issue #1678: NIFI-3694 updated latest logback,jetty,moved servlet-api a...

2017-04-18 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1678 @joewitt ran local test using smaller maxFileSize, maxHistory and totalSizeCap settings. Created a flow that used LogAttribute heavily. nifi-app.log appeared to rotate and was purged

[GitHub] nifi issue #1678: NIFI-3694 updated latest logback,jetty,moved servlet-api a...

2017-04-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1678 @joewitt I will take a look, thanks! --- 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

[GitHub] nifi issue #1669: NIFI-3695 - created the nifi admin toolkit which includes ...

2017-04-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1669 @brosander thanks so much for reviewing! Concerning documentation I do have a link attached. Will also look at the code changes you suggested. --- If your project is set up for it, you can

[GitHub] nifi pull request #1669: NIFI-3695 - created the nifi admin toolkit which in...

2017-04-17 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/1669#discussion_r111814074 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/groovy/org/apache/nifi/web

[GitHub] nifi pull request #1669: NIFI-3695 - created the nifi admin toolkit which in...

2017-04-12 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1669 NIFI-3695 - created the nifi admin toolkit which includes shell scrip… …ts and classes to support notification and basic node management in standalone and clustered nifi

[GitHub] nifi issue #1630: NIFI-3613: Fixed threading bug in JoltTransformJSON and ad...

2017-03-29 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1630 @markap14 reviewed and ran unit/performance tests with no issue. +1 will merge shortly. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi issue #1348: NIFI-3010 - Incorporate Expression Language support for Jo...

2017-01-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1348 @scottyaslan @mcgilman @moranr would appreciate your input, especially on recent styling changes. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1303: NIFI-2994: Provide default position for RPGs

2016-12-22 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1303 Re-tested with several combinations of empty values, system behaved as expected, returning bad requests on invalid input and return 201 Created when position values were available or position

[GitHub] nifi issue #1303: NIFI-2994: Provide default position for RPGs

2016-12-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1303 Testing with a Rest Client API I was able to produce a null pointer exception for the following case: "position":{"x":null,"y":null} "position":

[GitHub] nifi issue #1303: NIFI-2994: Provide default position for RPGs

2016-12-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1303 @mcgilman reviewing --- 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 and wishes

[GitHub] nifi issue #1319: NIFI-3167: Delete button in cluster table wrapping to next...

2016-12-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1319 Tested with 3 node cluster on Firefox, Safari and Chrome. Confirmed existing in Firefox was resolved with patch. +1 Will merge into master shortly. --- If your project

[GitHub] nifi pull request #1348: NIFI-3010 - Incorporate Expression Language support...

2016-12-20 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1348 NIFI-3010 - Incorporate Expression Language support for JoltTransformJSON This PR contains several feature enhancements: - Support of expression language within the specification

[GitHub] nifi issue #1261: NIFI-3024 Added key migration for sensitive processor prop...

2016-11-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1261 made it passed that issue however on subsequent test encoutered the following (includes command executed): HW11205:nifi-1.1.0 ydavis$ /Users/ydavis/dev/tools/nifi-1.1.0/toolkit/nifi

[GitHub] nifi issue #1261: NIFI-3024 Added key migration for sensitive processor prop...

2016-11-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1261 @alopresto I think I isolated part of the issue. Please confirm my use of this command: /Users/ydavis/dev/tools/nifi-1.1.0/toolkit/nifi-toolkit-1.1.0-pr-1261/bin/encrypt-config.sh -b

[GitHub] nifi issue #1261: NIFI-3024 Added key migration for sensitive processor prop...

2016-11-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1261 was able to get more info I think on the problem in an attempt to recover with migrating: HW11205:nifi-1.1.0 ydavis$ /Users/ydavis/dev/tools/nifi-1.1.0/toolkit/nifi-toolkit-1.1.0-pr

[GitHub] nifi issue #1261: NIFI-3024 Added key migration for sensitive processor prop...

2016-11-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1261 Encountered issue while attempting the below test cases (3 node cluster): #initial encryption /Users/ydavis/dev/tools/nifi-1.1.0/toolkit/nifi-toolkit-1.1.0-pr-1261/bin/encrypt

[GitHub] nifi issue #1261: NIFI-3024 Added key migration for sensitive processor prop...

2016-11-23 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1261 @alopresto thanks Andy will confirm --- 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

[GitHub] nifi issue #1261: NIFI-3024 Added key migration for sensitive processor prop...

2016-11-22 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1261 @alopresto reviewing --- 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

[GitHub] nifi issue #1221: NIFI-3024 - Encrypt-toolkit flow.xml.gz support

2016-11-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1221 @olegz I think @alopresto is taking over this PR and may create a new one, per his comment --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1238: NIFI-3051 Fixed issue serializing commented or empty login...

2016-11-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1238 @alopresto was able to run tests both in IDE and Maven and reconfirmed test scenarios worked as expected. +1 Ready for merge. --- If your project is set up for it, you can

[GitHub] nifi issue #1238: NIFI-3051 Fixed issue serializing commented or empty login...

2016-11-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1238 @alopresto agreed with fixing the test. I've actually logged the issue here: https://issues.apache.org/jira/browse/NIFI-3060 Once that is addressed and you've updated your

[GitHub] nifi issue #1238: NIFI-3051 Fixed issue serializing commented or empty login...

2016-11-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1238 @alopresto thanks for the update. Was able to retest with the following scenarios: encryption with fully commented file (as in the original template without edits) encryption

[GitHub] nifi issue #1238: NIFI-3051 Fixed issue serializing commented or empty login...

2016-11-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1238 @alopresto just thought of a particular scenario. If a user defines a provider with an identifier not equal to "ldap-provider" however has securable properties like those found i

[GitHub] nifi issue #1238: NIFI-3051 Fixed issue serializing commented or empty login...

2016-11-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1238 @alopresto actually correction, maven run of this test worked however individual execution of test in my IDE does not so this may not be an issue, probably a configuration problem on my end

[GitHub] nifi issue #1238: NIFI-3051 Fixed issue serializing commented or empty login...

2016-11-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1238 @alopresto I rebased this with master and I have 7 tests are failing. I believe this may be due to recent changes in logging for toolkit, since all failed tests rely on assertions against

[GitHub] nifi issue #1216: NIFI-2654 Enabled encryption coverage for login-identity-p...

2016-11-14 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1216 @alopresto I was able to test encrypt-config update with the following scenarios: tested encryption using new -l flag along with -n, - b, and -p flags (no migration). nifi.properties

[GitHub] nifi issue #1216: NIFI-2654 Enabled encryption coverage for login-identity-p...

2016-11-14 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1216 @alopresto will 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

[GitHub] nifi issue #1066: NIFI-2823 - Adding nifiDnPrefix, nifiDnSuffix options to s...

2016-11-09 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1066 @brosander was able to run tests to generate certs with prefix and suffix +1 Will merge into master shortly --- If your project is set up for it, you can reply to this email

[GitHub] nifi issue #1066: NIFI-2823 - Adding nifiDnPrefix, nifiDnSuffix options to s...

2016-11-08 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1066 @brosander will 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

[GitHub] nifi issue #1186: NIFI-2652 [WIP] Handle encrypted config key migration

2016-11-07 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1186 @alopresto re-ran the unit tests and scenarios looks good +1 --- 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

[GitHub] nifi issue #1186: NIFI-2652 [WIP] Handle encrypted config key migration

2016-11-07 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1186 @alopresto just to add clarity on unit test failures, I am specifically receiving a problem when running the ConfigEncryptionToolTest. Several tests are failing for me and the failure appears

[GitHub] nifi issue #1186: NIFI-2652 [WIP] Handle encrypted config key migration

2016-11-07 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1186 @alopresto I also received problems running unit test but confirmed that I could migrate keys using several scenarios: raw key -> raw key password -> password password

[GitHub] nifi issue #1186: NIFI-2652 [WIP] Handle encrypted config key migration

2016-11-07 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1186 @alopresto happy to review (understanding this is WIP) --- 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

[GitHub] nifi issue #1158: NIFI-2937 - Adding configJsonIn option to tls-toolkit clie...

2016-10-25 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1158 Was able to test toolkit successfully with following scenarios: a) Run with both -F -f flags (allowing single file reference for input/output) to ensure backwards compatibility

[GitHub] nifi issue #1158: NIFI-2937 - Adding configJsonIn option to tls-toolkit clie...

2016-10-24 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1158 @brosander will 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

[GitHub] nifi issue #1068: Ensuring Custom UI requests are correctly replicated when ...

2016-09-27 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1068 @mcgilman was able to verify that both advanced UIs for UpdateAttributes and JoltTransformProcessor were able to save configurations. Tests were performed on standalone NiFi as well as cluster

[GitHub] nifi issue #1068: Ensuring Custom UI requests are correctly replicated when ...

2016-09-27 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1068 will 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 and wishes so

[GitHub] nifi issue #1065: NIFI-2822 - Provide the JWT Authorization token

2016-09-26 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/1065 thx @mcgilman ! --- 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 and wishes so

[GitHub] nifi pull request #1065: NIFI-2822 - Provide the JWT Authorization token

2016-09-26 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/1065 NIFI-2822 - Provide the JWT Authorization token This change ensures that the JWT token used for authorization is added to headers when using the $http service. Testing requires a secured

  1   2   >