[JIRA] (JENKINS-56201) GCE plugin shouldn't cleanup nodes from different jenkins masters

2019-04-11 Thread janczuk.mate...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Mateusz Janczuk commented on  JENKINS-56201  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: GCE plugin shouldn't cleanup nodes from different jenkins masters   
 

  
 
 
 
 

 
 Great news! thanks Karol Lassak, Rachel Yen    
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
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.


[JIRA] (JENKINS-56201) GCE plugin shouldn't cleanup nodes from different jenkins masters

2019-04-10 Thread rachel...@google.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Rachel Yen commented on  JENKINS-56201  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: GCE plugin shouldn't cleanup nodes from different jenkins masters   
 

  
 
 
 
 

 
 Thanks to Karol Lassak, we'll have unique identifiers for the different clouds. I'll resolve this with our next release.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
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.


[JIRA] (JENKINS-56201) GCE plugin shouldn't cleanup nodes from different jenkins masters

2019-02-20 Thread ing...@ingwar.eu.org (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Karol Lassak commented on  JENKINS-56201  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: GCE plugin shouldn't cleanup nodes from different jenkins masters   
 

  
 
 
 
 

 
 At the moment slaves dont have any connection with "jenkins master", except instanceUniqueId so its not possible to distinguish from what master they were created..   There is another issue with that that all jenskins instances with same cloud name uses same pool (limit) of instances as mentioned here. https://issues.jenkins-ci.org/browse/JENKINS-52649   There are two options: 1. Either use different cloud name for each cloud (Info how to use it should be added to help), It allows of creating shared pools of slaves, but cleaning needs to be fixed somehow (probably by adding extra ID mentioned in point 2) 2. Another solution would be to just generate some random instance ID and use it as id.    
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)  
 

  
 

   





-- 
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.


[JIRA] (JENKINS-56201) GCE plugin shouldn't cleanup nodes from different jenkins masters

2019-02-19 Thread janczuk.mate...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Mateusz Janczuk created an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-56201  
 
 
  GCE plugin shouldn't cleanup nodes from different jenkins masters   
 

  
 
 
 
 

 
Issue Type: 
  Bug  
 
 
Assignee: 
 Evan Brown  
 
 
Components: 
 google-compute-engine-plugin  
 
 
Created: 
 2019-02-19 13:55  
 
 
Priority: 
  Major  
 
 
Reporter: 
 Mateusz Janczuk  
 

  
 
 
 
 

 
 Upgrade of plugin google-compute-engine-plugin to version 1.0.10 causes that vm instances created in this same GCP project but from different Jenkins Master are removed. It looks that these changes https://github.com/jenkinsci/google-compute-engine-plugin/blob/7ee660eae173d1010388c557c378f2b7f8bdd105/src/main/java/com/google/jenkins/plugins/computeengine/CleanLostNodesWork.java#L66 trying to cleanup all nodes with this same instanceUniqueId which is resolved only from cloud name. I think it is quite possible that for simplicity cloud name in most cases is identical with project id. So detection of unique instances should probably ensure that vm instances comes from this same jenkins master before they are deleted. As workaround different cloud names could be used on each jenkins master.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment