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

Bitbucket PRs are blocked by excluded branches not being able to report progress

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The recent Bitbucket behaviour is great but sadly the option of exclude branches that are filed as PRs is blocking PRs from being merged since the branch might be building and report as in progress and never actually finish because it is now being excluded from reporting.

        Attachments

          Issue Links

            Activity

            Hide
            thomaskeller Thomas Keller added a comment -

            This is really a blocker for the usage of this plugin. The only way to circumvent this issue is to build each PR twice, by disabling the option "exclude branches that are also filed as PRs", but this will just raise the load on the node for no reason.

            I figured if a branch pattern could be applied that would only apply to non-PR branches this could also be worked around (by including only named, important branches there), but sadly, the implemented regex / wildcard filters read NOTE: this filter will be applied to all branch like things, including change requests so this is not an option.

            Any reason why the quickfix from Benjamin Fuchs hasn't been considered yet?

            Show
            thomaskeller Thomas Keller added a comment - This is really a blocker for the usage of this plugin. The only way to circumvent this issue is to build each PR twice, by disabling the option "exclude branches that are also filed as PRs", but this will just raise the load on the node for no reason. I figured if a branch pattern could be applied that would only apply to non-PR branches this could also be worked around (by including only named, important branches there), but sadly, the implemented regex / wildcard filters read  NOTE: this filter will be applied to all branch like things, including change requests  so this is not an option. Any reason why the quickfix from Benjamin Fuchs hasn't been considered yet?
            Hide
            therealwaldo Will Freeman added a comment - - edited

            Note; this is a full on blocker for us, causing huge headaches on a daily basis.

             

            One example of the behaviour is as follows (in addition to others listed above):

            1. Build triggered, updates Bitbucket commit build status as 'build in progress', build starts
            2. Job is marked as disabled (can be because PR is merged, branch is gone, or simply that the configuration of the job has changed)
            3. Bitbucket status is frozen as 'in progress', regardless of if the job finishes.

             

            I believe this all hinges on the fact that a disabled job never fires an update if the job is disabled, regardless of the build state, and whether the PR exists/is closed, etc.

             

            Triggers are fired on change, not poll.

            You can also simulate by opening a pull request, closing it before the build finishes, then opening another PR with the exact same head, and you'll never be able to merge that PR (unless you push yet another change).

            Show
            therealwaldo Will Freeman added a comment - - edited Note; this is a full on blocker for us, causing huge headaches on a daily basis.   One example of the behaviour is as follows (in addition to others listed above): Build triggered, updates Bitbucket commit build status as 'build in progress', build starts Job is marked as disabled (can be because PR is merged, branch is gone, or simply that the configuration of the job has changed) Bitbucket status is frozen as 'in progress', regardless of if the job finishes.   I believe this all hinges on the fact that a disabled job never fires an update if the job is disabled, regardless of the build state, and whether the PR exists/is closed, etc.   Triggers are fired on change, not poll. You can also simulate by opening a pull request, closing it before the build finishes, then opening another PR with the exact same head, and you'll never be able to merge that PR (unless you push yet another change).
            Hide
            vollmilch Benjamin Fuchs added a comment - - edited

            Will Freeman Thomas Keller I just created a pull request from my workaround. We are using this already in our productiv system.

            Show
            vollmilch Benjamin Fuchs added a comment - - edited Will Freeman Thomas Keller I just created a pull request from my workaround. We are using this already in our productiv system.
            Hide
            mrkita Morgan Kita added a comment -

            This is a total blocker for our team, where we first build Feature-Branch builds without PRs and at some point create a PR, where there can be an active branch build at the time of the creation.

             

            This is definitely not a minor bug, and makes this feature unusable as is. If there is a workaround, can someone look into its mergability please?

            Show
            mrkita Morgan Kita added a comment - This is a total blocker for our team, where we first build Feature-Branch builds without PRs and at some point create a PR, where there can be an active branch build at the time of the creation.   This is definitely not a minor bug, and makes this feature unusable as is. If there is a workaround, can someone look into its mergability please?
            Hide
            martinm82 Martin M added a comment - - edited

            Benjamin Fuchs, cool that you have a workaround. Any chances you will create a PR so that everybody can benefit from it? Or is the change just a workaround/hack?
            Maybe some of the maintainers could take a look and judge whether the changes are fine?

            BTW, I guess this is the branch, right: https://github.com/jenkinsci/bitbucket-branch-source-plugin/compare/master...benjaminfuchs:JENKINS-45643

            Show
            martinm82 Martin M added a comment - - edited Benjamin Fuchs , cool that you have a workaround. Any chances you will create a PR so that everybody can benefit from it? Or is the change just a workaround/hack? Maybe some of the maintainers could take a look and judge whether the changes are fine? BTW, I guess this is the branch, right: https://github.com/jenkinsci/bitbucket-branch-source-plugin/compare/master...benjaminfuchs:JENKINS-45643

              People

              • Assignee:
                Unassigned
                Reporter:
                casz Joseph Petersen
              • Votes:
                5 Vote for this issue
                Watchers:
                11 Start watching this issue

                Dates

                • Created:
                  Updated: