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

The SCMSource.setOwner(owner) contract needs updating to include ensuring that an ID has been assigned





      We need to ensure that issues like JENKINS-48571 are more easily self-diagnosable by users.

      At first glance there are two ways this could be solved:

      1. We could enforce the id assignment by throwing an IllegalStateException or similar if the id is null at the time of setOwner(non-null)
      2. We could ensure an id assignment by assigning one if the id is null at the time of setOwner(non-null) 

      There may also be other potential solutions.

      Acceptance Criteria

      • Assessment criteria for selection of a proposed solution have been defined and reviewed by stephenconnolly and michaelneale
      • The list candidate solutions to be assessed has been defined
      • The results of the assessment process have been reviewed with stephenconnolly and michaelneale and the winner agreed.
      • The winning solution has been implemented.
      • The documentation has been updated to include the impact.
      • Minimization of the risk of "Build storms" has been included in the assessment criteria

      Critical assumpitions

      (if any of these prove to be broken in the process of resolving this ticket then a replan is required)

      • There is no good reason to call setId after the owner has been assigned.


        Issue Links


            stephenconnolly Stephen Connolly created issue -
            stephenconnolly Stephen Connolly made changes -
            Field Original Value New Value
            Link This issue is related to JENKINS-48571 [ JENKINS-48571 ]
            stephenconnolly Stephen Connolly made changes -
            Issue Type Bug [ 1 ] Improvement [ 4 ]
            abayer Andrew Bayer made changes -
            Priority Critical [ 2 ] Major [ 3 ]
            felipecassiors Felipe Santos made changes -
            Link This issue is related to JENKINS-61469 [ JENKINS-61469 ]
            abayer Andrew Bayer made changes -
            Assignee Andrew Bayer [ abayer ]


              Unassigned Unassigned
              stephenconnolly Stephen Connolly
              2 Vote for this issue
              4 Start watching this issue