Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-41789

GitLabTriggerPlugin Ignores Regex Match in DSL

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • job-dsl-plugin
    • job-dsl-plugin: 1.57
      gitlab-plugin: 1.44
      Jenkins: 2.32.2

      There is a flaw in the DSL support for the gitlab-plugin which causes the trigger to only work for standard matching, not regex. This is the default for the gitlab-plugin, so it just appears to work if you're not trying to use the regex match. I have a patch already written for this and will be submitting a pull request momentarily.

          [JENKINS-41789] GitLabTriggerPlugin Ignores Regex Match in DSL

          Jayme Howard added a comment -

          Jayme Howard added a comment - https://github.com/jenkinsci/job-dsl-plugin/pull/992 should be all set to merge.

          Code changed in jenkins
          User: Daniel Spilker
          Path:
          docs/Home.md
          job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/triggers/GitLabTriggerContext.groovy
          job-dsl-core/src/test/groovy/javaposse/jobdsl/dsl/helpers/triggers/TriggerContextSpec.groovy
          http://jenkins-ci.org/commit/job-dsl-plugin/b28e859123c0a6acc6c745ea35a46f2af26405b2
          Log:
          Merge remote-tracking branch 'isugimpy/JENKINS-41789'

          Compare: https://github.com/jenkinsci/job-dsl-plugin/compare/9fdd8567f838...b28e859123c0

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Spilker Path: docs/Home.md job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/triggers/GitLabTriggerContext.groovy job-dsl-core/src/test/groovy/javaposse/jobdsl/dsl/helpers/triggers/TriggerContextSpec.groovy http://jenkins-ci.org/commit/job-dsl-plugin/b28e859123c0a6acc6c745ea35a46f2af26405b2 Log: Merge remote-tracking branch 'isugimpy/ JENKINS-41789 ' Compare: https://github.com/jenkinsci/job-dsl-plugin/compare/9fdd8567f838...b28e859123c0

          Will be fixed in 1.58.

          Daniel Spilker added a comment - Will be fixed in 1.58.

            daspilker Daniel Spilker
            isugimpy Jayme Howard
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: