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

minimega,qmp: add "force" option to vm cdrom eject #1272

Merged
merged 3 commits into from
Mar 28, 2019

Conversation

jcrussell
Copy link
Contributor

Allow override of guest's "locked" cdrom. Needs testing.

Fixes #1256.

Allow override of guest's "locked" cdrom. Needs testing.

Fixes sandia-minimega#1256.
@jcrussell jcrussell added this to the 2.6 milestone Mar 12, 2019
@jcrussell
Copy link
Contributor Author

@mkunz7: can you test that this works for you?

@mkunz7
Copy link
Contributor

mkunz7 commented Mar 23, 2019

change needs a force as well...

It didn't seem to help osx though, I still have to run those commands to eject the disk before ejecting in mini and and before changing iso

@jcrussell
Copy link
Contributor Author

Does the debug log still look like:

2019/02/20 11:44:37 DEBUG qmp.go:116: qmp write: map[string]interface {}{"execute":"eject", "arguments":map[string]interface {}{"device":"ide0-cd0"}}
2019/02/20 11:44:37 DEBUG qmp.go:111: qmp read: map[string]interface {}{"error":map[string]interface {}{"class":"GenericError", "desc":"Device 'ide0-cd0' is locked and force was not specified, wait for tray to open and try again"}}

Or can you see the "force" parameter in the arguments?

@mkunz7
Copy link
Contributor

mkunz7 commented Mar 28, 2019

Just tested the latest, the force function works. Thank you. You still need to run the os commands to release the drive though. This will have to work for now until qemu patches the function they call to tell the os the drive has been ejected.

@mkunz7
Copy link
Contributor

mkunz7 commented Mar 28, 2019

if http://philjordan.eu/osx-virt/ ever writes the virtio-scsi driver for osx I think you could hotplug scsi cdroms from hmp

device_add virtio-scsi-pci
drive_add 0 file=/home/ubuntu/a.iso,if=none,id=test,readonly=on,format=raw
device_add scsi-cd,drive=test

Copy link
Contributor

@djfritz djfritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@djfritz djfritz merged commit cf3ee5c into sandia-minimega:master Mar 28, 2019
@jcrussell jcrussell deleted the issue-1256 branch March 28, 2019 16:06
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.

None yet

3 participants