Details
-
Improvement
-
Status: Resolved (View Workflow)
-
Minor
-
Resolution: Fixed
-
None
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
Naseer Siddique
added a comment - Perfect. Thanks! 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.
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.
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.
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