[Kimchi-devel] [PATCH v2 0/4] [UI] CDROM Management in a VM

[Hongliang Wang]

On 03/05/2014 02:25 AM, Aline Manera wrote:
>
> The UI looks good!
> I'd just suggest to keep the edit window silver instead of white
Thanks! It's changed by jQuery UI and I was unaware of it. Will include 
it in next patch.
>
> And I am not able to remove a cdrom
Any details? It works in my machine. Attaching one, replacing one, or 
detaching one works well.
> From the log I got:
>
> 127.0.0.1 - - [04/Mar/2014:15:22:19] "GET 
> /?cdrom=%2Fhome%2Falinefm%2Fisos%2FFedora-Live-Desktop-x86_64-19-1.iso&cdrom=%2Fhome%2Falinefm%2Fisos%2FopenSUSE-13.1-GNOME-Live-x86_64.iso&cdrom=%2Fhome%2Falinefm%2Fisos%2FopenSUSE-13.1-GNOME-Live-x86_64.iso 
> HTTP/1.1" 404 1606 "http://localhost:8000/" "Mozilla/5.0 (X11; Ubuntu; 
> Linux x86_64; rv:27.0) Gecko/20100101 Firefox/27.0"
Detaching a CDROM will send a request in format:

DELETE /vms/vm-name/storages/hdx

Not saw similar requests like this.
>
> And on UI
>
>
>         ERROR CODE
>
> 404 Not Found
>
>
>         REASON
>
> Unexpected query string parameters: cdrom
>
>
>         STACK
>
> Traceback (most recent call last):
>    File "/usr/lib/python2.7/dist-packages/cherrypy/_cprequest.py", line 656, in respond
>      response.body = self.handler()
>    File "/usr/lib/python2.7/dist-packages/cherrypy/lib/encoding.py", line 188, in __call__
>      self.body = self.oldhandler(*args, **kwargs)
>    File "/usr/lib/python2.7/dist-packages/cherrypy/_cpdispatch.py", line 40, in __call__
>      raise sys.exc_info()[1]
> HTTPError: (404, 'Unexpected query string parameters: cdrom')
>
> On 03/04/2014 06:01 AM, Hongliang Wang wrote:
>> Allow user to manipulate CDROM(s) in a VM:
>> 1) Attach one or more CDROM(s);
>> 2) Replace a CDROM with another ISO file path;
>> 3) Detach a CDROM from a VM.
>>
>>    Tested on Chrome.
>>
>>    Please apply the following patch first:
>>    * [UI] Window - Correct Footer Height
>>
>> v1 -> v2:
>>    2a) Separate one patch into 4 small ones to make it easier to review
>>        (Thanks to Ramon Medeiros's comment)
>>    2b) Updated API calling to list CDROM
>>        (Thanks to Aline's comment)
>>    2c) Separated VM updating window into 2 parts: general and storage
>>        (Thanks to Aline's comment)
>>    2d) Enhanced UI to allow user trigger replace CDROM window by clicking
>>        ISO file path text box
>>    2e) Changed uneditable fields from readonly to disabled
>>    2f) Removed authors infomation
>>        (Thanks to Aline's comment)
>>
>> Hongliang Wang (4):
>>    [UI] Add CDROM-related APIs
>>    [UI] Add i18n Variables for CDROM-related Function
>>    [UI] Add/Edit CDROM Support
>>    [UI] Attach/Replace/Detach a CDROM to/in/from a VM
>>
>>   ui/css/theme-default/guest-cdrom-add.css      |  62 +++++++++++
>>   ui/css/theme-default/guest-cdrom-edit.css     |  62 +++++++++++
>>   ui/css/theme-default/guest-edit.css           | 108 +++++++++++++++---
>>   ui/images/theme-default/guest-icon-sprite.png | Bin 0 -> 6748 bytes
>>   ui/js/src/kimchi.api.js                       |  74 +++++++++++++
>>   ui/js/src/kimchi.guest_cdrom_add_main.js      |  79 +++++++++++++
>>   ui/js/src/kimchi.guest_cdrom_edit_main.js     |  87 +++++++++++++++
>>   ui/js/src/kimchi.guest_edit_main.js           |  75 ++++++++++++-
>>   ui/pages/guest-cdrom-add.html.tmpl            |  73 ++++++++++++
>>   ui/pages/guest-cdrom-edit.html.tmpl           |  73 ++++++++++++
>>   ui/pages/guest-edit.html.tmpl                 | 153 +++++++++++++++++---------
>>   ui/pages/i18n.html.tmpl                       |   5 +
>>   12 files changed, 778 insertions(+), 73 deletions(-)
>>   create mode 100644 ui/css/theme-default/guest-cdrom-add.css
>>   create mode 100644 ui/css/theme-default/guest-cdrom-edit.css
>>   create mode 100644 ui/images/theme-default/guest-icon-sprite.png
>>   create mode 100644 ui/js/src/kimchi.guest_cdrom_add_main.js
>>   create mode 100644 ui/js/src/kimchi.guest_cdrom_edit_main.js
>>   create mode 100644 ui/pages/guest-cdrom-add.html.tmpl
>>   create mode 100644 ui/pages/guest-cdrom-edit.html.tmpl
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/kimchi-devel/attachments/20140305/1c7517a6/attachment.html>