Allright! we seem to be getting there. I will test running some scripts for actual jobs on my cluster, and will report back to you.
Thanks! Daniel On 12/16/08, Abhishek Kulkarni <[email protected]> wrote: > This patch fixes two minor bugs with bjs. > Interactive job submission didn't set the NODES environ variable correctly. > A check to ensure if the requested nodes are less than or equal to the nodes > in a pool was added back. > > Signed-off-by: Abhishek Kulkarni <[email protected]> > > Index: bjs.c > =================================================================== > --- bjs.c (revision 751) > +++ bjs.c (working copy) > @@ -581,7 +581,7 @@ > sexp_append_atom(sx, tmp); > for (l = j->nodes.next; l != &j->nodes; l = l->next) { > struct node_alloc_t *n = list_entry(l,struct node_alloc_t,nodes_list); > - sprintf(tmp, "%d", bjs_node_idx[n->node]->node); > + sprintf(tmp, "%s", bjs_node_idx[n->node]->node->name); > sexp_append_atom(sx, tmp); > } > client_send_sx(j->client, sx); > @@ -1149,13 +1149,11 @@ > return -1; > } > } else { > - /* FIXME > if (nodes > p->nnodes) { > bjs_client_error(c, "The maximum number of nodes for pool" > " %s is %d.", p->name, p->nnodes); > return -1; > - } > - */ > + } > } > > /*--- Check number of seconds requested ---*/ > >
