On 07/17/2014 12:00 PM, Sheldon wrote:
On 07/17/2014 09:25 PM, Aline Manera wrote:

On 07/17/2014 01:24 AM, Sheldon wrote:
On 07/17/2014 08:54 AM, Aline Manera wrote:

On 07/16/2014 09:35 PM, Aline Manera wrote:

On 07/15/2014 12:45 PM, shaohef@linux.vnet.ibm.com wrote:
From: ShaoHe Feng <shaohef@linux.vnet.ibm.com>

V1 -> V2:
make ticket as sub-resource of a VM

Do we need a new sub-resource?
Why not use PUT /vms/<my-vm> {passwd:...} ?
How do we get the passwd?
GET /vms/<my-vm>
{
"name": my-vm,
"cpu": 1,
"memory": 512,
"passwd": "abcd"
}

data = GET /vms/<my-vm>
data.passwd


Using the PUT /vms/<my-vm> also can make the UI logic simpler as the console password will be changed on VM Edit dialog
at VM Edit dialog, the disks and ifaces are all sub-collection.
PUT /vms/<my-vm>/ifaces/iface1

You are correct, but disks and ifaces have their own tab on VM Edit
I was thinking in add the password in the VM configuration tab, which means use the PUT method
I have no preference.
I can change it in the next version.


But we had better to get agreement before our patches. Though it is difficult sometimes.

Agree

In my V1 version I did take passwd as an attribute of VM instead of sub-resource.


Sorry I haven't had chance to review the V1


Anyway  we have get agreement that  "have their own tab" is the criterion of sub-
collection  and  sub-resource.



(which already uses this PUT method)



A ticket is the credential to access VM.
Only who get the ticket can access a VM.

test this patch set:
set the ticket
$sudo curl -k -u <user>:<password> -H "Content-Type: application/json" -H  \
"Accept: application/json" https://localhost:8001/vms/test-vm-8/ticket/ \
-X PUT -d '{"passwd": "abcd"}'

get the ticket
$sudo curl -k -u <user>:<password> -H "Content-Type: application/json" -H \
"Accept: application/json" https://localhost:8001/vms/test-vm-8/ticket/


ShaoHe Feng (5):
   vm ticket in backend: update API.md
   vm ticket in backend: update controller and API.json
   vm ticket in backend: update model
   vm ticket in backend: update mockmodel
   vm ticket in backend: update test case

  docs/API.md               | 20 +++++++++++++++
  src/kimchi/API.json       | 14 ++++++++++
  src/kimchi/control/vms.py | 13 ++++++++++
  src/kimchi/i18n.py        |  1 +
  src/kimchi/mockmodel.py   | 18 +++++++++++++
  src/kimchi/model/vms.py   | 65 +++++++++++++++++++++++++++++++++++++++++++++++
  tests/test_model.py       | 31 ++++++++++++++++++++++
  tests/test_rest.py        | 36 ++++++++++++++++++++++++++
  8 files changed, 198 insertions(+)


_______________________________________________
Kimchi-devel mailing list
Kimchi-devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/kimchi-devel






-- 
Thanks and best regards!

Sheldon Feng(冯少合)<shaohef@linux.vnet.ibm.com>
IBM Linux Technology Center



-- 
Thanks and best regards!

Sheldon Feng(冯少合)<shaohef@linux.vnet.ibm.com>
IBM Linux Technology Center