On 8/22/2014 10:00 PM, Aline Manera wrote:

On 08/22/2014 05:02 AM, Wen Wang wrote:
Hi Aline,

"Manage Media" under "Edit" -> "Storage" does not work well. There are two issues that are related to the back-end:

1) Edit cdrom media fail when VM is running, error are presented below:
{
  "reason":"KCHVMSTOR0009E: Error while updating storage device: internal error unable to execute QEMU command 'change': 
  Could not open '/home/wangwen/Downloads/CentOS-7.0-1406-x86_64-DVD.iso': Permission denied",
  
  "code":"500 Internal Server Error"
}           

2) Cannot change media to location except for /var/lib/libvirt/, which means the iso file only work under that directory, otherwise kimchi will popup error as below:
{
  "reason":"KCHVM0019E: Unable to start virtual machine 2c057a8e-f521-4076-9a49-5e8bca8ca113-vm-1. Details: internal error process exited while 
  connecting to monitor: qemu-kvm: -drive file=/home/wangwen/Downloads/CentOS-7.0-1406-x86_64-DVD.iso,if=none,media=cdrom,id=drive-ide0-1-0,readonly=on,
  format=raw: could not open disk image /home/wangwen/Downloads/CentOS-7.0-1406-x86_64-DVD.iso: Permission denied\n",
  
  "code":"500 Internal Server Error"
}

Could you please take a look at this?

Thanks for reporting it, Wen Wang!

Those are known-issues and related to the backend code.
I will check it and provide a patch soon.
Thanks Aline, really appreciated.



Best Regards
Wang Wen

On 8/22/2014 2:11 PM, Wen Wang wrote:
Changes:
V1 -> V2:
1) Have the "ui/css/theme-default/guest-media.css" deleted
2) Have the "edit" button in interface tab deleted when VM is running

1) Have the "Manage Media" function deleted from "Action" list
2) Enable "Edit" whether VM is running or off
3) Disable edit function of "General", "Storage", "Interface" except
"replace" of the cdrom under "Storage".

Wen Wang (2):
  UI: Delete Manage Media function from action list
  Allow admin user change permission settings when VM is running

 ui/css/theme-default/guest-media.css |   42 -------------------------
 ui/js/src/kimchi.guest_edit_main.js  |   56 ++++++++++++++++++++++-----------
 ui/js/src/kimchi.guest_main.js       |   14 --------
 ui/pages/guest-edit.html.tmpl        |    2 +-
 ui/pages/guest-media.html.tmpl       |   57 ----------------------------------
 ui/pages/guest.html.tmpl             |    1 -
 6 files changed, 39 insertions(+), 133 deletions(-)
 delete mode 100644 ui/css/theme-default/guest-media.css
 delete mode 100644 ui/pages/guest-media.html.tmpl