[ 
https://issues.apache.org/jira/browse/NIFI-2664?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437360#comment-15437360
 ] 

ASF GitHub Bot commented on NIFI-2664:
--------------------------------------

GitHub user bbende opened a pull request:

    https://github.com/apache/nifi/pull/946

    NIFI-2664 Moving System.setProperty for krb5.conf to NiFi startup, an…

    …d removing conflicting property from KerberosProvider config
    
    Rather than add code to all of the Kafka processors to set the system 
property, it makes more sense for NiFi to set it once during start up. Then it 
will be available to HDFS, HBase, and Kafka processors, as well as the Kerberos 
login provider. 
    
    We also had a potentially conflicting property in the login provider, where 
you could set the krb5.conf file there which would eventually set the system 
property and potentially overwrite the one that could have been set from 
nifi.properties. Now there is on the value in nifi.properties.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/bbende/nifi NIFI-2664

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/nifi/pull/946.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #946
    
----
commit f54a1a2487797228714f682655efdc781fd75bc5
Author: Bryan Bende <[email protected]>
Date:   2016-08-25T17:57:12Z

    NIFI-2664 Moving System.setProperty for krb5.conf to NiFi startup, and 
removing conflicting property from KerberosProvider config

----


> Kafka processors should use krb5 from nifi.properties if specified
> ------------------------------------------------------------------
>
>                 Key: NIFI-2664
>                 URL: https://issues.apache.org/jira/browse/NIFI-2664
>             Project: Apache NiFi
>          Issue Type: Bug
>            Reporter: Bryan Bende
>            Assignee: Bryan Bende
>             Fix For: 1.0.0
>
>
> Currently the Kafka processors work against a Kerberized Kafka, but are 
> relying on the krb5.conf being the default location of /etc.
> We should check if the krb5 location was specified in nifi.properties and if 
> so we should set the system property to use that file, otherwise it will be 
> confusing why it is looking in /etc and not using that file.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to