On 04/13/2012 01:54 PM, dennis jenkins wrote:
Patch 7 is going to add this with DETECTED_IP in the parameter list that then looks something like this:
<parameter name='ip_learning' value='dhcp'/>
<parameter name='DETECTED_IP' value='192.168.122.210'/>
<address type='pci' domain='0x0000' bus='0x00' slot='0x04' function='0x0'/>
Some other considerations:
We should probably reserve some variables. This DETECTED_IP will be read-only meaning no-one can set it by writing the XML. Then we have
- IP: list of IP addresses use by an interface
- MAC: MAC address of the interface
- ip_learning: for choosing the IP detection mechanism (capital letters??)
- DHCPSERVER: for list of trusted DHCP servers
Maybe the following should be also reserved
independent of when they may be supported.
Another idea may be whether this special variable DETECTED_IP could be extended to include the lease and use it for migration, i.e. DETECTED_IP=<ip addr>,<lease timeout> and then when migrating we need to know that the XML is being parse due to migration (rather than the XML coming from user input directly) and that we can pick up its contents and feed it into the IP lease mechanism. Just a thought for now. Maybe someone has a comment on whether this is a good idea to begin with...