[JIRA] [digitalocean-plugin] (JENKINS-35022) digitalOcean plugin logic leads to endless API requests, API rate limit exceeded

[alexey.byc...@percona.com (JIRA)]

Title: Message Title

Alexey Bychko created an issue   Jenkins / JENKINS-35022 digitalOcean plugin logic leads to endless API requests, API rate limit exceeded Issue Type: Bug Assignee: Unassigned Components: digitalocean-plugin Created: 2016/May/23 2:27 PM Environment: Jenkins 1.642.2 digitalocean-plugin 0.10 Priority: Major Reporter: Alexey Bychko to reproduce (exact numbers are not important): create DigitalOcean cloud in Jenkins with 10 droplets limit. create 10 different images under that cloud set limit for every droplet == total droplets limit start a lot of build jobs to utilize all the images in the same time (say 10 jobs per image) result: jenkins perform queries to DigitalOcean API all the time to list droplets, create new droplet and so on. it leads to "API rate limit exceeded" error and inability to delete idle slaves automatically. capacity per image is set == total cloud capacity because we want to be able to use the whole cloud capacity with every image created. possible reason: digitalocean-plugin wrongly checks running slaves and perhaps has a bug in logic. possible fix: diff --git a/src/main/java/com/dubture/jenkins/digitalocean/Cloud.java b/src/main/java/com/dubture/jenkins/digitalocean/Cloud.java index 9956c91..13ac3f4 100644 --- a/src/main/java/com/dubture/jenkins/digitalocean/Cloud.java +++ b/src/main/java/com/dubture/jenkins/digitalocean/Cloud.java @@ -168,8 +168,8 @@ public class Cloud extends hudson.slaves.Cloud { count ++; } } - - if (count >= Math.min(instanceCap, slaveTotalInstanceCap)) { + // we can provision new cloud instance only if (count < instanceCap) AND (count < slaveTotalInstanceCap) + if ((count < instanceCap) && (count < slaveTotalInstanceCap)) { return true; } @@ -184,7 +184,7 @@ public class Cloud extends hudson.slaves.Cloud { } } - return count >= Math.min(instanceCap, slaveTotalInstanceCap); + return ((count < instanceCap) && (count < slaveTotalInstanceCap)); } /** Add Comment   This message was sent by Atlassian JIRA (v6.4.2#64017-sha1:e244265) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.