[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
+1 pending CI build


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread nickwallen
Github user nickwallen commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
Addressed the problem; see the latest commit.  Tested by deploying the 
Profiler on Quick Dev Vagrant.  Integration tests should be happy now too.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
Yep, I think it's probably due for some more up-to-date instructions on at 
least the stellar enrichment adapter to make some of this easier.  If you have 
other scenarios that you're trying to do, I might be able to point you to 
testing instructions from other PRs that are similar enrichment-wise, too.  
Just let me know.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread nickwallen
Github user nickwallen commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
Agreed. I'll have a working step-by-step set of instructions for 
METRON-421.  That's a good thing to have.  Maybe I'll add it as a 'Getting 
Started' type section to the README in `metron-profiler-client`.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
I understand; I'll defer the request until METRON-421, then.  Sorry 
enrichment is biting 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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread nickwallen
Github user nickwallen commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
The scope of this PR is to fix a bug that causes the Profiler topology to 
fail during deployment due to missing dependencies.  This is supposed to fix an 
issue that you saw last week.  (Although, this seems to have broken the CI 
build so looks like I'll need to chase that now.)

I think you are thinking of METRON-421 which involves getting the Profiler 
Client API working in the Enrichment topology.  That work continues. Our lack 
of documentation around Enrichment is making METRON-421 quite painful.  But the 
link you sent above might help.  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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...

2016-09-19 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/incubator-metron/pull/259
  
Great job, this looks good.  Could you walk us through a testing scenario 
in the quickdev environment with sample configs for the profiler and sample 
enrichment configs, please?  Similar to 
[here](https://github.com/apache/incubator-metron/pull/210#issuecomment-239825715).


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---