On 12/30/2013 01:13 AM, Sheldon wrote:

now we have support network  get/create/delete/activate/deactivate

Now we need to support the attach/detach/ a Network to a guest.
And get the info of Network attached to a guest.


libvirt supports several types of network interfaces:
'bridge','network','user','ethernet','direct','hostdev','mcast', 'server', 'client'

but there is not a unique "name" attribute for "interface" element.(more libvirt network interfaces info)

That is not good to define our REST API.

But for "network" type resource, there is a "network" attribute of "source" resource.
And the value of "network" attribute  is the name of network that we defined by /networks POST method.

the "network" type + "network" attribute is unique.

We can define the Network Interfaces like this to support "network" type Interfaces:
GET /vms/*name*/Interfaces?type=network&network="default"
But this is not RESTFUL.


Maybe We can also name a Network Interface by the Interface mac.
The mac is unique.
GET /vms/*name*/Interfaces/mac

I think using the mac address is enough for us.

I'd also suggest to use 'ifaces' instead of 'Interfaces'

# list the iface info
GET /vms/<name>/ifaces/<mac>

# attach new iface
POST /vms/<name>/ifaces/<mac> {network: <network-id>}

# dettach a iface
DELETE /vms/<name>/ifaces/<mac>

The best way for user is get the NICS name in guest. such as eth0, eth1....
GET /vms/*name*/Interfaces/eth0                                 
GET /vms/*name*/Interfaces/eth1          

But there is a problem, we need get the nic name and mac from guest.
and Make a map in kimchi DB.
also the nic name may change, we need  make the map consistence with guest nic name.

I think it is not needed.
Only the mac address should be enough.
Just for reference: I also looked at virt-manager and it doesn't display information
about nic name, only mac and network name and type.

-- 
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