[GitHub] incubator-metron pull request #189: Metron-302

GitHub user mmiklavc opened a pull request: https://github.com/apache/incubator-metron/pull/189 Metron-302 Create base spec for metron rpm's https://issues.apache.org/jira/browse/METRON-302 You can merge this pull request into a Git repository by running: $ git pull h

[GitHub] incubator-metron pull request #189: Metron-302 Build Common Deployment RPM

Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/189#discussion_r70807503 --- Diff: metron-deployment/packaging/docker/rpm-docker/build.sh --- @@ -0,0 +1,6 @@ +#!/bin/bash + +rm -rf SRPMS/ RPMS/ && \ +rpm

[GitHub] incubator-metron pull request #189: Metron-302 Build Common Deployment RPM

Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/189#discussion_r70808062 --- Diff: metron-deployment/packaging/docker/rpm-docker/build.sh --- @@ -0,0 +1,6 @@ +#!/bin/bash + +rm -rf SRPMS/ RPMS/ && \ +rpmbu

[GitHub] incubator-metron pull request #189: Metron-302 Build Common Deployment RPM

Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/189#discussion_r70809689 --- Diff: metron-deployment/packaging/docker/rpm-docker/build.sh --- @@ -0,0 +1,6 @@ +#!/bin/bash --- End diff -- forgot the licens

[GitHub] incubator-metron pull request #190: METRON-306: Create Docker container for ...

GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/190 METRON-306: Create Docker container for RPM creation Enable RPM build for non-RH variant environments. You can merge this pull request into a Git repository by running: $ git pull

Re: [DISCUSS] Bylaws discussion

Ok, it's been a month of crickets. I'm going to put this up for a vote. On Thu, Jun 16, 2016 at 11:29 AM, Casey Stella wrote: > I'd like to get the Apache bylaws that we have on the website discussed > and possibly voted in. > > Does anyone have anything to object to in the bylaws as listed her

[GitHub] incubator-metron issue #180: METRON-281 Simplify deployment to alternative A...

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/180 +1 by inspection --- 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 wish

[GitHub] incubator-metron issue #179: METRON-117 Deployment Report Created for All De...

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/179 +1, by inspection --- 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 wis

[GitHub] incubator-metron issue #181: METRON-279 Removed duplicative roles; metron-ka...

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/181 +1, by inspection --- 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 wis

[GitHub] incubator-metron issue #183: METRON-266 Remove duplicate entries of sniff_in...

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/183 +1 by inspection --- 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 wish

[GitHub] incubator-metron issue #182: METRON-225 Amazon-ec2 run script continues runn...

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/182 +1, by inspection --- 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 wis

[GitHub] incubator-metron issue #185: METRON-294 Refactor parser topology builder

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/185 +1, by inspection --- 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 wis

[GitHub] incubator-metron issue #187: Metron 321

Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/187 +1, looks good. --- 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 wis

[GitHub] incubator-metron issue #187: Metron-321 Small update to and cleanup of .giti...

Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/187 Apologies about the titles missing text - did github defaults change bc I don't recall having to set that before? --- If your project is set up for it, you can reply to this email and hav

[GitHub] incubator-metron pull request #187: Metron-321 Small update to and cleanup o...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/187 --- 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 featu

[GitHub] incubator-metron pull request #140: METRON-153 Upgrade to CentOS 7

Github user mmiklavc closed the pull request at: https://github.com/apache/incubator-metron/pull/140 --- 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 fea

Re: [DISCUSS] Bylaws discussion

I don't quite understand the proposal. How do these bylaws differ from what is already in-place? Is there a way I can see the diff between what we have now? On Thu, Jul 14, 2016 at 10:09 AM, Casey Stella wrote: > Ok, it's been a month of crickets. I'm going to put this up for a vote. > > On T

[GitHub] incubator-metron issue #140: METRON-153 Upgrade to CentOS 7

Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/140 Closing this PR - We are going to take a different approach on Centos 7 support. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: [DISCUSS] Bylaws discussion

We do not currently have bylaws, so we default we are in sort of nebulous territory from what I can tell. Take the voting on code modifications, for instance, we are bound by the rules here , from my understanding (since we don't have voted-in bylaws):

[GitHub] incubator-metron issue #177: METRON-262: Update codelab image scripts

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/177 +1 by inspection --- 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 wish

Re: [DISCUSS] Bylaws discussion

I think lazy consensus for code modification is appropriate for where we are at with Metron. As the community matures, we can revisit if needed. I am a +1 on the bylaws as you have them written. On Thu, Jul 14, 2016 at 10:53 AM, Casey Stella wrote: > We do not currently have bylaws, so we defa

[GitHub] incubator-metron pull request #177: METRON-262: Update codelab image scripts

Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/177 --- 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 featu

Re: [DISCUSS] Bylaws discussion

What does "Votes relating to code changes are not subject to a strict timetable but should be made as timely as possible" mean to you? One issue I'm concerned about with lazy consensus is having adequate time to review a code change. Requiring 2 +1's has been pain sometimes, but it kind of makes s

Re: [DISCUSS] Bylaws discussion

It means to me that we don't want to put a minimum time for code review there to avoid slowing things down too much. At some point I think we have to trust our committers to not +1 really bad stuff. That being said, accidents happen (I've made many of them ;) and I think it's on us to impose expe

[GitHub] incubator-metron issue #189: Metron-302 Build Common Deployment RPM

Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/189 Closing this PR - this will instead be included as part of another merged PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-metron pull request #189: Metron-302 Build Common Deployment RPM

Github user mmiklavc closed the pull request at: https://github.com/apache/incubator-metron/pull/189 --- 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 fea

[GitHub] incubator-metron pull request #191: Metron-214 Build RPM Packages for Deploy...

GitHub user mmiklavc opened a pull request: https://github.com/apache/incubator-metron/pull/191 Metron-214 Build RPM Packages for Deployment This PR includes multiple sub-tasks as well. https://issues.apache.org/jira/browse/METRON-214 METRON-299 Build Parsers Deployment

[GitHub] incubator-metron issue #191: Metron-214 Build RPM Packages for Deployment

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/191 Why do we not have a separate PR per JIRA? It appears that the various tasks have various authors, wouldn't it be better to have each author submit their own JIRA so that they can

[GitHub] incubator-metron issue #191: Metron-214 Build RPM Packages for Deployment

Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/191 That's a good question. If we don't squash this pr prior to committing, will that give the authors the proper credit? On the second, this is the completion of METRON-214. ---

[GitHub] incubator-metron issue #191: Metron-214 Build RPM Packages for Deployment

Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/191 Ok, after some out of band discussion, this is fine..lots of small subtasks that had a lot of interdependency. I'm +1 upon inspection. --- If your project is set up for it, you can reply