Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem setting DisableConcurrentBuildsJobProperty #86

Open
colinjcahill opened this issue Oct 11, 2018 · 0 comments
Open

Problem setting DisableConcurrentBuildsJobProperty #86

colinjcahill opened this issue Oct 11, 2018 · 0 comments

Comments

@colinjcahill
Copy link

Hey, thanks for this project, we're making good use of it.

I saw that pipeline support was added in #61. We are using the same pipeline plugin referenced in this issue, but there seems to be an issue parsing this block: <org.jenkinsci.plugins.workflow.job.properties.DisableConcurrentBuildsJobProperty/>. Instead of setting this to concurrent: false, it sets the output to concurrent: true.

Based on this line, it looks to me like it is searching for a property like this one, but can't find it and is subsequently setting that property to true.

Here is a tiny repro case:

config.xml:

<?xml version='1.1' encoding='UTF-8'?>
<flow-definition plugin="[email protected]">
  <actions/>
  <description>jjwrecker example</description>
  <properties>
    <org.jenkinsci.plugins.workflow.job.properties.DisableConcurrentBuildsJobProperty/>
  </properties>
  <disabled>false</disabled>
</flow-definition>

output/config.yml:

- job:
    concurrent: true
    description: jjwrecker example
    disabled: false
    name: test
    project-type: pipeline

Maybe the plugin has changed its conventions since this was originally implemented. Sorry if I am missing something obvious, I just started using jjwrecker yesterday :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant