| Issue 3099: | Using the its-base branch limiting feature will disables its actions | |
| 2 people starred this issue and may be notified of changes. | Back to list |
Sign in to add a comment
|
The its-base documentation[1] says.. "The issue tracker system integration can be limited to specific branches by setting `plugin.@PLUGIN@.branch`. The branches may be configured using explicit branch names, ref patterns, or regular expressions". This feature does not seem to work. I've attempted to tried setting it to both 'branches = master' and 'branches = refs/heads/master'. In both cases the actions from an its based plugin does not run. Affected Version: master What steps will reproduce the problem? 1. Install an its plugin (its-jira, its-bugzilla, etc..). 2. Set branch limiting feature in project.config of "All-Projects" to 'branch = master' or 'branch = refs/heads/master' 3. Setup the rules for the its-based plugin to do something on an action (patchset-created, comment-added, etc..) 4. Push a new change from the master branch, add new patchsets to that change, add reviews, etc.. What is the expected output? What do you see instead? I expect the its-based plugin to do the action that I setup in step #3 but no actions ever occur. Please provide any additional information below. It works when if I don't set any branches, specifically I mean when I remove the 'branch' setting from project.config. [1] https://gerrit.googlesource.com/plugins/its-base/+/master/src/main/resources/Documentation/config-common.md
Jan 12, 2015
Project Member
#1
zaro0508
Status:
ChangeUnderReview
Jan 20, 2015
Another change to mitigate this issue: https://gerrit-review.googlesource.com/#/c/63230/
May 12, 2015
Hi, I ran into this problem as well. It seems like the logic was working fine, but the wrong argument was being passed. We should be using refNames to match against the refPattern, not branch name. https://gerrit-review.googlesource.com/#/c/67850/
May 13, 2015
(No comment was entered for this change.)
Status:
Submitted
May 13, 2015
https://gerrit-review.googlesource.com/67850 |
||||||||||
| ► Sign in to add a comment | |||||||||||