[Kimchi-devel] [PATCH] Issue#366: Guest Name Can't be Changed

Hongliang Wang hlwang at linux.vnet.ibm.com
Fri Apr 11 02:44:56 UTC 2014


OK. Thanks for your effort and the information.

On 04/11/2014 06:13 AM, Rodrigo Trujillo wrote:
> I tested and can confirm that Adams patch already fixed this problem.
> So lets use it, because it also implements the vm cpu/memory edit 
> functionality.
>
> Thanks,
>
> Rodrigo
>
> On 04/10/2014 06:06 PM, Rodrigo Trujillo wrote:
>> Hi,
>>
>> Adam sent a patch to change this window, to allow update memory and 
>> cpus.
>> I think this patch already fix this problem. I am going to test and 
>> let you guys know
>>
>>
>>
>> On 04/10/2014 02:09 PM, Christy Perez wrote:
>>> NACK.
>>>
>>> Tested-By: Christy Perez <christy at linux.vnet.ibm.com>
>>>
>>> This didn't change the name for my guest. There are no errors in the
>>> logs. :/
>>>
>>> I'm running Fedora 20 and using Firefox 28.
>>>
>>> Regards,
>>>
>>> - Christy
>>>
>>>
>>> On Thu, 2014-04-10 at 16:27 +0800, Hongliang Wang wrote:
>>>> When changing a guest name in Guest Tab, the change takes no effect 
>>>> because of
>>>> a UI bug. Fix it in this patch.
>>>>
>>>> Signed-off-by: Hongliang Wang <hlwang at linux.vnet.ibm.com>
>>>> ---
>>>>   ui/js/src/kimchi.guest_edit_main.js | 2 +-
>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/ui/js/src/kimchi.guest_edit_main.js 
>>>> b/ui/js/src/kimchi.guest_edit_main.js
>>>> index 9375c51..b771263 100644
>>>> --- a/ui/js/src/kimchi.guest_edit_main.js
>>>> +++ b/ui/js/src/kimchi.guest_edit_main.js
>>>> @@ -119,7 +119,7 @@ kimchi.guest_edit_main = function() {
>>>>           var editableFields = [ 'name' ];
>>>>           var data = {};
>>>>           $.each(editableFields, function(i, field) {
>>>> -            data[field] = $('#form-guest-edit [name="' + field + 
>>>> '"]').val();
>>>> +            data[field] = $('input[name="' + field + '"]', 
>>>> guestEditForm).val();
>>>>           });
>>>>           kimchi.updateVM(kimchi.selectedGuest, data, function() {
>>>>               kimchi.listVmsAuto();
>>>
>>> _______________________________________________
>>> Kimchi-devel mailing list
>>> Kimchi-devel at ovirt.org
>>> http://lists.ovirt.org/mailman/listinfo/kimchi-devel
>>>
>>
>> _______________________________________________
>> Kimchi-devel mailing list
>> Kimchi-devel at ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/kimchi-devel
>>
>




More information about the Kimchi-devel mailing list