Github user dlyle65535 commented on the issue:

    https://github.com/apache/incubator-metron/pull/206
  
    Hi JJ,
    
    Thanks for taking this on. Looks great! One small thing, could you look at 
moving the repeated group vars (e.g. hdp_stack, hdp_repo_def and the ambari 
variables) to defaults in the appropriate roles?
    
    Also, since you are in the neighborhood: METRON-43 could be satisfied by 
removing the passwd_less_ssh role. Since it's no longer needed anyway, rather 
than update it to reflect these changes, how would you feel about a twofer?


---
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