-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUG: Attempting to copy non-existent file with pfcp #28
Comments
Well, at least it exits with an error message. The user won't be fooled, but it would be nice to have the exit code updated so that a script can test for error. Not necessary for Secure Campaign, I would say, but would be good to add in the near future. |
This is due to the wrapper script that actually calls pftool. We could have it pass back the error pretty easily I believe.
|
What is a non-existent file? From: Gregory Geller [mailto:notifications@github.com] When attempting to copy a non-existent file using pftool (pfcp), it should generate an error. It does, but $? is still set to 0. -bash-4.1$ pfcp a b get_base_path -- Failed to stat path a MPI_ABORT was invoked on rank 0 in communicator MPI_COMM_WORLD NOTE: invoking MPI_ABORT causes Open MPI to kill all MPI processes. exactly when Open MPI kills them. mpirun has exited due to process rank 0 with PID 11519 on
This may have caused other processes in the application to be terminated by signals sent by mpirun (as reported here). Launched /opt/campaign/pftool/installed/bin/pfcp from host cc-fta03.localdomain at: Mon Jun 20 16:22:25 MDT 2016 — |
There's a test to use pfcp to copy a file that isn't there. Say someone mistypes a filename, for example. What does pfcp do when you tell it to copy a file that is not present? |
Come to think of it, I actually commented out that test, in a private version, because ... what if you don't have fuse mounted? [Apologies to Greg, since I suggested he report this. I'd suggest he throw things up here, so we have a record. I didn't make the connection with the unmounted-fuse thing until just now.] |
So, if we don't want to set $? when attempting to copy a non-existent file, then what do you suggest I do to automate a test for proper behavior? Look for "Failed" or "MPI_ABORT" in the output? |
I don't understand Jeff's comment. I think this is a bug and specifically for the reason you mention. A script needs some way to detect if a command fails. |
We should fix pfcp like Dave suggested. Users will always have marfs appear to be mounted somehow. Us squirrels can just deal. |
Jeff and I had a chat conversation about this. The pf* Python scripts were written with the assumption that file systems would always be mounted. This is not the case on the Batch FTAs in MarFS. the pf* scripts need to be changed so that they do not assume the file system is mounted. They need to look at the output of PFTool and return an error if PFTool returns an error. In the case of a FNF (file not found), the pf* scripts need to return some intelligible output to the user as well as set the return code to an error that a script can catch and process. |
Sounds good. I will look into it this morning. |
This bug also exists with pfcm and pfls. I have fixed both of them with my latest commit. |
When attempting to copy a non-existent file using pftool (pfcp), it should generate an error.
It does, but $? is still set to 0.
The text was updated successfully, but these errors were encountered: