SLURMGraph plugin

[chrisgorgo]

No description provided.

[blakedewey]

I think the documentation on the "Graph" plugins is a little weak and I always have trouble distinguishing the plugin functions. Maybe this would be a good time to update the docs on these, since we are adding more graph options.

[chrisgorgo]

Good point!

On Mon, Jul 13, 2015 at 4:14 PM, Blake Dewey [email protected]
wrote:

I think the documentation on the "Graph" plugins is a little weak and I
always have trouble distinguishing the plugin functions. Maybe this would
be a good time to update the docs on these, since we are adding more graph
options.

—
Reply to this email directly or view it on GitHub
#1136 (comment).

[chrisgorgo]

What about now?

[satra]

run_without_submitting shouldn't apply to a graph plugin - that option is to ensure nodes can run on the monitoring/submitting node. in the graph plugin case, since the scheduler takes over, this check would be unnecessary. i know this was taken from the SGEgraph code (and i didn't review it closely enough :( ).

[chrisgorgo]

I cleaned it up and added a small fix to SGEGraph

[satra]

LGTM