Skip to content
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

suppress stdout for cvmfs_server commands #3612

Open
jose-caballero opened this issue Jun 8, 2024 · 3 comments
Open

suppress stdout for cvmfs_server commands #3612

jose-caballero opened this issue Jun 8, 2024 · 3 comments
Assignees
Milestone

Comments

@jose-caballero
Copy link

Hi,

can I suggest to implement the Rule of Silence for the cvmfs_server commands?
http://catb.org/esr/writings/taoup/html/ch01s06.html

Only when a command like "cvmfs_server publish" fails, it could send to stderr the error. Otherwise, remain silent unless a -v (verbose) flag is passed.

Cheers,
Jose

@vvolkl
Copy link
Contributor

vvolkl commented Jun 9, 2024

Hi Jose,

I like the suggestion! However I suspect this may break user scripts - I'll put it on the agenda for tomorrow's coordination meeting.

@jose-caballero
Copy link
Author

Maybe the opposite. Leave as it is, but add a -q (quiet) option to suppress the stdout, and only send error messages and tracebacks to the stderr.

@DrDaveD
Copy link
Contributor

DrDaveD commented Jun 10, 2024

I wouldn't want to change the default behavior at this point since people expect it, but I could see some interest in a -q option on the publish command which people may run interactively. The snapshot command is usually put into log files and sometimes the default verbose output is useful for debugging things so I don't think it would be good to reduce that output. It's true that it can sometimes be challenging to find the most useful output when looking for problems but mostly I find it is the messages about aborting because another snapshot is already in progress that get in the way more than the progress messages. They look like this:

another update is in progress... aborting
ERROR from cvmfs_server, skipping push!

Maybe those messages should be changed to be just informational and not say anything about aborting or that there was an ERROR, since it is normal routine operation.

@DrDaveD DrDaveD self-assigned this Jul 10, 2024
@DrDaveD DrDaveD added this to the 2.12 milestone Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants