You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by nickwallen <gi...@git.apache.org> on 2016/09/16 21:30:37 UTC
[GitHub] incubator-metron pull request #259: METRON-429 Profiler Missing Dependencies...
GitHub user nickwallen opened a pull request:
https://github.com/apache/incubator-metron/pull/259
METRON-429 Profiler Missing Dependencies When Uber Jar Deployed
When the Profiler is packaged and deployed on a Metron environment, the topology does not start because the uber jar is missing HBase as a dependency. Also, corrected the 'Getting Started' instructions in the README.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nickwallen/incubator-metron METRON-429
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-metron/pull/259.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 #259
----
commit 49c8ffb57d45608970b6eb46a3b45905c7a9da9b
Author: Nick Allen <ni...@nickallen.org>
Date: 2016-09-15T14:55:47Z
METRON-429 Profiler Missing Dependencies When Uber Jar Deployed
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by cestella <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by cestella <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by cestella <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by nickwallen <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by nickwallen <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request #259: METRON-429 Profiler Missing Dependencies...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/259
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by cestella <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #259: METRON-429 Profiler Missing Dependencies When U...
Posted by nickwallen <gi...@git.apache.org>.
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---