[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-28 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 thanks for helping me work through it! I will be back to Metron to contribute in the future 😄 ---

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-14 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 Hey, awesome! Thanks to you and the rest of the metron community for your patience and support ---

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-14 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @merrimanr @ottobackwards Hey I think I got this working! See my recent commit. I had accidentally smashed together two commands, which is why node stuff was failing. I was able to get

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-14 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @nickwallen Thank you so much!! I was able to get `master` building with your help and the advice of the other commenters. Now that I know I can build, I'll be able to test my changes

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-12 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @mmiklavc Thank you for the help! Unfortunately, this did not work for me. Running `mvn clean install -DskipTests` fails with this error: ``` [ERROR] Failed to execute goal on project

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-12 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 Unfortunately, I cannot even build the current `master` branch of this project (with none of my changes) successfully with that command. When running `mvn clean package -DskipTests

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-10 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @merrimanr thank you, understood. I will try to reproduce the issue on my end and see if I can find what is failing. ---

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-09 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 I broke this into two steps (was previously one): ``` # Install Software Collections repo (needs to be done in separate command) \ +&& yum install -y \ +

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-09 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @merrimanr Oh I force-pushed over the old commit. I don't like having failing commits in the history. Apologies if that was not what you expected. In the future I can push new ones individually. ---

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-04 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @ottobackwards I tried to get this running on my local but was hitting some weird networking errors, even on current master (without any of my changes). I'm not very familiar with vagrant

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-03 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 just pushed the `scl` fix ---

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-03 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 Running `mvn clean package -DskipTests` inside the container built with this version of `ansible-docker` succeeds: ![image](https://user-images.githubusercontent.com/7608904/42256070

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-03 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 I THINK I found the issue with my original PR. The command to register the scl repo doesn't get picked up by `yum` until it exits. **this fails to find devtoolset-4** ``` yum

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-03 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 ok that's fair! And I do appreciate that you had an issue template saying "hey do these things before you make contributions". I'll update the PR when I've had a chance to r

[GitHub] metron issue #1091: METRON-1650: Cut size of packaging docker containers

2018-07-03 Thread jameslamb
Github user jameslamb commented on the issue: https://github.com/apache/metron/pull/1091 @merrimanr ok no problem! Yeah I can run those later and report back. IMHO it's a problem that your CI doesn't catch this though? Like ideally shouldn't these package builds should

[GitHub] metron pull request #1091: METRON-1650: Cut size of packaging docker contain...

2018-07-02 Thread jameslamb
GitHub user jameslamb opened a pull request: https://github.com/apache/metron/pull/1091 METRON-1650: Cut size of packaging docker containers ## Contributor Comments I have been looking through the project source code, and found that the packaging docker containers