On Mon, Mar 28, 2016 at 10:16 AM, Numan Siddique <nusid...@redhat.com>
wrote:

>
>
> On Mon, Mar 28, 2016 at 6:58 PM, Russell Bryant <russ...@ovn.org> wrote:
>
>>
>>
>> On Thu, Mar 17, 2016 at 2:34 AM, Numan Siddique <nusid...@redhat.com>
>> wrote:
>>
>>> On 03/16/2016 10:44 PM, Russell Bryant wrote:
>>> > It could also be this:
>>> >
>>> > dhcp_offer(offer_ip=10.0.0.3, netmask=255.255.255.0, mtu=1300,
>>> > dns={8.8.8.8, 7.7.7.7});
>>>
>>> Thanks. This seems better to me.
>>>
>>
>> Where do you get the source MAC and IP for the DHCP responses?
>>
>>
> ​For the initial testing I had harcoded source MAC to some value and left
> the source ip unchanged i.e it 0.0.0.0
>
>
> https://github.com/numansiddique/ovs/blob/dhcp_pause_rfc_working/ovn/northd/ovn-northd.c#L1132
>
> I think it should be ok if the ovn-controller sets the source ip to
> 0.0.0.0, but not sure what should be set for source mac.
>

I'm not sure if a src of 0.0.0.0 is a good idea.  Wouldn't that typically
be the IP address of the DHCP server?  I'd be worried about some clients
treating it as invalid.

I think it should be specified by logical flows.  Something like:

    dhcp_offer(offer_ip=10.0.0.3, netmask=255.255.255.0,
mtu=1300, dns={8.8.8.8, 7.7.7.7}); eth.dst = eth.src; eth.src =
<SOMETHING>; ip.src = <SOMETHING>; outport = inport; inport = ""; output;

As for what addresses to use ... I don't know.  I think it should be
specified via OVN_Northbound, somehow.  It's easy enough for Neutron to
generate an IP and MAC to use.  It already creates DHCP ports for this
purpose.  Maybe it could just be additional logical port options, like
you're doing for the rest of the DHCP parameters?

-- 
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to