Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

addnode options / loadbalancer stuff #20

Open
wants to merge 18 commits into
from

Conversation

Projects
None yet
3 participants

I added:

-- options to addnode and create_nodes
-- ability to balance specified queue to loadbalancer (and various related options)
-- generalized remove_from_sge function in ClusterSetup
-- added sge_utils.py for various SGE utility functions used in several places

Daniel Yamins and others added some commits Jun 9, 2010

Daniel Yamins compound node instance types ce7abd9
@yamins81 yamins81 Merge branch 'master' of git://github.com/jtriley/StarCluster
Conflicts:
	starcluster/cluster.py
	starcluster/config.py
	starcluster/static.py
93764c3
@yamins81 yamins81 1) additional node-type arguments to addnode, 2) generalized remove_f…
…rom_sge, 3) load balancer for specified queue and works for queues with 0 nodes
f8b19fb
@yamins81 yamins81 pep 8 compliance 6fee461
@yamins81 yamins81 load balancer takes queue option; add_nodes takes various node option…
…s; remove_from_sge generalized; added sge_utils.py module
80be1f6

@jtriley jtriley and 1 other commented on an outdated diff Jun 1, 2011

starcluster/sge_utils.py
+h_data INFINITY
+s_stack INFINITY
+h_stack INFINITY
+s_core INFINITY
+h_core INFINITY
+s_rss INFINITY
+h_rss INFINITY
+s_vmem INFINITY
+h_vmem INFINITY
+"""
+
+HGCONF_TEMPLATE = """group_name $HGNAME
+hostlist NONE
+"""
+
+import BeautifulSoup
@jtriley

jtriley Jun 1, 2011

Owner

Do we really need BeautifulSoup here? If possible it'd be nice to use xml.minidom instead since it's included in the standard library. If we really need it we'll need to add BeautifulSoup to the dependency list in setup.py....

@yamins81

yamins81 Jun 1, 2011

No, I'm totally sure that this can be replaced with xml.minidom code ... it was just easier for me since I'm more used to BueaituflSoup. I'll have alok at replacing it in the next couple of days ...

@jtriley

jtriley Jun 1, 2011

Owner

OK, great, thanks!

@yamins81

yamins81 Jun 3, 2011

OK that should be done ...

I merged stuff from your master as well, there were a couple of conflicts in the loadbalancer ... I think I merged them correctly but can you go over those to make sure?

canesin commented Nov 23, 2011

Hi, I have seen that the pull don't apply cleanly .. how is the progress of adding it to the master or develop branch ??

Owner

jtriley commented Dec 16, 2011

@canesin My apologies for the delay, yes this pull request needs to be updated/cleaned in order to be merged into the develop branch. I'm in the process of trying to push out a 0.92.2 release in the next few days so this will have to wait until after the next release.

@jtriley jtriley referenced this pull request Jan 20, 2012

Closed

Various things #66

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment