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

Improve the performance of scheduling a build

XMLWordPrintable

      The current API in scm-api allows to re-index a branch source using the method SCMSourceOwner.onSCMSourceUpdated().

      Currently we are using this method to force a reindexing when a webhook event is received:

      for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
    for (SCMSource source : owner.getSCMSources()) {
        if (source instanceof GitHubSCMSource) {
            GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
            if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                    gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                owner.onSCMSourceUpdated(gitHubSCMSource);
            }
        }
    }
}

      We need to improve this because when Jenkins receives an event from a webhook we want to be able to determinate:

      1. if we have to re-index my GitHub Organization
      2. if we have to re-index all my branches and pull requests in a GitHub repository
      3. Or simplify if we have to schedule a build because there is a new commit in a pull requests

          [JENKINS-34600] Improve the performance of scheduling a build

          Manuel Recena Soto created issue -
          Manuel Recena Soto made changes -
          Description Original: The current API in {{scm-api}} allows to re-index a branch source using the method {{SCMSourceOwner.onSCMSourceUpdated()}}}.

          Currently we are using this method to force a reindexing when a webhook event is received:

          {code:java}
          for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
              for (SCMSource source : owner.getSCMSources()) {
                  if (source instanceof GitHubSCMSource) {
                      GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
                      if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                              gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                          owner.onSCMSourceUpdated(gitHubSCMSource);
                      }
                  }
              }
          }
          {code}
                       		New: The current API in {{scm-api}} allows to re-index a branch source using the method {{SCMSourceOwner.onSCMSourceUpdated()}}.

          Currently we are using this method to force a reindexing when a webhook event is received:

          {code:java}
          for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
              for (SCMSource source : owner.getSCMSources()) {
                  if (source instanceof GitHubSCMSource) {
                      GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
                      if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                              gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                          owner.onSCMSourceUpdated(gitHubSCMSource);
                      }
                  }
              }
          }
          {code}
            
          Manuel Recena Soto made changes -
          Summary Original: Improve the performance of schedule a build New: Improve the performance of scheduling a build
          Manuel Recena Soto made changes -
          Description Original: The current API in {{scm-api}} allows to re-index a branch source using the method {{SCMSourceOwner.onSCMSourceUpdated()}}.

          Currently we are using this method to force a reindexing when a webhook event is received:

          {code:java}
          for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
              for (SCMSource source : owner.getSCMSources()) {
                  if (source instanceof GitHubSCMSource) {
                      GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
                      if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                              gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                          owner.onSCMSourceUpdated(gitHubSCMSource);
                      }
                  }
              }
          }
          {code}
                       		New: The current API in {{scm-api}} allows to re-index a branch source using the method {{SCMSourceOwner.onSCMSourceUpdated()}}.

          Currently we are using this method to force a reindexing when a webhook event is received:

          {code:java}
          for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
              for (SCMSource source : owner.getSCMSources()) {
                  if (source instanceof GitHubSCMSource) {
                      GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
                      if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                              gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                          owner.onSCMSourceUpdated(gitHubSCMSource);
                      }
                  }
              }
          }
          {code}

          We need to improve this because when Jenkins receives an event from a webhook we want to be able to determinate:

          # if we have to re-index all my GitHub Organization
          # if we have to re-index all my branches and pull requests in a GitHub repository
          # Or simplify if we have to schedule a build because there is a new commit in a pull requests
            
          Manuel Recena Soto made changes -
          Description Original: The current API in {{scm-api}} allows to re-index a branch source using the method {{SCMSourceOwner.onSCMSourceUpdated()}}.

          Currently we are using this method to force a reindexing when a webhook event is received:

          {code:java}
          for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
              for (SCMSource source : owner.getSCMSources()) {
                  if (source instanceof GitHubSCMSource) {
                      GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
                      if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                              gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                          owner.onSCMSourceUpdated(gitHubSCMSource);
                      }
                  }
              }
          }
          {code}

          We need to improve this because when Jenkins receives an event from a webhook we want to be able to determinate:

          # if we have to re-index all my GitHub Organization
          # if we have to re-index all my branches and pull requests in a GitHub repository
          # Or simplify if we have to schedule a build because there is a new commit in a pull requests
                       		New: The current API in {{scm-api}} allows to re-index a branch source using the method {{SCMSourceOwner.onSCMSourceUpdated()}}.

          Currently we are using this method to force a reindexing when a webhook event is received:

          {code:java}
          for (final SCMSourceOwner owner : SCMSourceOwners.all()) {
              for (SCMSource source : owner.getSCMSources()) {
                  if (source instanceof GitHubSCMSource) {
                      GitHubSCMSource gitHubSCMSource = (GitHubSCMSource) source;
                      if (gitHubSCMSource.getRepoOwner().equals(changedRepository.getUserName()) &&
                              gitHubSCMSource.getRepository().equals(changedRepository.getRepositoryName())) {
                          owner.onSCMSourceUpdated(gitHubSCMSource);
                      }
                  }
              }
          }
          {code}

          We need to improve this because when Jenkins receives an event from a webhook we want to be able to determinate:

          # if we have to re-index my GitHub Organization
          # if we have to re-index all my branches and pull requests in a GitHub repository
          # Or simplify if we have to schedule a build because there is a new commit in a pull requests
            
          Manuel Recena Soto made changes -
          Assignee New: Manuel Recena Soto [ recena ]
          Jesse Glick made changes -
          Labels New: api performance
          Jesse Glick made changes -
          Link New: This issue is related to JENKINS-34727 [ JENKINS-34727 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 170761 ] New: JNJira + In-Review [ 184027 ]
          Dominic Scheirlinck made changes -
          Link New: This issue is related to JENKINS-36121 [ JENKINS-36121 ]
          Stephen Connolly made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Closed [ 6 ]

            recena Manuel Recena Soto
            recena Manuel Recena Soto
            Votes:
            2 Vote for this issue
            Watchers:
            8 Start watching this issue

              Created:
              Updated:
              Resolved: