VDNS feature enhancements

Bug #1466683 reported by Nipa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Medium
Nipa
Trunk
Fix Committed
Medium
Nipa

Bug Description

Add queries from vrouter-agent to multiple named servers instead of just one server. DNS client that initiates DNS query will receive the first
valid response OR last bad response.

Part 2: Resolved via Bug #1477742, to check DNS-Server connection to named and end of config marker before advertising service is operationally UP to
discovery server.

Tags: quench vdns symc
Nipa (nipak)
tags: added: symc
Nipa (nipak)
no longer affects: opencontrail
tags: added: vdns
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

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

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

Revision history for this message
Nipa (nipak) wrote :

For part 2, use the new discovery infrastructure to reevaluate publish.

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

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

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

Review in progress for https://review.opencontrail.org/12653
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/12653
Committed: http://github.org/Juniper/contrail-controller/commit/e4656ea4ea7f477fbbd04826547fe7fa03f92b66
Submitter: Zuul
Branch: R2.20

commit e4656ea4ea7f477fbbd04826547fe7fa03f92b66
Author: Nipa Kumar <email address hidden>
Date: Mon Jul 13 17:37:33 2015 -0700

vDNS enhancement to send DNS queries to both known DNS named Servers.
a) Common timer is implemented for each of the DNS named Server DNS query request.
b) DNS handler is deleted on receiving query response from any of the named Server
OR when max retry is reached.
c) Introspect enhanced to show all DNS named server and traces show the query and
valid response along with the named server contacted

Change-Id: I5fae4b80ce745ab70b2c4937eab800b9d3927cc2
Partial-Bug: 1466683

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

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

commit c491f8e0d4e51f45b7951ca839ea0c78265e6dcd
Author: Nipa Kumar <email address hidden>
Date: Mon Jul 13 17:37:33 2015 -0700

vDNS enhancement to send DNS queries to both known DNS named Servers.
a) Common timer is implemented for each of the DNS named Server DNS query request.
b) DNS handler is deleter on receiving query response from any of the named Server
OR when max retry is reached.
c) Introspect enhanced to show all DNS named server and traces show the query and
valid response along with the named server contacted.

Change-Id: I5fae4b80ce745ab70b2c4937eab800b9d3927cc2
Partial-Bug: 1466683

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

Review in progress for https://review.opencontrail.org/13274
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/13274
Committed: http://github.org/Juniper/contrail-controller/commit/efb77a2199322df5f892f87d5e0de2dc89623c2f
Submitter: Zuul
Branch: R2.20

commit efb77a2199322df5f892f87d5e0de2dc89623c2f
Author: Nipa Kumar <email address hidden>
Date: Mon Aug 24 17:49:46 2015 -0700

vDNS changes to handle cases when named server responds with NULL data, in this case query to
the other named server should still be running independently. We cleanup the dns-handler only
if we receive a valid response OR responses from both named were received and invalid.
Also if both named servers are DOWN, there will be no response and the timer is triggered
to retry periodically for a few times.

o We send the first valid response from any of the named Server to the VM/client.
o We send the last invalid response if all responses from all named Servers are
invalid, so client/VM can avoid retries etc.,

Change-Id: I8b0ed327536ed337da131891c88fba5c557098dd
Partial-Bug: 1466683

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

Review in progress for https://review.opencontrail.org/13357
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/13408
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/13357
Committed: http://github.org/Juniper/contrail-controller/commit/d32b21b3cd608e10c74ea89f3f5a129a3884790f
Submitter: Zuul
Branch: R2.20

commit d32b21b3cd608e10c74ea89f3f5a129a3884790f
Author: Nipa Kumar <email address hidden>
Date: Wed Aug 26 15:22:51 2015 -0700

Add test case to cover various named response conditions.

Change-Id: Ibd75f9d714acf67a9d9e62b73b04db9f5d49b315
Partial-Bug:1466683

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

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

commit 964d51b7d42fc37215bb1afbb9146d53f2f81c45
Author: Nipa Kumar <email address hidden>
Date: Mon Aug 24 17:49:46 2015 -0700

vDNS changes to handle cases when named server responds with NULL data, in this case query to
the other named server should still be running independently. We cleanup the dns-handler only
if we receive a valid response OR responses from both named were received and invalid.
Also if both named servers are DOWN, there will be no response and the timer is triggered
to retry periodically for a few times.

o We send the first valid response from any of the named Server to the VM/client.
o We send the last invalid response if all responses from all named Servers are
invalid, so client/VM can avoid retries etc.,

Change-Id: I8b0ed327536ed337da131891c88fba5c557098dd
Partial-Bug: 1466683

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

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

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

Review in progress for https://review.opencontrail.org/13496
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/13497
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/13497
Committed: http://github.org/Juniper/contrail-controller/commit/0528ce1831838e5572ff00532e5fd6b1ce63afc7
Submitter: Zuul
Branch: R2.20

commit 0528ce1831838e5572ff00532e5fd6b1ce63afc7
Author: Nipa Kumar <email address hidden>
Date: Tue Sep 1 14:58:11 2015 -0700

Do not send DNS response to clients when parsing fails or response is not DNS_QUERY

Change-Id: I8b78b804f36aba369c146d963d8d91b4812647b7
Partial-Bug:1466683

Revision history for this message
Nipa (nipak) wrote :

Part 2 above will be solved by bug:Bug #1477742

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

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

commit 668db7180858fd00789a743387f1be5deeed1589
Author: Nipa Kumar <email address hidden>
Date: Wed Aug 26 15:22:51 2015 -0700

Add test case to cover various named response conditions.

Change-Id: Ibd75f9d714acf67a9d9e62b73b04db9f5d49b315
Partial-Bug:1466683

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

Review in progress for https://review.opencontrail.org/13540
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/13540
Committed: http://github.org/Juniper/contrail-controller/commit/b3be4523f26b484c0c98dee353d3027144213964
Submitter: Zuul
Branch: master

commit b3be4523f26b484c0c98dee353d3027144213964
Author: Nipa Kumar <email address hidden>
Date: Tue Sep 1 14:58:11 2015 -0700

Do not send DNS response to clients when parsing fails or response is not DNS_QUERY

Change-Id: I8b78b804f36aba369c146d963d8d91b4812647b7
Partial-Bug:1466683

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

Review in progress for https://review.opencontrail.org/13927
Submitter: Vinay Vithal Mahuli (<email address hidden>)

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.