Requests for Help: add FreeBSD procstat support to akuma library for Jenkins

[Allan Jude]

On 2015-06-25 15:49, George Neville-Neil wrote:
> 
> 
> On 18 Jun 2015, at 11:05, Craig Rodrigues wrote:
> 
>> Hi,
>>
>> When setting up Jenkins, if you install new plugins, there
>> is a checkbox:
>>
>> Restart Jenkins when installation is complete and no jobs are running
>>
>> This checkbox allows you to restart Jenkins automatically when new
>> plugins are installed.  This feature works on OS X, Linux, and Solaris,
>> but not FreeBSD.
>>
>> I tracked down the problem on FreeBSD.  It turns out in this file
>> in the Jenkins core Jenkins:
>> https://github.com/kohsuke/hudson/blob/master/core/src/main/java/hudson/lifecycle/UnixLifecycle.java
>>
>>
>> the akuma library ( https://github.com/kohsuke/akuma/ ) is used
>> to figure out what command-line arguments
>> were passed to the Java VM that is running Jenkins.
>> This allows Jenkins to restart the VM with the same command-line
>> arguments.
>>
>> The akuma library has platform specific code for
>> OS X, Linux, and Solaris, but not FreeBSD.
>>
>> I am a bit overstretched with multiple things at this point.
>> I was wondering if someone with Java coding experience can help
>> with the following.
>>
>> (1)  Take the akuma library from https://github.com/kohsuke/akuma/
>>
>> (2)  Implement FreeBSD support in akuma, by having it execute
>>    "procstat -c" to figure out the command-line arguments for a process.
>>
>> (3)  Make sure that all tests in akuma pass
>>
>> (4)  Send a GitHub pull request to incorporate the change upstream.
>>
>>
> I do not code in Java but also didnt' see a followup to this.  Has anyone
> taken it upon themselves to fix this problem?  If so, please speak up.
> 
> Best,
> George
> _______________________________________________
> freebsd-testing at freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/freebsd-testing
> To unsubscribe, send any mail to "freebsd-testing-unsubscribe at freebsd.org"

I have looked into it, although the problem I foresee is that procstat
-k doesn't separate the different argv values with \0, but with just a
space. So it won't properly handle quoted arguments.

My interest in this was mainly based on my patch to libxo procstat, and
how it could solve the above problem. I am not sure how useful that ends
up being, since it would rely on a feature which isn't in any release of
FreeBSD yet. It is infact still waiting on a reviewer to get committed.


-- 
Allan Jude

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/freebsd-testing/attachments/20150625/073ef416/attachment.bin>