<div dir="ltr"><div><div><div><div>Hi Ryan,<br></div>No real reason, except that Justin added remove_nodes more than a year after I wrote the LB:<br><br><a href="https://github.com/jtriley/StarCluster/blame/develop/starcluster/cluster.py#L200">https://github.com/jtriley/StarCluster/blame/develop/starcluster/cluster.py#L200</a><br><br></div>So the loadbalancer uses remove_node() instead of remove_nodes() plural. Me/You/Anyone else could modify and test the load balancer with the new function, it should not be too difficult. I would be interested in adding it but don't have a lot of spare time right now.<br><br></div>Best,<br></div>Raj<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 21, 2015 at 3:11 PM, Ryan G <span dir="ltr"><<a href="mailto:ngsbioinformatics@gmail.com" target="_blank">ngsbioinformatics@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I'm using loadbalance to shrink my cluster after submitting a bunch of jobs. I noticed that StarClustser may identify a group of nodes that can be shutdown. It then shuts them down one by one by running each plugin for every node. Why not run the plugins once and remove the group of nodes at the same time? This seems a bit inefficient.<br></div></div>
<br>_______________________________________________<br>
StarCluster mailing list<br>
<a href="mailto:StarCluster@mit.edu">StarCluster@mit.edu</a><br>
<a href="http://mailman.mit.edu/mailman/listinfo/starcluster" rel="noreferrer" target="_blank">http://mailman.mit.edu/mailman/listinfo/starcluster</a><br>
<br></blockquote></div><br></div>