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

          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

          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 ?

          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

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

          (cherry picked from commit cc932c49afa8c2946a32555285e1aa52dc724ddf)

          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/de9835a6797bc7afef9b1605fa21ae0dcf3c6d37 Log: [FIXED JENKINS-31219] A CloudProvisioningListener can prevent provisioning of all clouds instead of just the targeted cloud (cherry picked from commit cc932c49afa8c2946a32555285e1aa52dc724ddf)

          dogfood added a comment -

          Integrated in jenkins_main_trunk #4450
          [FIXED JENKINS-31219] A CloudProvisioningListener can prevent (Revision de9835a6797bc7afef9b1605fa21ae0dcf3c6d37)

          Result = SUCCESS
          ogondza : de9835a6797bc7afef9b1605fa21ae0dcf3c6d37
          Files :

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

          dogfood added a comment - Integrated in jenkins_main_trunk #4450 [FIXED JENKINS-31219] A CloudProvisioningListener can prevent (Revision de9835a6797bc7afef9b1605fa21ae0dcf3c6d37) Result = SUCCESS ogondza : de9835a6797bc7afef9b1605fa21ae0dcf3c6d37 Files : core/src/main/java/hudson/slaves/NodeProvisioner.java

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

              Created:
              Updated:
              Resolved: