Bugzilla – Full Text Bug Listing
| Summary: | [JAXB] Unable to specify account on Simple PBS Settings tab | ||
|---|---|---|---|
| Product: | [Tools] PTP | Reporter: | Greg Watson <g.watson> |
| Component: | RM | Assignee: | Albert L. Rossi <arossi> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | arossi |
| Version: | 5.0 | ||
| Target Milestone: | 5.0.2 | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
|
Description
Greg Watson
Jay ran into the same issue. Pretty simple to add that widget. Go ahead and add it to the basic PBS if you want. What you're really pointing out here is this question: how "simple" is the simple / basic? Of course, you can always switch to the advanced tab. You don't have to resort to your own script. I think "simple" should at least cover the arguments to qsub. (In reply to comment #3) > I think "simple" should at least cover the arguments to qsub. There are no necessary arguments to qsub, as far as I know. Every installation can set up the system call to allow as few or as many. Using this idea, you should also allow _all_ the -l resource arguments in Simple, since on NCSA's machines you could specify these on the command line too (in fact, I think you basically pass all the PBS pragmas as command-line flags). A text field that allows the user to enter a list of resources wouldn't be a bad thing. Except it would become difficult to validate anything in it. Since the submission mechanism is not to use the command-line flags because those are not necessarily supported, but to replace these values on the script, one would also have to rewrite the script so that it has a single -l line. These are not difficult changes if you really think it makes this more usable -- now that the documentation/tutorial exists to demonstrate how to modify the underlying definition to get this. You could even do away with the Advanced Tab altogether, and just provide the "Standard" and the Customized tab controllers. I've modified the two PBS definitions to carry Account text widget. The fix will be committed when I also commit the validator patch. Fixed in 5.0.2 |