[Engine-devel] Extend host_nics action with UpdateNetworks
This is a multi-part message in MIME format. --------------040202050707070305020800 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Hi, I am checking the SetupNetwork feature in 3.1 (si01) and I think we are missing a very important implementation. SetupNetworks in vdsm layer is able to receive a nics collection which describe which nics will be updated and handle only those nics. This implementation is missing from engine-backend because backend automatically treat missing nics as nics we would like to remove. A very buggy scenario would be, missing mgmt network. The idea is to send (via REST API) a collection which contain only the interfaces (nics) we would like to update. This is actually an UPDATE collection procedure. it could be added as a different action: http://engine ip:port/hosts/id/nics/*updatenetworks* I believe it is a part of the SetupNetwork feature that needs to be in 3.1. Thanks --------------040202050707070305020800 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit <html> <head> <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1"> </head> <body bgcolor="#FFFFFF" text="#000000"> Hi,<br> I am checking the SetupNetwork feature in 3.1 (si01) and I think we are missing a very important implementation.<br> SetupNetworks in vdsm layer is able to receive a nics collection which describe which nics will be updated and handle only those nics.<br> This implementation is missing from engine-backend because backend automatically treat missing nics as nics we would like to remove. <br> A very buggy scenario would be, missing mgmt network.<br> <br> The idea is to send (via REST API) a collection which contain only the interfaces (nics) we would like to update. <br> <br> This is actually an UPDATE collection procedure. it could be added as a different action:<br> <a class="moz-txt-link-freetext" href="http://engine">http://engine</a> ip:port/hosts/id/nics/<b>updatenetworks</b> <br> <br> I believe it is a part of the SetupNetwork feature that needs to be in 3.1.<br> <br> Thanks<br> <br> </body> </html> --------------040202050707070305020800--
----- Original Message -----
From: "avi tal" <atal@redhat.com> To: engine-devel@ovirt.org Cc: "Oded Ramraz" <oramraz@redhat.com>, "Roy Golan" <rgolan@redhat.com>, "michael pasternak" <mpastern@redhat.com>, "Simon Grinberg" <simon@redhat.com> Sent: Tuesday, March 27, 2012 2:11:27 PM Subject: Extend host_nics action with UpdateNetworks
Hi, I am checking the SetupNetwork feature in 3.1 (si01) and I think we are missing a very important implementation. SetupNetworks in vdsm layer is able to receive a nics collection which describe which nics will be updated and handle only those nics. This implementation is missing from engine-backend because backend automatically treat missing nics as nics we would like to remove. A very buggy scenario would be, missing mgmt network.
The idea is to send (via REST API) a collection which contain only the interfaces (nics) we would like to update.
This is actually an UPDATE collection procedure. it could be added as a different action: http://engine ip:port/hosts/id/nics/ updatenetworks
I believe it is a part of the SetupNetwork feature that needs to be in 3.1.
+1 While for web UI it makes sense to do it in Read/Modify/Write fashion, meaning that the setup network dialogue shows the complete picture, allows the user to modify and then write it back. It does not make sense to force the same on an API user. Consider a user that just wants to remove/add two logical networks on bond X. Does he really have to read it all, then add the networks, then send back? It makes sense to explicitly say update: add only those two on bond X - do not touch the rest. This means though that the updateNetworks command has to explicitly say remove/add/update per network.
Thanks
participants (2)
-
avi tal -
Simon Grinberg