Re: [DISCUSS] Time for Jenkins to require Java 8 to run

2016-10-18 Thread nicolas de loof
@Oleg I totally agree we need to investigate java 9 support - for your information, jenkins doesn't boot on java9+jigsaw, see https://github.com/x-stream/xstream/issues/74 2016-10-18 23:42 GMT+02:00 Oleg Nenashev : > Weak -1 regarding JDK8, but the opinion is not that

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Surya Gaddipati
>. Adding a one-off method which is not in general safe to call would just make a compatible transition harder. ok yea this sounds like a reasonable concern. I would be glad to help with lock-free queue implementation which would be ideal for 1 computer <=> 1 build type of situation.

Re: [DISCUSS] Time for Jenkins to require Java 8 to run

2016-10-18 Thread nicolas de loof
Have just read Java 9 is postponed to 2017/07/27, give us some extra time to look into this 2016-10-19 7:47 GMT+02:00 nicolas de loof : > @Oleg I totally agree we need to investigate java 9 support - for your > information, jenkins doesn't boot on java9+jigsaw, see >

Re: Plugin to add a custom build step.

2016-10-18 Thread Daniel Beck
> On 17.10.2016, at 20:07, Kul Bhushan Srivastava wrote: > > Please suggest. I don't know what else is wrong. Compare what you did with the mvn hpi plugin archetype, it basically does exactly what you're doing, and it works. mvn -U

Re: maven-plugin release ready to be done

2016-10-18 Thread Arnaud Héritier
On Tue, Oct 18, 2016 at 4:01 AM, Olivier Lamy wrote: > Hey there, > All done! (just a matter of upgrading the tycho version in it test as you > may use a symlink on your machine) > See my comment :

Re: Violations Report for Atomiq - Code Similarity Finder and SOurce Monitor

2016-10-18 Thread Daniel Beck
> On 18.10.2016, at 09:48, venkatakrishnan nagarajan > wrote: > > Hi, As am unware of java stuff, I explored python to create new plugin. I > havent got any hint nowhere. That's because it's not possible. -- You received this message because you are subscribed

Re: Release Plugin-Errors

2016-10-18 Thread Daniel Beck
> On 18.10.2016, at 11:46, lakshm...@qualesce.com wrote: > > Can you please tell what is the process to follow for releasing the plugin > after over plugin has forked in github? > https://wiki.jenkins-ci.org/display/JENKINS/Hosting+Plugins#HostingPlugins-Releasingtojenkinsci.org -- You

Re: Violations Report for Atomiq - Code Similarity Finder and SOurce Monitor

2016-10-18 Thread venkatakrishnan nagarajan
How about displaying the HTML report files under Violation report? If yes > then how to configure. But am able to display the HTML files under Post Build Actions-> Archive the artifacts. Thanks -- You received this message because you are subscribed to the Google Groups "Jenkins

Re: Release Plugin-Errors

2016-10-18 Thread lakshmi . p
Hi, Can you please tell what is the process to follow for releasing the plugin after over plugin has forked in github? On Monday, October 17, 2016 at 4:55:32 PM UTC+5:30, Daniel Beck wrote: > > > > On 17.10.2016, at 11:52, laks...@qualesce.com wrote: > > > > Can you brief me in which way

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Oliver Gondža
On 2016-10-18 14:52, Surya Gaddipati wrote: In theory, add should be safe and only remove requiring the lock... Would you be open to accepting the following patch to jenkins core | Jenkins.getInstance().addNodeWithoutQueueLock(node) This will further expose the nasty implementation detail

Re: Quoting ArgumentListBuilder addQuoted(""). Fails to run on unix.

2016-10-18 Thread Joseph P
I'd be happy to report a issue. Was thinking someone here had experience with escaping it properly before creating an issue. Den tirsdag den 18. oktober 2016 kl. 13.16.00 UTC+2 skrev Daniel Beck: > > This is not where you report bugs. > >

Re: Quoting ArgumentListBuilder addQuoted(""). Fails to run on unix.

2016-10-18 Thread Joseph P
Issue created https://issues.jenkins-ci.org/browse/JENKINS-39066 Den mandag den 17. oktober 2016 kl. 20.38.56 UTC+2 skrev Joseph P: > > Hi > > I have a login command which accepts no password by "" or '' > Command execute fine on Windows. > > This is how the ProcStarter gets going. > ProcStarter

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Stephen Connolly
On 18 October 2016 at 14:00, Oliver Gondža wrote: > On 2016-10-18 14:52, Surya Gaddipati wrote: > > In theory, add should be safe and only remove requiring the lock... >>> >> Would you be open to accepting the following patch to jenkins core >> >> | >>

