Skip to content
This repository was archived by the owner on Aug 20, 2025. It is now read-only.

Conversation

@nickwallen
Copy link
Contributor

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.

@cestella
Copy link
Member

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.

@nickwallen
Copy link
Contributor Author

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.

@cestella
Copy link
Member

I understand; I'll defer the request until METRON-421, then. Sorry enrichment is biting you.

@nickwallen
Copy link
Contributor Author

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.

@cestella
Copy link
Member

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.

@nickwallen
Copy link
Contributor Author

Addressed the problem; see the latest commit. Tested by deploying the Profiler on Quick Dev Vagrant. Integration tests should be happy now too.

@cestella
Copy link
Member

+1 pending CI build

@asfgit asfgit closed this in eb67393 Sep 20, 2016
@nickwallen nickwallen deleted the METRON-429 branch November 30, 2016 19:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants