Branch: refs/heads/master Home: https://github.com/jenkinsci/google-compute-engine-plugin Commit: 2a685d5729e25996d12ddff85ca6482961049c98 https://github.com/jenkinsci/google-compute-engine-plugin/commit/2a685d5729e25996d12ddff85ca6482961049c98 Author: Michael Russell <michael.russ...@elastic.co> Date: 2018-12-19 (Wed, 19 Dec 2018)
Changed paths: M src/main/java/com/google/jenkins/plugins/computeengine/ComputeEngineCloud.java M src/main/java/com/google/jenkins/plugins/computeengine/InstanceConfiguration.java Log Message: ----------- Add all configuration labels to launched slaves If a configuration defined multiple labels only the requiredLabel would be added to the slave when it was created. This meant that if you had a configuration with the labels "linux ubuntu" and started a job for "ubuntu" it would start a new slave with only the label "ubuntu. If you started another job for "linux" the current slave wouldn't match so the plugin would launch a new slave. With these small changes all of the labels will be added to the new slave. Apart from this being unexpected I'm also fixing this so that this plugin plays nicely with the https://wiki.jenkins.io/display/JENKINS/Single+Use+Slave+Plugin plugin. All of our dynamic slaves are created with the tag "immutable" which we are using to determine which slaves should be only used once. This means we don't need need to keep track of which slaves are immutable and which shouldn't be set offline after a single job. Commit: 7c4e44e5c0b42f70428e04edb8947069114ee29a https://github.com/jenkinsci/google-compute-engine-plugin/commit/7c4e44e5c0b42f70428e04edb8947069114ee29a Author: Rachel Yen <rachel...@google.com> Date: 2018-12-19 (Wed, 19 Dec 2018) Changed paths: M src/main/java/com/google/jenkins/plugins/computeengine/ComputeEngineCloud.java M src/main/java/com/google/jenkins/plugins/computeengine/InstanceConfiguration.java Log Message: ----------- Removed String labels as argument Commit: cb5446d346819235322e88f7ba93360e424de503 https://github.com/jenkinsci/google-compute-engine-plugin/commit/cb5446d346819235322e88f7ba93360e424de503 Author: Rachel Yen <rachel...@google.com> Date: 2018-12-19 (Wed, 19 Dec 2018) Changed paths: M src/test/java/com/google/jenkins/plugins/computeengine/ComputeEngineCloudIT.java Log Message: ----------- Added integration test for multiple labels Commit: 82ed40bc07235385a7643c3bb0420f8d5c0087f4 https://github.com/jenkinsci/google-compute-engine-plugin/commit/82ed40bc07235385a7643c3bb0420f8d5c0087f4 Author: Rachel Yen <rachel...@google.com> Date: 2018-12-19 (Wed, 19 Dec 2018) Changed paths: M src/main/java/com/google/jenkins/plugins/computeengine/InstanceConfiguration.java Log Message: ----------- unused import Commit: 5ee20e4f2491823876b5f0f3b8eb1378c90a30fa https://github.com/jenkinsci/google-compute-engine-plugin/commit/5ee20e4f2491823876b5f0f3b8eb1378c90a30fa Author: Rachel Yen <rachel...@google.com> Date: 2018-12-19 (Wed, 19 Dec 2018) Changed paths: M src/test/java/com/google/jenkins/plugins/computeengine/ComputeEngineCloudIT.java Log Message: ----------- Added integration test for checking multiple labels in node Compare: https://github.com/jenkinsci/google-compute-engine-plugin/compare/d803bcf492ee...5ee20e4f2491 **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/ Functionality will be removed from GitHub.com on January 31st, 2019. -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.