DHCP in l2 mode

Bug #1415014 reported by Manish Singh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.0
Won't Fix
High
Manish Singh
R2.1
Fix Committed
High
Manish Singh
Trunk
Fix Committed
High
Manish Singh

Bug Description

In L2 mode(w.r.t. config) use dhcp flag from IPAM to identify if dhcp needs to be enabled/disabled instead of relying on forwarding mode.

Tags: vrouter
Manish Singh (manishs)
tags: added: vrouter
Changed in juniperopenstack:
milestone: none → r2.1-fcs
assignee: nobody → Manish Singh (manishs)
information type: Proprietary → Public
Revision history for this message
Manish Singh (manishs) wrote :

This bug will also cover knobs for dhcp-enable / disable and gateway as zero and non zero.

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/6795
Committed: http://github.org/Juniper/contrail-vrouter/commit/b51beacf90b05088424611f0df8abdc7952fad45
Submitter: Zuul
Branch: R2.1

commit b51beacf90b05088424611f0df8abdc7952fad45
Author: Divakar <email address hidden>
Date: Thu Jan 29 21:21:34 2015 -0800

Partial-Bug:#1415014. Allowing multiple Discard nexthop addition from aget

Change-Id: I3adb761af41863c39bd57fe532ea0658e992b716

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/6906
Committed: http://github.org/Juniper/contrail-controller/commit/78bae98b52173a754669168478cd2c4151955f8c
Submitter: Zuul
Branch: R2.1

commit 78bae98b52173a754669168478cd2c4151955f8c
Author: manishsingh <email address hidden>
Date: Mon Feb 2 17:00:01 2015 +0530

Remove forwarding mode setting using IPAM gateway attributes.

Though it allows 0.0.0.0 gateway but it is not used for
forwarding mode.
Partial-Bug: #1415014

Change-Id: I72c2a3faf3ededfdd05c677d06ce654b8798618f

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/6886
Committed: http://github.org/Juniper/contrail-controller/commit/92dc17a4ea40803e19869fb99a6d2f7e4108cda7
Submitter: Zuul
Branch: R2.1

commit 92dc17a4ea40803e19869fb99a6d2f7e4108cda7
Author: manishsingh <email address hidden>
Date: Mon Feb 2 11:14:25 2015 -0800

Handle enabling/disabling of DHCP flag and gateway.

If DHCP is enabled for IPAM then dont flood the request coming from
source. This is achieved by not setting DHCP Flood flag in source bridge route.
If DHCP is disabled then flood flag for DHCP is enabled on the bridge
entry of source.

Partial-Bug: #1415014

Change-Id: I86ad76462c212898543ed540f95fb84a010a81a3

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/7076
Committed: http://github.org/Juniper/contrail-controller/commit/f7cc78ba6d6ebe4c707beccc2762255c2bb4c7a0
Submitter: Zuul
Branch: master

commit f7cc78ba6d6ebe4c707beccc2762255c2bb4c7a0
Author: manishsingh <email address hidden>
Date: Mon Feb 2 11:14:25 2015 -0800

Handle enabling/disabling of DHCP flag and gateway.

If DHCP is enabled for IPAM then dont
flood the request coming from source.
This is achieved by not setting DHCP Flood flag in source bridge route.If
DHCP is disabled then flood flag for DHCP is enabled on the bridge entry of
source.
Partial - Bug: #1415014

Conflicts:
 src/vnsw/agent/oper/vn.cc

Change-Id: Ibd30c3758b1d1dfcdf8b86a886d89b8ad35615bc

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/7056
Committed: http://github.org/Juniper/contrail-vrouter/commit/fef814779581dd9279b0d6fcf2c3371cd457fd98
Submitter: Zuul
Branch: master

commit fef814779581dd9279b0d6fcf2c3371cd457fd98
Author: Divakar <email address hidden>
Date: Thu Jan 29 21:21:34 2015 -0800

Partial-Bug:#1415014. Allowing multiple Discard nexthop addition from aget

Change-Id: I3adb761af41863c39bd57fe532ea0658e992b716

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :
Download full text (7.1 KiB)

Reviewed: https://review.opencontrail.org/7463
Committed: http://github.org/Juniper/contrail-controller/commit/a9f90b4857b78c121ffdc2592efb23b4f78ef448
Submitter: Zuul
Branch: master

commit a9f90b4857b78c121ffdc2592efb23b4f78ef448
Author: Praveen K V <email address hidden>
Date: Thu Jan 29 00:43:29 2015 +0530

Merge following commits from R2.1 branch.

Dont allocate nh-index 0

nh-index 0 is reserved by vrouter. So, pre-allocate the first index so that nh
added by agent use index 1 and above

Change-Id: Ieed6d8666fc45399a51b280f1aae075425cee52c
(cherry picked from commit 3f2005b4c7be556cfdec9929b33b330ecc6ade3a)

Problem: TOR path not deleted on VRF delete.
When VRF is deleted in TSN mode, notification is received for physical_device_vn
entry.This entry contain vn_entry which can in turn can be used to extract VRF.
Multicast handler on adding the entry puts a state where it adds the VRF
entry name so that it can be used at the time of deletion because when VRF is
deleted vn_entry from physical_device_vn may not contain any VRF.
Issue was that search for multicast object was being done using vrf from VN
at time of delete. This should be done using the vrf_name stored in state.
Fixed the same here. Along with this fix also added a check to verify VRF is
in deleted state when IsTorDeleted being calculated.
Closes-bug: #1416808

Change-Id: Ica526b736a07723db40c6bb31a2f91f756cb1373
(cherry picked from commit 84347759dde36c758f37f4d12a7540de97786dbd)

Remove forwarding mode setting using IPAM gateway attributes.

Though it allows 0.0.0.0 gateway but it is not used for
forwarding mode.
Partial-Bug: #1415014

Change-Id: I72c2a3faf3ededfdd05c677d06ce654b8798618f
(cherry picked from commit 78bae98b52173a754669168478cd2c4151955f8c)

DNS disable knob : when DNS server is given as 0.0.0.0 in the dns_servers
field for the subnet or in the DHCP options list, vrouter doesnt provide
DNS proxy function any more. Do not send DNS server or DNS domain options
in DHCP response in such a case.

When there is any DNS server specified in the DHCP options list, do not
add the vrouter DNS server address any more.

If the DNS server in IPAM subnet config comes unspecified, set it to be
the same as the GW address.

Change-Id: If3e66bcfd7c137064de1471cc515e4742ea14b73
Fixes-Bug: 1416711
(cherry picked from commit 2a0d85dc4cc521e8c417182eedebe14dc9b4848b)

Conflicts:

 src/vnsw/agent/oper/vn.cc

Fix the VLAN-ID comparison in logical interface

Added checks for following,

1. VLAN-ID is not reused on multiple logical-ports on a single
physical-port
2. VLAN-ID must be set during creation of logical-interface
3. Do not allow change of vlan-id after logical-interface is
created
4. Rename interface_object to li_object to avoid confusion in
_check_interface_name

Closes-Bug: #1416323
Change-Id: I444f83d8cf8d34a6848026fe3fd0b8e23a7798f4
(cherry picked from commit 0a778eb1dab3e94c94cf0a37f36ad30f593c4e3b)

Adding 10Sec delay for ovsdb session to start txn

Issue:
------
On reconnect TOR Agent use to start the audit process immediately
causing all the previouslt installed entries to get delete as
stale, then re-added on config availabilty

Fix:
----
For R2.1, on connection ...

Read more...

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.