Replacment for getBuilders()

2016-10-18 Thread fkpkot
Hi, In our plugin we use the getBuilders method in order to learn which builders were active during the build and get their working files and etc. As far as i understand - moving to pipeline has disabled it due to design logic (one cannot simply know what went on during the build :) ) and so

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Surya Gaddipati
Hi Stephen, I really appreciate your quick responses, >> It's when you then end up down at https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/model/Queue.java#L1525-L1526 that the issues start cropping up. I am not quite sure what you mean 'issues start cropping up'

Re: Violations Report for Atomiq - Code Similarity Finder and SOurce Monitor

2016-10-18 Thread venkatakrishnan nagarajan
Hi, As am unware of java stuff, I explored python to create new plugin. I havent got any hint nowhere. Is there any possibility to display my own files like HTML/BMP/XML in violation report plugin? Thanks On Tuesday, October 4, 2016 at 4:48:09 PM UTC, Daniel Beck wrote: > > > > On 04.10.2016,

Re: Quoting ArgumentListBuilder addQuoted(""). Fails to run on unix.

2016-10-18 Thread Daniel Beck
This is not where you report bugs. https://wiki.jenkins-ci.org/display/JENKINS/How+to+report+an+issue > On 17.10.2016, at 20:38, Joseph P wrote: > > Hi > > I have a login command which accepts no password by "" or '' > Command execute fine on Windows. > > This is how the

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Surya Gaddipati
>thus encapsulated the API calls you are struggling with. Oh yea I remember seeing that somewhere too. I remember seeing a commit on jenkins core for this too. But thats not the issue I am talking about here. Let me summarize this discussion, Jenkins.getInstance().addNode(node); This

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Jesse Glick
On Tue, Oct 18, 2016 at 12:10 PM, Surya Gaddipati wrote: > neither concerrentmodificationexception nor scheduling > on zombie node are applicable here . If you are bypassing a lock, a CME seems like a risk. > a patch to core that adds nodes to jenkins > without

fetching nodes from chefserver

2016-10-18 Thread Lavanya pujari
I am working on integrating jenkins with chef. So Inorder to communicate I need to fetch the nodes from chef server where I am able to establish connection between these two. How can I fetch the nodes? -- You received this message because you are subscribed to the Google Groups "Jenkins

Re: Replacment for getBuilders()

2016-10-18 Thread Jesse Glick
On Tue, Oct 18, 2016 at 9:20 AM, wrote: > Has anyone found nice workaround which doesn't involve ugly parameter > passing? What exactly needs to be worked around? Each step should be self-contained. If it needs to be told where to look for files, tell it. -- You received

Re: fetching nodes from chefserver

2016-10-18 Thread Daniel Beck
> On 18.10.2016, at 16:46, Lavanya pujari wrote: > > I am working on integrating jenkins with chef. Not a question for the Jenkins developers list. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To

Re: Release Plugin-Errors

2016-10-18 Thread Jesse Glick
On Fri, Aug 19, 2016 at 5:02 AM, Robert Sandell wrote: > Could it be because you have no javadoc? > > [ERROR] javadoc: error - No public or protected classes found to document. No, this error would mean there are no sources at all, or they are all paclage-private. Either

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Jesse Glick
On Mon, Oct 17, 2016 at 5:47 PM, Surya Gaddipati wrote: > yes like that, there are a couple of differences IIRC there was a more abstract plugin which purported to provide an API to allocate a node for the duration of one build only, and which thus encapsulated the API

Re: Updating to remoting 3 in the core

2016-10-18 Thread Oleg Nenashev
We are ready to merge https://github.com/jenkinsci/jenkins/pull/2492. Is somebody has any concerns, please shout ASAP четверг, 13 октября 2016 г., 10:52:55 UTC+2 пользователь Arnaud Héritier написал: > > +1 > > On Thu, Oct 13, 2016 at 8:25 AM, Oleg Nenashev > wrote: > >>

Re: Adding a node with Queue.withLock in Jenkins

2016-10-18 Thread Pavel Janousek
On the other hand, is it really necessary to call all associated Listeners (like in [1].update() - L222, L236, L242) when the Queue is still locked (is it guaranteed and documented for a ProvisioningListener contract)? It can be a quite time spent operation/call... [1]

Re: [DISCUSS] Time for Jenkins to require Java 8 to run

2016-10-18 Thread Oleg Nenashev
Weak -1 regarding JDK8, but the opinion is not that strong. I'm pretty sure that now it will be a pretty big problem for low-end platfroms like Embedded Systems with hand-made OpenJDK builds. We should rather consider investing into Java 9 Early Access IMHO. People will likely start submitting