• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • Jenkins 2.60.1

      In the changes introduced in 977f922136e8bcdad42ad1448df45a36b2e64f68, it appears that a NPE was introduced here as p can be null.

      FATAL: null
      java.lang.NullPointerException
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:340)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at jenkins.scm.RunWithSCM.calculateCulprits(RunWithSCM.java:129)
      	at hudson.model.AbstractBuild.calculateCulprits(AbstractBuild.java:335)
      	at jenkins.scm.RunWithSCM.getCulprits(RunWithSCM.java:93)
      	at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:625)
      	at hudson.model.Run.execute(Run.java:1760)
      	at hudson.matrix.MatrixRun.run(MatrixRun.java:146)
      	at hudson.model.ResourceController.execute(ResourceController.java:97)
      	at hudson.model.Executor.run(Executor.java:405)
      

          [JENKINS-45516] Fix NPE when calculating culprits

          ian s added a comment -

          Will this not be fixed in 2.60? We've had a lot of issues with this and redhat-stable is still on 2.60

          ian s added a comment - Will this not be fixed in 2.60? We've had a lot of issues with this and redhat-stable is still on 2.60

          Oleg Nenashev added a comment -

          tkj the issue is an LTS candidate. It will likely get backported to 2.60.3 if there is no regressions

          Oleg Nenashev added a comment - tkj the issue is an LTS candidate. It will likely get backported to 2.60.3 if there is no regressions

          ian s added a comment -

          oleg_nenashev oh, i see. Thanks for clarifying

          ian s added a comment - oleg_nenashev oh, i see. Thanks for clarifying

          Jesse Glick added a comment -

          For whatever reason (olivergondza?) this was not considered for 2.60.x, which is now closed, and as the next LTS is 2.73.1, I suppose lts-candidate can be removed.

          Jesse Glick added a comment - For whatever reason ( olivergondza ?) this was not considered for 2.60.x, which is now closed, and as the next LTS is 2.73.1, I suppose lts-candidate can be removed.

          Oleg Nenashev added a comment -

          Yep. I was busy and didn't double-check if all possible changes for 2.60.3 got backported.

           

          Oleg Nenashev added a comment - Yep. I was busy and didn't double-check if all possible changes for 2.60.3 got backported.  

          Code changed in jenkins
          User: Oliver Gondža
          Path:
          content/_data/changelogs/lts.yml
          http://jenkins-ci.org/commit/jenkins.io/b2e34180b23e401165008389e58f4fa57e506b9d
          Log:
          Nothing JENKINS-45516

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: content/_data/changelogs/lts.yml http://jenkins-ci.org/commit/jenkins.io/b2e34180b23e401165008389e58f4fa57e506b9d Log: Nothing JENKINS-45516

          Code changed in jenkins
          User: Oliver Gondža
          Path:
          content/_data/changelogs/lts.yml
          http://jenkins-ci.org/commit/jenkins.io/c599c45154a77a11fe2126a71cf47ca9293bf5d6
          Log:
          Noting JENKINS-45516

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: content/_data/changelogs/lts.yml http://jenkins-ci.org/commit/jenkins.io/c599c45154a77a11fe2126a71cf47ca9293bf5d6 Log: Noting JENKINS-45516

          The fix is included: https://github.com/jenkinsci/jenkins/commits/stable-2.60 (be6d02e648d5c41e5b16e797bdc55873d0f56c56). I must have forgotten updating the tags here. Amending changelog: https://github.com/jenkins-infra/jenkins.io/pull/1100

          Oliver Gondža added a comment - The fix is included: https://github.com/jenkinsci/jenkins/commits/stable-2.60 (be6d02e648d5c41e5b16e797bdc55873d0f56c56). I must have forgotten updating the tags here. Amending changelog: https://github.com/jenkins-infra/jenkins.io/pull/1100

          Jesse Glick added a comment -

          Adding label accordingly.

          Jesse Glick added a comment - Adding label accordingly.

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          content/_data/changelogs/lts.yml
          http://jenkins-ci.org/commit/jenkins.io/879c82c2d49fedb8ec5ab9bc02ebf129a0b12b0f
          Log:
          Merge pull request #1100 from jenkins-infra/olivergondza-patch-1

          Noting JENKINS-45516

          Compare: https://github.com/jenkins-infra/jenkins.io/compare/7b63b9f4d82a...879c82c2d49f

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: content/_data/changelogs/lts.yml http://jenkins-ci.org/commit/jenkins.io/879c82c2d49fedb8ec5ab9bc02ebf129a0b12b0f Log: Merge pull request #1100 from jenkins-infra/olivergondza-patch-1 Noting JENKINS-45516 Compare: https://github.com/jenkins-infra/jenkins.io/compare/7b63b9f4d82a...879c82c2d49f

            jcmoore Curt Moore
            jcmoore Curt Moore
            Votes:
            2 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved: