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

ghprb failed to test code as it will look like after accepting pull request

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Critical
    • Resolution: Fixed
    • Component/s: ghprb-plugin
    • Labels:
      None
    • Environment:
      ghprb v1.29.6
      Jenkins v1.638
      matrix-project v1.6
      github-api v1.71
      git v2.4.0
      swarm v2.0
      swarm-client v2.0
    • Similar Issues:

      Description

      A recent PR:

      https://github.com/dCache/dcache/pull/2096

      triggered a corresponding build, as anticipated:

      https://ci.dcache.org/view/dCache%202.14/job/dCache-v2.14/88/

      This build detected no problems: the PR merged successfully, the corresponding code compiled cleanly and the downstream jobs (which deploy the build and run functional tests) all passed.

      However, when the pull-request (along with several others) was accepted, the build failed. Analysis of the problem showed that this pull-request alone was responsible for the problem and that, if the pull request was actually testing, the build could not have succeeded – the problem is also manually reproducible: creating a branch from the latest tagged release and manually cherry-picking the corresponding patch results in code that does not build.

      Therefore, for whatever reason, the ghprb plugin is not testing how the code will look after accepting the pull request.

        Attachments

          Activity

          paulmillar Paul Millar created issue -
          paulmillar Paul Millar made changes -
          Field Original Value New Value
          Priority Major [ 3 ] Critical [ 2 ]
          paulmillar Paul Millar made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Closed [ 6 ]
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 168003 ] JNJira + In-Review [ 209597 ]
          josefrosel Josef Rosel made changes -
          Link This issue is blocked by JENKINS-46874 [ JENKINS-46874 ]
          josefrosel Josef Rosel made changes -
          Link This issue is blocked by JENKINS-46874 [ JENKINS-46874 ]

            People

            Assignee:
            janinko Honza Brázdil
            Reporter:
            paulmillar Paul Millar
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: