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

Warnings not being printed #502

Closed
baryluk opened this issue Dec 25, 2023 · 2 comments
Closed

Warnings not being printed #502

baryluk opened this issue Dec 25, 2023 · 2 comments
Assignees
Labels
enhancement An issue to improve current behavior minor
Milestone

Comments

@baryluk
Copy link

baryluk commented Dec 25, 2023

$ tomb lock secret.tomb -k secret.tomb.key
tomb  .  File is not yet a tomb: secret.tomb
tomb  .  Valid tomb file found: secret.tomb
tomb  .  Commanded to lock tomb secret.tomb
losetup: secret.tomb: failed to set up loop device: Permission denied
tomb [E] Loopback mount failed: secret.tomb on /dev/loop1

This is not due to sudo, but actually because root has no permission to the location this file is on (sshfs). As you can see, it is able to run losetup (using sudo), but losetup fails. This can be solved on my side, by doing sshfs mount with -o allow_other.

Anyway, diagnostic is a bit weird a missing. The thing is if I strace -f -s 1000 -v -e execve ..., I see some extra diagnostics:

tomb [W] Loop mount of volumes is not possible on this machine, this error
tomb [W] often occurs on VPS and kernels that don't provide the loop module.
tomb [W] It is impossible to use Tomb on this machine under these conditions.

Not sure why it does not show normally.

tomb 2.10, from Debian testing

@jaromil
Copy link
Member

jaromil commented May 11, 2024

Hi! I think this is due to the default debug level which excludes warnings. Worth cheking if they aren't many and if generally useful then enable them to be shown by default.

@jaromil jaromil self-assigned this May 11, 2024
@jaromil jaromil added minor enhancement An issue to improve current behavior labels May 11, 2024
@jaromil jaromil added this to the 3.0 milestone May 11, 2024
@jaromil
Copy link
Member

jaromil commented May 12, 2024

tests currently show warnings by default so this seems fixed.

@jaromil jaromil closed this as completed May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An issue to improve current behavior minor
Projects
None yet
Development

No branches or pull requests

2 participants