grisu / gricli

Grisu commandline client
7 stars 2 forks source link

jobnames with spaces not supported #96

Closed sina-masoud-ansari closed 13 years ago

sina-masoud-ansari commented 13 years ago

Not sure if this is a feature or a bug ; )

[smas036@login1 ~]$ griclish-dev Logged in to backend: BeSTGRID gricli> set jobname myjob gricli> set group /nz/nesi gricli> set application UnixCommands gricli> submit "sleep 40" job name is myjob gricli>

[smas036@login1 ~]$ griclish-dev Logged in to backend: BeSTGRID gricli> set group /nz/nesi gricli> set application UnixCommands gricli> set jobname "my job" gricli> submit "sleep 40" javax.xml.ws.soap.SOAPFaultException: Unknown error while trying to create job: null at com.sun.xml.ws.fault.SOAP11Fault.getProtocolException(SOAP11Fault.java:189) at com.sun.xml.ws.fault.SOAPFaultBuilder.createException(SOAPFaultBuilder.java:122) at com.sun.xml.ws.client.sei.SyncMethodHandler.invoke(SyncMethodHandler.java:119) at com.sun.xml.ws.client.sei.SyncMethodHandler.invoke(SyncMethodHandler.java:89) at com.sun.xml.ws.client.sei.SEIStub.invoke(SEIStub.java:118) at $Proxy41.createJob(Unknown Source) at grisu.frontend.model.job.JobObject.createJob(JobObject.java:460) at grisu.gricli.command.SubmitCommand.createJob(SubmitCommand.java:39) at grisu.gricli.command.SubmitCommand.execute(SubmitCommand.java:50) at grisu.gricli.Gricli.runCommand(Gricli.java:204) at grisu.gricli.Gricli.executionLoop(Gricli.java:86) at grisu.gricli.Gricli.main(Gricli.java:179) command failed. Either connection to server failed, or this is gricli bug. Please send /home/smas036/.grisu.beta/gricli.debug to eresearch-admin@auckland.ac.nz together with description of what triggered the problem

yuriyh commented 13 years ago

this one should be fixed upstream by providing appropriate error message.

makkus commented 13 years ago

Yeah, gricli should check whether jobname has got whitespace in it and throw error. Not supported by Grisu.