-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
device identity #304
base: main
Are you sure you want to change the base?
device identity #304
Conversation
9a7ff4b
to
0168517
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #304 +/- ##
==========================================
+ Coverage 75.09% 75.23% +0.13%
==========================================
Files 52 52
Lines 8541 8799 +258
==========================================
+ Hits 6414 6620 +206
- Misses 2127 2179 +52 ☔ View full report in Codecov by Sentry. |
2c86f03
to
2db1c50
Compare
1. `ask-to-attach` is not even implemented for PCI (assigned devices will be ignored) 2. manual attaching could lead to unusable devices 3. `auto-attach` there is no good reason for ignoring absent of PCI device
If front-vm is down, we still want assignment
Listing all assignments by default could be overwhelming. Now to see all assignment a flag `-s` must be used. Assignments are indicated by '*' before qube name.
Even if device is `auto-attached` it could be manually reattached, so to avoid confusion just show as `attached`
22aac29
to
05115ae
Compare
Changes here looks okay, but I haven't tried running this version yet. |
Important note on breaking backward compatibility: there may exists scripts using |
implements QubesOS/qubes-issues/issues/9325