Agent should not connect to contrail-dns when it doesnt have end of RIB

Bug #1605341 reported by Hari Prasad Killi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.21.x
Fix Committed
Medium
Nipa
R3.0
Fix Committed
Medium
Nipa
R3.1
Fix Committed
Medium
Nipa
R3.2
Fix Committed
Medium
Nipa
R4.0
Fix Committed
Medium
Nipa
Trunk
Fix Committed
Medium
Nipa

Bug Description

When contrail-dns doesnt have end of RIB, connections from agent should not be successful. If the connection is closed and agent sends a re-subscribe to discovery after a few retries, agent may move to a contrail-dns which can provide service.

Tags: vdns
Revision history for this message
Nipa (nipak) wrote :

On master image, Agent has a list of all DNS publishers in the system as agent subscribes with instances=0. When Agent detects a TCP connection DOWN (as control-node was being upgraded), agent after a few retries will try other DNS Servers in the list. Note at this point the upgraded DNS Servers which may be in the publisher list( as ttl did not expire), may be able to accept TCP connections. These new TCP connections need to be closed if end-of-rib is not computed.

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/22515
Submitter: Nipa Kumar (<email address hidden>)

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

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

commit ba406943f224a192575d08ccc7d8231dab72e042
Author: Nipa Kumar <email address hidden>
Date: Wed Jul 27 16:59:23 2016 -0700

Close new Agent DNS XMPP connections until end-of-config is detected.

Closes-Bug:1605341

Provide path to store the session key generated for dynamic updates
Closes-Bug:1400879

