Skip to content

qvm-device: add --force flag for dom0 device attachment#466

Open
nihalxkumar wants to merge 1 commit intoQubesOS:mainfrom
nihalxkumar:main
Open

qvm-device: add --force flag for dom0 device attachment#466
nihalxkumar wants to merge 1 commit intoQubesOS:mainfrom
nihalxkumar:main

Conversation

@nihalxkumar
Copy link
Copy Markdown

@nihalxkumar nihalxkumar commented Apr 8, 2026

@parulin
Copy link
Copy Markdown

parulin commented Apr 8, 2026

It should also be documented.

@nihalxkumar
Copy link
Copy Markdown
Author

It should also be documented.

Thanks @parulin. The introduction of this change is still in discussion. Would of course document it.

@marmarek
Copy link
Copy Markdown
Member

The build fails because of missing man page entry:

11:59:54 [qb.build_rpm.core-admin-client.host-fc41.build] DEBUG: Undocumented arguments for command 'qvm-device attach': '--force, -f'

@rustybird
Copy link
Copy Markdown
Contributor

When invoked without --force, IMO it should behave like qvm-remove or qvm-volume clear do when invoked without --force. They interactively prompt to confirm:

https://github.com/QubesOS/qubes-core-admin-client/blob/v4.3.30/qubesadmin/tools/qvm_volume.py#L175-L180

@ben-grande
Copy link
Copy Markdown
Contributor

About the use of --force or a flag with another name, wait for a response to Rusty's comment on QubesOS/qubes-issues#10825 (comment).

@ben-grande
Copy link
Copy Markdown
Contributor

Also, next time, don't use your main branch, create a separate branch for making commits. Now, if you force push to base on a different branch, I think Github will force close this PR, so keep as is.

@ben-grande
Copy link
Copy Markdown
Contributor

Please add "Fixes: QubesOS/qubes-issues#10825" to the commit message.

@nihalxkumar
Copy link
Copy Markdown
Author

@ben-grande commit message or PR description or both?

@ben-grande
Copy link
Copy Markdown
Contributor

Both is better.

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

Successfully merging this pull request may close these issues.

Make it more difficult to attach domU devices to dom0

5 participants