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

Add an option to permit the disable of Strict Forbidden files

    Details

    • Similar Issues:

      Description

      Add option to a gerrit project to permit the disable of the default strict forbidden file verification of an event.

      Enabling this option will allow an event to trigger a build if the event contains BOTH one or more wanted file paths/topics/branches AND one or more forbidden file paths.

      In other words, with this option, the build will not get triggered if the change contains only forbidden files, otherwise it will get triggered.

      A checkbox that has "Disable Strict Forbidden File Verification" shall be added.

        Attachments

          Activity

          Hide
          nasjenkins Naseer Siddique added a comment -

          I would like something slightly more general. We currently define our trigger using project-pattern (repository name) and branch-pattern (** for all files). We don't use file-paths. But we'd also like the same functionality as specified above, where we'd like the project to not trigger only if ALL the changed items are in forbidden-file-paths.

          Looking at the existing semantics, the following tags are additive:

          branches
          file-paths
          topics

          and the following is subtractive:

          forbidden-file-paths

          If a checkbox is added, it should account for all of the additive tags instead of just file-paths.

          Show
          nasjenkins Naseer Siddique added a comment - I would like something slightly more general. We currently define our trigger using project-pattern (repository name) and branch-pattern (** for all files). We don't use file-paths. But we'd also like the same functionality as specified above, where we'd like the project to not trigger only if ALL the changed items are in forbidden-file-paths. Looking at the existing semantics, the following tags are additive: branches file-paths topics and the following is subtractive: forbidden-file-paths If a checkbox is added, it should account for all of the additive tags instead of just file-paths.
          Hide
          scoheb Scott Hebert added a comment -

          I have updated this issue to be inline with the actual PR (https://github.com/jenkinsci/gerrit-trigger-plugin/pull/252) that I started working on. The semantics you described is exactly what this fix will address.

          Show
          scoheb Scott Hebert added a comment - I have updated this issue to be inline with the actual PR ( https://github.com/jenkinsci/gerrit-trigger-plugin/pull/252 ) that I started working on. The semantics you described is exactly what this fix will address.
          Hide
          nasjenkins Naseer Siddique added a comment -

          Perfect. Thanks!

          Show
          nasjenkins Naseer Siddique added a comment - Perfect. Thanks!
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Scott Hebert
          Path:
          src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritDynamicUrlProcessor.java
          src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/data/GerritProject.java
          src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger/config.jelly
          src/main/webapp/trigger/help-GerritTriggerConfiguration.html
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/gerritnotifier/job/rest/BuildCompletedRestCommandJobHudsonTest.java
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritProjectListTest.java
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTriggerTest.java
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/WorkflowTest.java
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/data/GerritProjectInterestingTest.java
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/data/GerritProjectWithFilesInterestingTest.java
          src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/mock/DuplicatesUtil.java
          http://jenkins-ci.org/commit/gerrit-trigger-plugin/897bb7bdd132530b28401f0a628ce46d194f3c01
          Log:
          Option to permit disable Strict Forbidden files

          Add option to gerrit project to permit the disable of the default
          strict forbidden file verification of an event

          Enabling this option will allow an event to trigger a build if the event contains BOTH one or more wanted file paths/topics/branches AND one or more forbidden file paths.

          In other words, with this option, the build will not get triggered if the change contains only forbidden files, otherwise it will get triggered.

          [FIXED JENKINS-30620]

          Change-Id: Icb84111c82a38db722ec09bf7538441bc1b9424a

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Scott Hebert Path: src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritDynamicUrlProcessor.java src/main/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/data/GerritProject.java src/main/resources/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger/config.jelly src/main/webapp/trigger/help-GerritTriggerConfiguration.html src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/gerritnotifier/job/rest/BuildCompletedRestCommandJobHudsonTest.java src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritProjectListTest.java src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTriggerTest.java src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/WorkflowTest.java src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/data/GerritProjectInterestingTest.java src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/data/GerritProjectWithFilesInterestingTest.java src/test/java/com/sonyericsson/hudson/plugins/gerrit/trigger/mock/DuplicatesUtil.java http://jenkins-ci.org/commit/gerrit-trigger-plugin/897bb7bdd132530b28401f0a628ce46d194f3c01 Log: Option to permit disable Strict Forbidden files Add option to gerrit project to permit the disable of the default strict forbidden file verification of an event Enabling this option will allow an event to trigger a build if the event contains BOTH one or more wanted file paths/topics/branches AND one or more forbidden file paths. In other words, with this option, the build will not get triggered if the change contains only forbidden files, otherwise it will get triggered. [FIXED JENKINS-30620] Change-Id: Icb84111c82a38db722ec09bf7538441bc1b9424a

            People

            • Assignee:
              scoheb Scott Hebert
              Reporter:
              scoheb Scott Hebert
            • Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: