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

4.0 - Cannot change netvm where existing netvm is not running #6493

Closed
unman opened this issue Mar 29, 2021 · 2 comments · Fixed by QubesOS/qubes-core-admin#537
Closed

4.0 - Cannot change netvm where existing netvm is not running #6493

unman opened this issue Mar 29, 2021 · 2 comments · Fixed by QubesOS/qubes-core-admin#537
Labels
C: networking 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. r4.1-dom0-stable r4.2-host-cur-test

Comments

@unman
Copy link
Member

unman commented Mar 29, 2021

Qubes OS version

4.0

Affected component(s) or functionality

qube networking

Brief summary

As title

How Reproducible

100%

To Reproduce

Steps to reproduce the behavior:

  1. Kill netvm of running qube
  2. Attermpt to change netvm - qvm-prefs <qube> netvm <new netvm>

Expected behavior

netvm will be changed, or informative error given.

Actual behavior

Error -
qvm-prefs: error: no such property: 'netvm'

Screenshots

Additional context

When attempting to change netvm to a non-running qube, the error message is informative, if wordy -
qvm-prefs: error: Cannot dynamically attach to stopped Net VM....

Solutions you've tried

Restarting the old netvm allows the netvm to be changed.

Relevant documentation you've consulted

Related, non-duplicate issues

@unman unman added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug labels Mar 29, 2021
@andrewdavidwong andrewdavidwong added C: networking needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. labels Mar 29, 2021
@andrewdavidwong andrewdavidwong added this to the Release 4.0 updates milestone Mar 29, 2021
marmarta added a commit to marmarta/qubes-manager that referenced this issue May 12, 2023
If user tries to change netvm of a running vm to
a halted netvm, asks if they want to start
that halted netvm.

related to QubesOS/qubes-issues#8165
related to QubesOS/qubes-issues#6493
@andrewdavidwong andrewdavidwong added diagnosed Technical diagnosis has been performed (see issue comments). pr submitted A pull request has been submitted for this issue. and removed needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. labels May 26, 2023
@marmarek marmarek added the backport pending On closed issues, indicates fix released for newer version will be backported to older version. label Jul 4, 2023
marmarek pushed a commit to QubesOS/qubes-core-admin that referenced this issue Jul 4, 2023
Do not try to detach device from a halted netvm.

fixes QubesOS/qubes-issues#6493

(cherry picked from commit 1ffb526)
@qubesos-bot
Copy link

Automated announcement from builder-github

The component core-admin (including package qubes-core-dom0-4.1.32-1.fc32) has been pushed to the r4.1 testing repository for dom0.
To test this update, please install it with the following command:

sudo qubes-dom0-update --enablerepo=qubes-dom0-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component core-admin (including package qubes-core-dom0-4.1.32-1.fc32) has been pushed to the r4.1 stable repository for dom0.
To install this update, please use the standard update command:

sudo qubes-dom0-update

Or update dom0 via Qubes Manager.

Changes included in this update

@marmarek marmarek removed the backport pending On closed issues, indicates fix released for newer version will be backported to older version. label Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: networking 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. r4.1-dom0-stable r4.2-host-cur-test
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants