[GitHub] camel pull request #2229: camel-jsonpath - It is not necessary to explicitly...

2018-02-16 Thread yihtserns
GitHub user yihtserns opened a pull request: https://github.com/apache/camel/pull/2229 camel-jsonpath - It is not necessary to explicitly set default jsonProvider, as ConfigurationBuilder.build will do that anyway You can merge this pull request into a Git repository by running:

Re: camel builds causing problems with jenkins

2018-02-16 Thread Andrew Bayer
The problem is entirely in the per module files on the master - with Pipeline, you're only going to call junit once, which will read all the relevant files on the agent and write just one file on the master. It'll definitely work much, much better. Feel free to reach out to me (my day job is workin

Re: camel builds causing problems with jenkins

2018-02-16 Thread Zoran Regvart
Thank Chris you for clarifying, those log files are created by Jenkins Maven support[1], and they are created for each Maven module of the project that's being built, with Camel having 840 maven modules. I don't think this approach scales with such big number of Maven modules. So Camel devs, seems

Re: camel builds causing problems with jenkins

2018-02-16 Thread Chris Lambertus
Here is a small sample of the types of logs we’re seeing: cml@jenkins-master:~$ cat camel-20180215.lsof | grep camel | grep log | wc -l 2277 java18713 jenkins 4931w REG 8,17 0 215356692 /x1/jenkins/jenkins-home/jobs/Camel.trunk.notest.java9/modules/org.apache.cam

[GitHub] camel pull request #2228: CAMEL-12260: Default value for String field result...

2018-02-16 Thread oscerd
Github user oscerd closed the pull request at: https://github.com/apache/camel/pull/2228 ---

[GitHub] camel pull request #2228: CAMEL-12260: Default value for String field result...

2018-02-16 Thread dmvolod
GitHub user dmvolod opened a pull request: https://github.com/apache/camel/pull/2228 CAMEL-12260: Default value for String field results is null for CSV/Bindy The DataField.defaultValue always "" for String by default. You can merge this pull request into a Git repository by running

Re: [camel] branch master updated: Regen

2018-02-16 Thread Claus Ibsen
Hmm that would has weird as I have done a git clean -d -f and it didnt remove the folder. But I remove the folder manually, and git hasnt complained. Usually git clean will remove untraced folders. On Fri, Feb 16, 2018 at 8:28 AM, Andrea Cosentino wrote: > This component has been removed, ma

Re: [camel] branch master updated: Regen

2018-02-16 Thread Andrea Cosentino
This component has been removed, maybe you've to remove the folder.. -- Andrea Cosentino  -- Apache Camel PMC Member Apache Karaf Committer Apache Servicemix PMC Member Email: ancosen1...@yahoo.com Twitter: @oscerd2 Github: oscerd On Friday, February 16, 2018,

Re: camel builds causing problems with jenkins

2018-02-16 Thread Zoran Regvart
Hi Chris, thank you for troubleshooting this, can you clarify one thing for me, when you mention log files, are these the `.log` files generated during the test phase of the build or the XML/TXT files with JUnit reports? I would think that the job type being Maven and the automatic gathering of JU

[GitHub] camel pull request #2227: Remove execute permissions from unnecessary files

2018-02-16 Thread oscerd
Github user oscerd closed the pull request at: https://github.com/apache/camel/pull/2227 ---

[GitHub] camel pull request #2227: Remove execute permissions from unnecessary files

2018-02-16 Thread dmvolod
GitHub user dmvolod opened a pull request: https://github.com/apache/camel/pull/2227 Remove execute permissions from unnecessary files This PR removes execute permissions from unnecessary files. This is not very good to have it on *nix systems otherwise for .sh and other executable