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.

Sign up for GitHub

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

Jump to bottom

qubes-vm-update doesn't print an error when arguments are bad #9565

Closed
RandyTheOtter opened this issue Nov 9, 2024 · 1 comment · Fixed by QubesOS/qubes-core-admin-linux#172
Closed

qubes-vm-update doesn't print an error when arguments are bad #9565

RandyTheOtter opened this issue Nov 9, 2024 · 1 comment · Fixed by QubesOS/qubes-core-admin-linux#172
Assignees
@alimirjamali
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: updates diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. ux User experience

Comments

@RandyTheOtter
Copy link

How to file a helpful issue

Qubes OS release

4.2.3

Brief summary

If you run qubes-vm-update with nonexistent target, it will not print out any error message. This is annoying when there are plenty of targets and you don't know which one causes it.

Steps to reproduce

qubes-vm-update --targets=whatever

Expected behavior

return 64, print something like: error: target 'whatever' not found

Actual behavior

return 64
@RandyTheOtter RandyTheOtter added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Nov 9, 2024
alimirjamali added a commit to alimirjamali/qubes-core-admin-linux that referenced this issue Nov 9, 2024
@alimirjamali
Print & log argument errors instead of just logging
7a5a0fa 
fixes: QubesOS/qubes-issues#9565
@alimirjamali alimirjamali mentioned this issue Nov 9, 2024
Merged
@alimirjamali
Copy link

PR Submitted

Review priority: Low

@andrewdavidwong andrewdavidwong added diagnosed Technical diagnosis has been performed (see issue comments). pr submitted A pull request has been submitted for this issue. C: updates affects-4.2 This issue affects Qubes OS 4.2. labels Nov 9, 2024
@andrewdavidwong andrewdavidwong added the ux User experience label Nov 9, 2024
@qubesos-bot qubesos-bot mentioned this issue Nov 15, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alimirjamali alimirjamali

Labels
affects-4.2 This issue affects Qubes OS 4.2. C: updates diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. ux User experience
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Print & log argument errors instead of just logging alimirjamali/qubes-core-admin-linux
3 participants
@alimirjamali @andrewdavidwong @RandyTheOtter