[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-31 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/497 +1 by inspection and via vagrant. --- 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

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-30 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/497 Just a comment, METRON-797 and METRON-793 are both in master as of the time of this comment, so these docs can be done on current master. --- If your project is set up for it, you can

Re: [GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-30 Thread Matt Foley
Okay, try this: https://github.com/mattf-horton/incubator-metron/blob/METRON-804/metron-deployment/vagrant/Kerberos-setup.md I wasn’t able to build a PR to your branch, seems there’s a non-ff in the way the previous patch was merged. Anyway, if you just grab that file and diff against yours,

Re: [GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-30 Thread Matt Foley
That’s weird. Mine looks fine: https://github.com/mattf-horton/incubator-metron/blob/METRON-804-notes/METRON-804-mf.tiff But the tooling was exactly that of https://github.com/mattf-horton/incubator-metron/tree/METRON-804/site-book/bin What additional changes did you make? Oh, I just looked

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-30 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/497 @mattf-horton Thanks again for the patch! I made a couple more minor tweaks to get the images and indentation correct for the nested lists. I'm unable to get a nested list code block to

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-29 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/497 Hi @anandsubbu - just to be clear, this doc requires METRON-793 and METRON-797. Are you running with those branches? --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-29 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/497 @mattf-horton Thanks! I will pull this in and review. --- 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

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-29 Thread kylerichardson
Github user kylerichardson commented on the issue: https://github.com/apache/incubator-metron/pull/497 @mmiklavc Nice document! Planning to run through it myself later this week. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-metron issue #497: METRON-804: Create a document to describe kerbe...

2017-03-29 Thread mattf-horton
Github user mattf-horton commented on the issue: https://github.com/apache/incubator-metron/pull/497 @mmiklavc , Markdown requires you to indent a codeblock (beginning and ending triple back-ticks) if you wish it to be "under" a list item. Otherwise it interrupts the list, as you've