Hello,
I would like to add our plugin to the Jenkins plugin list.
On Aug 4, 2015, at 00:07, Kirill Merkushev <twil...@gmail.com> wrote:yep, why not to pr this to original plugin? It can be in separate package and have optional dependency
понедельник, 3 августа 2015 г., 10:01:03 UTC+3 пользователь Dennis Schulte написал:Hello,
I would like to add our plugin to the Jenkins plugin list.
- URL: https://github.com/codecentric/job-dsl-promotions-plugin
- plugin name: job-dsl-promotions-plugin
- github name: denschu
- jenkins-ci.org UserName: denschu
- description: This plugin is an extension for the existing Job DSL Plugin. With this extension it is possible to generate promotions with the Job DSL. See the Promoted Builds Plugin for further informations about the meaning of Promotions for a Jenkins Job.
Thank you and Best RegardsDennis
--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/4d221649-0665-49bb-96b0-eefb7500b6b7%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/16a2df29-c855-45bb-b9e7-de7b092bc3d7%40googlegroups.com.
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAMui946-_ttvjgAYM5Ad37HkjoDUM3wqydGfdKBkrMVwv9DB6w%40mail.gmail.com.
--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/e6946a1a-a207-4bc4-8fc2-b42021b996f3%40googlegroups.com.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/672AFF66-3334-44B4-B46D-719905104EEF%40gmail.com.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAAe5jyNwkCaURREs3WAbpeFk8QHHgct7UvUAnEby-EVBXF7DGA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwpJ0Rb6bcSeZqb%2B92uXoBQCnuo8dsoKqP%2BtejTCCYYDQ%40mail.gmail.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/6AA7F6BE-3E4C-4038-A8BD-A7EFD4F03151%40gmail.com.
I just want to make sure that we send a clear signal to Dennis before he creates a pull request that has little or no chance to get merged.I don't like that option. The optional dependencies have been added before the extension point was available. They will be kept for compatibility, but any new DSL features which require more logic than simply generating a config.xml should be implemented through the extension point. Otherwise the Job DSL plugin will end up with dozens of optional dependencies for all plugins which require extra configuration and relying on API which might not be considered as public and stable API by the plugin maintainers because it was not intended to be used by something like the Job DSL plugin. And I don't want to spend time with tracking API changes from plugins. I'm maintaining the Job DSL plugin for some month now and there have been plugins which changed the on-disk config.xml format (accidentially and not, e.g. JENKINS-26561 and JENKINS-29678). And that broke the Job DSL plugin from a user perspective. I expect that API-level changes happen more often because plugin developers are aware to keep the on-disk config stable, but do not expect that "internal" plugin classes are used from the outside (e.g. JENKINS-30013). The Job DSL plugin on the other hand offers a stable extension point for all plugins to contribute DSL features designed for that very specific reason.If this boils down to having Groovy code or not in a plugin, the decision is up the the plugin's maintainers. Groovy code is as testable and maintainable as Java code if done right. You can use JUnit to test Groovy classes, you do not have to use Groovy frameworks like Spock. And you can use CodeNarc for static code analysis.
The Job DSL extension point can be implemented in Java or Groovy. I expected a discussion like this and avoided any Groovy dependencies when designing the extension point API.
On Sun, Sep 27, 2015 at 10:39 AM, Oleg Nenashev <o.v.ne...@gmail.com> wrote:
Yes, it ay be the best option, because Job DSL plugin already optionally depends on 4 plugins due to the same reason.Daniel (Spilker), WDYT?
On 27 Sep 2015, at 10:17, Stephen Connolly <stephen.al...@gmail.com> wrote:
Why can't he optional dep not be inverted and just have job-dsl optionally depend on promoted builds.Would seem to me that promotion is the more "core" functionality than job-dsl so the onus should be on job-dsl to add support for promoted builds rather than the other way.
In any case I see nothing wrong with having this as a separate plugin either
On Sunday 27 September 2015, Damien <damien...@gmail.com> wrote:
Hi All,as Oleg mentioned, he is the current maintainer. As my current job responsibilities does not involve using Jenkins as much, I don't have a strong opinion what the direction is. Groovy or Java is irrelevant to me as long as a strong focus on maintainability and testability remains.good luck with the merge.On Sat, Sep 26, 2015 at 3:16 PM, Oleg Nenashev <o.v.ne...@gmail.com> wrote:The previous maintainer (Damien Nozay, in Cc) of Promoted Builds plugin granted me the persmissions to merge PRs and relelase . Technically I'm the current plugin's maintainer.
I don't think that Groovy is a showstopper for integrating the functionality into Promoted Builds plugin. Personally I would prefer to have a Java-only code due to the maintenability & static analysis reasons, but I will accept Groovy-based pieces if they are tested enough and do not require a special build flow.
Best regards,
Oleg--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-dev+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAAe5jyNwkCaURREs3WAbpeFk8QHHgct7UvUAnEby-EVBXF7DGA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
Sent from my phone--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwpJ0Rb6bcSeZqb%2B92uXoBQCnuo8dsoKqP%2BtejTCCYYDQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
I just want to make sure that we send a clear signal to Dennis before he creates a pull request that has little or no chance to get merged.I don't like that option. The optional dependencies have been added before the extension point was available. They will be kept for compatibility, but any new DSL features which require more logic than simply generating a config.xml should be implemented through the extension point. Otherwise the Job DSL plugin will end up with dozens of optional dependencies for all plugins which require extra configuration and relying on API which might not be considered as public and stable API by the plugin maintainers because it was not intended to be used by something like the Job DSL plugin. And I don't want to spend time with tracking API changes from plugins. I'm maintaining the Job DSL plugin for some month now and there have been plugins which changed the on-disk config.xml format (accidentially and not, e.g. JENKINS-26561 and JENKINS-29678). And that broke the Job DSL plugin from a user perspective. I expect that API-level changes happen more often because plugin developers are aware to keep the on-disk config stable, but do not expect that "internal" plugin classes are used from the outside (e.g. JENKINS-30013). The Job DSL plugin on the other hand offers a stable extension point for all plugins to contribute DSL features designed for that very specific reason.If this boils down to having Groovy code or not in a plugin, the decision is up the the plugin's maintainers. Groovy code is as testable and maintainable as Java code if done right. You can use JUnit to test Groovy classes, you do not have to use Groovy frameworks like Spock. And you can use CodeNarc for static code analysis.
The Job DSL extension point can be implemented in Java or Groovy. I expected a discussion like this and avoided any Groovy dependencies when designing the extension point API.
On Sun, Sep 27, 2015 at 10:39 AM, Oleg Nenashev <o.v.ne...@gmail.com> wrote:
Yes, it ay be the best option, because Job DSL plugin already optionally depends on 4 plugins due to the same reason.Daniel (Spilker), WDYT?
On 27 Sep 2015, at 10:17, Stephen Connolly <stephen.al...@gmail.com> wrote:
Why can't he optional dep not be inverted and just have job-dsl optionally depend on promoted builds.Would seem to me that promotion is the more "core" functionality than job-dsl so the onus should be on job-dsl to add support for promoted builds rather than the other way.
In any case I see nothing wrong with having this as a separate plugin either
On Sunday 27 September 2015, Damien <damien...@gmail.com> wrote:
Hi All,as Oleg mentioned, he is the current maintainer. As my current job responsibilities does not involve using Jenkins as much, I don't have a strong opinion what the direction is. Groovy or Java is irrelevant to me as long as a strong focus on maintainability and testability remains.good luck with the merge.On Sat, Sep 26, 2015 at 3:16 PM, Oleg Nenashev wrote:The previous maintainer (Damien Nozay, in Cc) of Promoted Builds plugin granted me the persmissions to merge PRs and relelase . Technically I'm the current plugin's maintainer.
I don't think that Groovy is a showstopper for integrating the functionality into Promoted Builds plugin. Personally I would prefer to have a Java-only code due to the maintenability & static analysis reasons, but I will accept Groovy-based pieces if they are tested enough and do not require a special build flow.
Best regards,
Oleg--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-dev+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAAe5jyNwkCaURREs3WAbpeFk8QHHgct7UvUAnEby-EVBXF7DGA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
Sent from my phone--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwpJ0Rb6bcSeZqb%2B92uXoBQCnuo8dsoKqP%2BtejTCCYYDQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
Cheers
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAAe5jyNwkCaURREs3WAbpeFk8QHHgct7UvUAnEby-EVBXF7DGA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
Sent from my phone--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwpJ0Rb6bcSeZqb%2B92uXoBQCnuo8dsoKqP%2BtejTCCYYDQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/6AA7F6BE-3E4C-4038-A8BD-A7EFD4F03151%40gmail.com.
--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/08b8b6af-2e25-469c-b56a-032f2e468c72%40googlegroups.com.
Cheers
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-dev+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAAe5jyNwkCaURREs3WAbpeFk8QHHgct7UvUAnEby-EVBXF7DGA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
Sent from my phone--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwpJ0Rb6bcSeZqb%2B92uXoBQCnuo8dsoKqP%2BtejTCCYYDQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/6AA7F6BE-3E4C-4038-A8BD-A7EFD4F03151%40gmail.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CAPfivLD51KwNLwhrfu27EA%2B%2Btjg98G_ygDhRGikL8zRYK%3D0NJg%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/B88B0980-815D-468F-9823-788F4DD8C7FE%40gmail.com.
For more options, visit https://groups.google.com/d/optout.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMxXb_BrU3%3DdsR97mQ-5TEQ8%2B8ox0VrwAV%2BKZgSdSpVQcg%40mail.gmail.com.
Statement is right, but has no any clarification rather then “must die". Jenkins already supports loading plugins and allows doing optional dependencies.Could you provide any real reason to not use optional dependency in this case?
On Sep 30, 2015, at 01:46, Stephen Connolly <stephen.al...@gmail.com> wrote:
Java dependencies are not the same as Linux package dependencies
On Tuesday 29 September 2015, Kanstantsin Shautsou <kanstan...@gmail.com> wrote:
Optional dependencies works fine for many years in linux distros. What is the problem in jenkins?
Redundant hard deps should also die.
PS bottom-quote on top-quote is bad manner.
> On Sep 29, 2015, at 18:21, Stephen Connolly <stephen.alan.connolly@gmail.com> wrote:
>
> to quote Jesse: "optional dependencies must die"
>
> On 29 September 2015 at 13:47, Daniel Beck <m...@beckweb.net> wrote:
>>
>> On 29.09.2015, at 14:20, Stephen Connolly <stephen.alan.connolly@gmail.com> wrote:
>>
>>> Please do not add a dependency on job-DSL to promoted-builds (at least until job-dsl has fixed the security design issues)
>>
>> Wouldn't that dependency be optional anyway?
>>
>> --
>> You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-dev+unsubscribe@googlegroups.com.
>> To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/F102D219-20B7-47A2-9882-DEAA572D6DB8%40beckweb.net.
>> For more options, visit https://groups.google.com/d/optout.
>
> --
> You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
> To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
> To unsubscribe from this group and all its topics, send an email to jenkinsci-dev+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwq%3Dvis9A8PsKzVvxwQEDUuEbhNzHfEkwnqU_7YXgNHFw%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-dev+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/B88B0980-815D-468F-9823-788F4DD8C7FE%40gmail.com.
For more options, visit https://groups.google.com/d/optout.
Dennis
hi,
> On Sep 29, 2015, at 18:21, Stephen Connolly <stephen.al...@gmail.com> wrote:
>
> to quote Jesse: "optional dependencies must die"
>
> On 29 September 2015 at 13:47, Daniel Beck <m...@beckweb.net> wrote:
>>
>> On 29.09.2015, at 14:20, Stephen Connolly <stephen.al...@gmail.com> wrote:
>>
>>> Please do not add a dependency on job-DSL to promoted-builds (at least until job-dsl has fixed the security design issues)
>>
>> Wouldn't that dependency be optional anyway?
>>
>> --
>> You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
>> To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/F102D219-20B7-47A2-9882-DEAA572D6DB8%40beckweb.net.
>> For more options, visit https://groups.google.com/d/optout.
>
> --
> You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
> To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
> To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMwq%3Dvis9A8PsKzVvxwQEDUuEbhNzHfEkwnqU_7YXgNHFw%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to the Google Groups "Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/B88B0980-815D-468F-9823-788F4DD8C7FE%40gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
Sent from my phone--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMxXb_BrU3%3DdsR97mQ-5TEQ8%2B8ox0VrwAV%2BKZgSdSpVQcg%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to a topic in the Google Groups "Jenkins Developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jenkinsci-dev/IZ3kBFhIYYg/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jenkinsci-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/598b7005-9a4b-464c-aad8-ff251959fb7b%40googlegroups.com.