Change-Id: I2965b6039a20715024b2fb69ff9554ca9df14078

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/23336
Submitter: Nipa Kumar (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/23873
Submitter: Nipa Kumar (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/23336
Submitter: Nipa Kumar (<email address hidden>)

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

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

commit 0ceda0a025d694f69c95f02363311a30dc10c1da
Author: Nipa Kumar <email address hidden>
Date: Tue Aug 16 17:16:38 2016 -0700

Support dynamic configuration update of named

An additional file contrail-named-base.conf will contain the base named configurtion
such as options stanza, key stanza, controls stanza and logging stanza.

All configs generated due to vdns configuration resulting in view stanza will be written
to contrail-named.conf

Before sending "reconfig" command to named, both the files will be merged
into contrail-named.conf

The above gives flexibility to user to update and apply changes without restarting
the dns daemon which generated the contrail-named.conf

An additional helper script(applynamedconfig.py) will merge the files and apply the
config(contrail-named-base.conf) to named daemon.

Change-Id: If6089189a1104f648d8d7d3f638c58f9883deb50
Closes-Bug:1605341

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/24107
Submitter: Nipa Kumar (<email address hidden>)

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

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

commit 2c3817baab8e957bb186b9388f0e304bcb40df6a
Author: Nipa Kumar <email address hidden>
Date: Tue Sep 13 13:15:05 2016 -0700

Typo in applynamedconfig.py script

Change-Id: I0103c7cf2e42bb617acb4f0d229c0994f46d205b
Closes-Bug:1605341

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

Reviewed: https://review.opencontrail.org/23873
Committed: http://github.org/Juniper/contrail-packages/commit/5dd2461d49530aba85be516bd56a86ae139c5449
Submitter: Zuul
Branch: master

commit 5dd2461d49530aba85be516bd56a86ae139c5449
Author: Nipa Kumar <email address hidden>
Date: Fri Sep 2 11:23:02 2016 -0700

Add applynamedconfig script to install scripts

Change-Id: I5e16f72d8d953af5559ea5e8512b5d405d341075
Partial-Bug:1605341

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

Review in progress for https://review.opencontrail.org/32928
Submitter: Nipa Kumar (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/32939
Submitter: Nipa Kumar (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/32928
Committed: http://github.com/Juniper/contrail-controller/commit/f7099f0977494621002d05c7a44edda359b9d6ec
Submitter: Zuul (<email address hidden>)
Branch: R2.21.x

commit f7099f0977494621002d05c7a44edda359b9d6ec
Author: Nipa Kumar <email address hidden>
Date: Fri Jun 16 18:23:38 2017 -0700

Porting to R2.21 as work-day wants to use DNS forward mode.

To persist the changes in contrail-named.conf while vdns are added/deleted or
subnets added/deleted we separate building of contrail-named into static portion
and dynamic portion.

Partial-Bug:1694927
Partial-Bug:1605341

Change-Id: I94bf61fe3941b552e4a5946a75c60a5792cc6a51

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

Reviewed: https://review.opencontrail.org/32939
Committed: http://github.com/Juniper/contrail-packages/commit/24718b620575f682a03c929a2d983b43e677531e
Submitter: Zuul (<email address hidden>)
Branch: R2.21.x

commit 24718b620575f682a03c929a2d983b43e677531e
Author: Nipa Kumar <email address hidden>
Date: Sat Jun 17 09:41:01 2017 -0700

Add applynamedconfig.py to both debian & centos package for R2.21.x for work-day

Change-Id: Ic07ab8a33994ce3a976f89b475900aec71a19239
Partial-Bug:1605341

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/32971
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/32972
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/32974
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/32975
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.2

Review in progress for https://review.opencontrail.org/32976
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.0

Review in progress for https://review.opencontrail.org/32977
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/32978
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/32983
Submitter: Nipa Kumar (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/32984
Submitter: Nipa Kumar (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/32977
Committed: http://github.com/Juniper/contrail-packages/commit/5b9967033517a06395e8526e3d7a412b2d5ea5f5
Submitter: Zuul (<email address hidden>)
Branch: R4.0

commit 5b9967033517a06395e8526e3d7a412b2d5ea5f5
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 11:41:39 2017 -0700

Add applynamedconfig.py to centos package

Change-Id: Ie01152b88d4ae2ffa80e597e7e48f3177e66e8d6
Partial-Bug:1605341

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

Reviewed: https://review.opencontrail.org/32978
Committed: http://github.com/Juniper/contrail-packages/commit/b86a49da6f8927b8b921d3f0b50a18faaa401f0d
Submitter: Zuul (<email address hidden>)
Branch: master

commit b86a49da6f8927b8b921d3f0b50a18faaa401f0d
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 11:41:39 2017 -0700

Add applynamedconfig.py to centos package

Change-Id: Ie01152b88d4ae2ffa80e597e7e48f3177e66e8d6
Partial-Bug:1605341

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

Reviewed: https://review.opencontrail.org/32976
Committed: http://github.com/Juniper/contrail-packages/commit/16cfb8da2ff823f742f845aaf62ff44951017caa
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit 16cfb8da2ff823f742f845aaf62ff44951017caa
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 11:41:39 2017 -0700

Add applynamedconfig.py to centos package

Change-Id: Ie01152b88d4ae2ffa80e597e7e48f3177e66e8d6
Partial-Bug:1605341

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

Reviewed: https://review.opencontrail.org/32984
Committed: http://github.com/Juniper/contrail-controller/commit/4c91f4a471c0d045186a6eb5c825dcdeb3546ae8
Submitter: Zuul (<email address hidden>)
Branch: R3.1

commit 4c91f4a471c0d045186a6eb5c825dcdeb3546ae8
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 15:15:08 2017 -0700

Porting provisioning to persist global named configuration

To persist the changes in contrail-named.conf while vdns are added/deleted or
subnets added/deleted, separate building of contrail-named into static portion
and dynamic portion.

Change-Id: I7e9d8fedfd525b5431fa03d31cc952d1d98caa37
Partial-Bug:1694927
Partial-Bug:1605341

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

Reviewed: https://review.opencontrail.org/32975
Committed: http://github.com/Juniper/contrail-packages/commit/94c625457cef070b0cc1b5f6b929836a1907e892
Submitter: Zuul (<email address hidden>)
Branch: R3.1

commit 94c625457cef070b0cc1b5f6b929836a1907e892
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 11:37:47 2017 -0700

Add applynamedconfig.py to both debian & centos package

Change-Id: Ib82266055a5e748bd70201cc5eed76d8ef711088
Partial-Bug:1605341

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

Reviewed: https://review.opencontrail.org/32983
Committed: http://github.com/Juniper/contrail-controller/commit/e61105239b8b3363af37b37cfd6766e738fb8b6d
Submitter: Zuul (<email address hidden>)
Branch: R3.0

commit e61105239b8b3363af37b37cfd6766e738fb8b6d
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 15:15:08 2017 -0700

Porting provisioning to persist global named configuration

To persist the changes in contrail-named.conf while vdns are added/deleted or
subnets added/deleted, separate building of contrail-named into static portion
and dynamic portion.

Change-Id: I7e9d8fedfd525b5431fa03d31cc952d1d98caa37
Partial-Bug:1694927
Partial-Bug:1605341

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

Reviewed: https://review.opencontrail.org/32974
Committed: http://github.com/Juniper/contrail-packages/commit/25da6c641622437e4fc7b7f167585c3281d5c4a8
Submitter: Zuul (<email address hidden>)
Branch: R3.0

commit 25da6c641622437e4fc7b7f167585c3281d5c4a8
Author: Nipa Kumar <email address hidden>
Date: Mon Jun 19 11:34:12 2017 -0700

Add applynamedconfig.py to both debian & centos package

Change-Id: If6088d52e979002ecae94229833847b195714417
Partial-Bug:1605341

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.