GitHub user nickwallen opened a pull request:

    https://github.com/apache/incubator-metron/pull/94

    METRON-123 Pycapa requires 'kafka_broker_url'

    Pycapa needs to know the URL of the Kafka Broker to send packet data to. 
During a standard deployment, such as single node vagrant or amazon-ec2, this 
URL needs to be retrieved from Ambari.  When using Pycapa to test other 
functionality, like fastcapa, Ambari is not used.  In these cases the URL need 
to be specified by the user instead of being retrieved from Ambari.
    
    The Pycapa role at `metron-deployment/roles/pycapa` previously had a 
dependency on `ambari_gather_facts` through which it retrieved the Kafka Broker 
URL. This dependency was removed in METRON-110 which introduced this bug.
    
    ### Changes
    - Changed the `ambari_gather_facts` Ansible role so that it only retrieves 
information from Ambari if the information is not already known.
    - Updated Pycapa so that the service start script can be installed or not 
depending on a variable.  As a sensor, we need the start script installed.  As 
a testing tool, we do not need the start script installed.
    - Update Fastcapa test environment so extra dummy variables are provided 
instead of having the process reach out to a nonexistent Ambari server.
    
    ### Validation
    Running Pycapa in Single Node Vagant provided the "positive" test, using 
Ambari to retrieve value, and using the Fastcapa Test Environment provided the 
"negative" test, using a user-provided value.
    - Single Node Vagrant Environment `cd 
metron-deploymet/vagrant/singlenode-vagrant; vagrant up`
    - Fastcapa Test Environment `cd metron-deployment/vagrant/fastcapa-vagrant; 
vagrant up`

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

    $ git pull https://github.com/nickwallen/incubator-metron METRON-123

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

    https://github.com/apache/incubator-metron/pull/94.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 #94
    
----
commit 9b526e564d102cceefc31dbb90eea43a4859f1ba
Author: Nick Allen <[email protected]>
Date:   2016-04-28T17:12:08Z

    METRON-123 Fixed issue where pycapa was not able to find kafka_broker_url

----


---
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, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to