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

A CloudProvisioningListener can prevent provisioning of all clouds instead of just the targeted cloud

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

      This is a regression introduced in 92a113c152fcd2cb3adc21b1d8b38aa522b802b4

      The CloudProvisioningListener.canProvision(Cloud,Label,int) method is supposed to

      Allow extensions to prevent a cloud from provisioning.

      However the current implementation will prevent all subsequent clouds from provisioning also as the break CLOUD; should actually have been continue CLOUD;

          [JENKINS-31219] A CloudProvisioningListener can prevent provisioning of all clouds instead of just the targeted cloud

          Stephen Connolly created issue -
          Stephen Connolly made changes -
          Summary Original: A CloudProvisioningListener can prevent provisioning of all clouds New: A CloudProvisioningListener can prevent provisioning of all clouds instead of just the targeted cloud

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/slaves/NodeProvisioner.java
          http://jenkins-ci.org/commit/jenkins/cc932c49afa8c2946a32555285e1aa52dc724ddf
          Log:
          [FIXED JENKINS-31219] A CloudProvisioningListener can prevent provisioning of all clouds instead of just the targeted cloud

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/slaves/NodeProvisioner.java http://jenkins-ci.org/commit/jenkins/cc932c49afa8c2946a32555285e1aa52dc724ddf Log: [FIXED JENKINS-31219] A CloudProvisioningListener can prevent provisioning of all clouds instead of just the targeted cloud
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          core/src/main/java/hudson/slaves/NodeProvisioner.java
          http://jenkins-ci.org/commit/jenkins/9339a34967dd095ba6fbaba40b41e33835fde866
          Log:
          Merge pull request #1891 from stephenc/jenkins-31219

          [FIXED JENKINS-31219] A CloudProvisioningListener can prevent provisi…

          Compare: https://github.com/jenkinsci/jenkins/compare/8c8bbbbbad0c...9339a34967dd

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/slaves/NodeProvisioner.java http://jenkins-ci.org/commit/jenkins/9339a34967dd095ba6fbaba40b41e33835fde866 Log: Merge pull request #1891 from stephenc/jenkins-31219 [FIXED JENKINS-31219] A CloudProvisioningListener can prevent provisi… Compare: https://github.com/jenkinsci/jenkins/compare/8c8bbbbbad0c...9339a34967dd

          Daniel Beck added a comment -

          stephenconnolly Should this be an lts-candidate?

          Daniel Beck added a comment - stephenconnolly Should this be an lts-candidate ?
          Stephen Connolly made changes -
          Labels New: lts-candidate

          danielbeck yep, added the label

          Stephen Connolly added a comment - danielbeck yep, added the label

          dogfood added a comment -

          Integrated in jenkins_2.0 #5
          [FIXED JENKINS-31219] A CloudProvisioningListener can prevent (Revision cc932c49afa8c2946a32555285e1aa52dc724ddf)

          Result = SUCCESS
          stephen connolly : cc932c49afa8c2946a32555285e1aa52dc724ddf
          Files :

          • core/src/main/java/hudson/slaves/NodeProvisioner.java

          dogfood added a comment - Integrated in jenkins_2.0 #5 [FIXED JENKINS-31219] A CloudProvisioningListener can prevent (Revision cc932c49afa8c2946a32555285e1aa52dc724ddf) Result = SUCCESS stephen connolly : cc932c49afa8c2946a32555285e1aa52dc724ddf Files : core/src/main/java/hudson/slaves/NodeProvisioner.java
          Oliver Gondža made changes -
          Labels Original: lts-candidate New: 1.642.2-fixed

            stephenconnolly Stephen Connolly
            stephenconnolly Stephen Connolly
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: