Ubuntu Cloud Archive

OVN: Missing reverse DNS for instances

Bug #1951872 reported by Dr. Jens Harbott
30
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Ubuntu Cloud Archive
New
Undecided
Unassigned
neutron
Fix Released
Medium
yatin

Bug Description

When using OVN, reverse DNS for instances is not working. With dhcp-agent:

ubuntu@vm1:~$ host 10.0.0.11 10.0.0.3
Using domain server:
Name: 10.0.0.3
Address: 10.0.0.3#53
Aliases:

11.0.0.10.in-addr.arpa domain name pointer vm3.openstackgate.local.

With OVN:

ubuntu@vm1:~$ host 10.0.0.11 8.8.8.8
Using domain server:
Name: 8.8.8.8
Address: 8.8.8.8#53
Aliases:

Host 11.0.0.10.in-addr.arpa. not found: 3(NXDOMAIN)

Expected result: Get the same answer as with ML2/OVS.

Dr. Jens Harbott (j-harbott)
tags: added: dns ovn
Rodolfo Alonso (rodolfo-alonso-hernandez)
Changed in neutron:
importance: Undecided → Medium
yatin (yatinkarel)
Changed in neutron:
assignee: nobody → yatin (yatinkarel)
Revision history for this message
yatin (yatinkarel) wrote :

So i was able to reproduce it locally, and found that reverse DNS is supported since OVN-21.06.0 [1]
.

Need to adapt neutron to support this(Atleast required to add enteries in "DNS" NB Table). Will check and work over it.

[1] https://github.com/ovn-org/ovn/commit/82a4e44e308171cb545211eb2534475ef16a4c0e

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/822294

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/822294
Committed: https://opendev.org/openstack/neutron/commit/5b597d6e4ee647502d3f2aecd46de63f409317b4
Submitter: "Zuul (22348)"
Branch: master

commit 5b597d6e4ee647502d3f2aecd46de63f409317b4
Author: yatinkarel <email address hidden>
Date: Mon Dec 20 13:07:25 2021 +0530

    [OVN] Add reverse DNS records

    PTR DNS requests support is available since
    ovn-21.06.0[1].

    This patch adds/removes required enteries for each ip
    address of the port in DNS NB table, For example for ip
    addresses "10.0.0.4 fd5a:cdd8:f382:0:f816:3eff:fe5b:bb6"
    and fqdn "vm1.ovn.test." following enteries are added:-
    - 4.0.0.10.in-addr.arpa="vm1.ovn.test"
    - 6.b.b.0.b.5.e.f.f.f.e.3.6.1.8.f.0.0.0.0.2.8.3.f.8.d.d.c.a.5.d.f.ip6.arpa="vm1.ovn.test"

    [1] https://github.com/ovn-org/ovn/commit/82a4e44

    Closes-Bug: #1951872
    Change-Id: If03a2ad2475cdb390c4388d6869cd0b2a0555eb7

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
Frode Nordahl (fnordahl) wrote :

I would be interested in having this backported all the way to Ussuri and will prepare cherry-pick reviews. Our users are free to choose which version of OVN they want to use with OpenStack, and some may also choose to cherry-pick the addition of support for PTR records in OVN core.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/xena)

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/neutron/+/823594

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/wallaby)

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/neutron/+/823595

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/victoria)

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/neutron/+/823596

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/neutron/+/823631

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/823594
Committed: https://opendev.org/openstack/neutron/commit/5b30b14ad726e41793c76ecd0fca8ee66cf51607
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 5b30b14ad726e41793c76ecd0fca8ee66cf51607
Author: yatinkarel <email address hidden>
Date: Mon Dec 20 13:07:25 2021 +0530

    [OVN] Add reverse DNS records

    PTR DNS requests support is available since
    ovn-21.06.0[1].

    This patch adds/removes required enteries for each ip
    address of the port in DNS NB table, For example for ip
    addresses "10.0.0.4 fd5a:cdd8:f382:0:f816:3eff:fe5b:bb6"
    and fqdn "vm1.ovn.test." following enteries are added:-
    - 4.0.0.10.in-addr.arpa="vm1.ovn.test"
    - 6.b.b.0.b.5.e.f.f.f.e.3.6.1.8.f.0.0.0.0.2.8.3.f.8.d.d.c.a.5.d.f.ip6.arpa="vm1.ovn.test"

    [1] https://github.com/ovn-org/ovn/commit/82a4e44

    Closes-Bug: #1951872
    Change-Id: If03a2ad2475cdb390c4388d6869cd0b2a0555eb7
    (cherry picked from commit 5b597d6e4ee647502d3f2aecd46de63f409317b4)

tags: added: in-stable-xena
tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/823595
Committed: https://opendev.org/openstack/neutron/commit/591570c5f4069234bef460274aa5442b60eed5de
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 591570c5f4069234bef460274aa5442b60eed5de
Author: yatinkarel <email address hidden>
Date: Mon Dec 20 13:07:25 2021 +0530

    [OVN] Add reverse DNS records

    PTR DNS requests support is available since
    ovn-21.06.0[1].

    This patch adds/removes required enteries for each ip
    address of the port in DNS NB table, For example for ip
    addresses "10.0.0.4 fd5a:cdd8:f382:0:f816:3eff:fe5b:bb6"
    and fqdn "vm1.ovn.test." following enteries are added:-
    - 4.0.0.10.in-addr.arpa="vm1.ovn.test"
    - 6.b.b.0.b.5.e.f.f.f.e.3.6.1.8.f.0.0.0.0.2.8.3.f.8.d.d.c.a.5.d.f.ip6.arpa="vm1.ovn.test"

    [1] https://github.com/ovn-org/ovn/commit/82a4e44

    Closes-Bug: #1951872
    Change-Id: If03a2ad2475cdb390c4388d6869cd0b2a0555eb7
    (cherry picked from commit 5b597d6e4ee647502d3f2aecd46de63f409317b4)
    (cherry picked from commit 5b30b14ad726e41793c76ecd0fca8ee66cf51607)

tags: added: in-stable-victoria
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/823596
Committed: https://opendev.org/openstack/neutron/commit/3664aa1bb421cc1d7add0c2c5209af27b384ec1b
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 3664aa1bb421cc1d7add0c2c5209af27b384ec1b
Author: yatinkarel <email address hidden>
Date: Mon Dec 20 13:07:25 2021 +0530

    [OVN] Add reverse DNS records

    PTR DNS requests support is available since
    ovn-21.06.0[1].

    This patch adds/removes required enteries for each ip
    address of the port in DNS NB table, For example for ip
    addresses "10.0.0.4 fd5a:cdd8:f382:0:f816:3eff:fe5b:bb6"
    and fqdn "vm1.ovn.test." following enteries are added:-
    - 4.0.0.10.in-addr.arpa="vm1.ovn.test"
    - 6.b.b.0.b.5.e.f.f.f.e.3.6.1.8.f.0.0.0.0.2.8.3.f.8.d.d.c.a.5.d.f.ip6.arpa="vm1.ovn.test"

    [1] https://github.com/ovn-org/ovn/commit/82a4e44

    Closes-Bug: #1951872
    Change-Id: If03a2ad2475cdb390c4388d6869cd0b2a0555eb7
    (cherry picked from commit 5b597d6e4ee647502d3f2aecd46de63f409317b4)
    (cherry picked from commit 5b30b14ad726e41793c76ecd0fca8ee66cf51607)
    (cherry picked from commit 591570c5f4069234bef460274aa5442b60eed5de)

tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/ussuri)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/823631
Committed: https://opendev.org/openstack/neutron/commit/6f7bf0f0129b2714f99d18faa8a0bd87a2a3799e
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 6f7bf0f0129b2714f99d18faa8a0bd87a2a3799e
Author: yatinkarel <email address hidden>
Date: Mon Dec 20 13:07:25 2021 +0530

    [OVN] Add reverse DNS records

    PTR DNS requests support is available since
    ovn-21.06.0[1].

    This patch adds/removes required enteries for each ip
    address of the port in DNS NB table, For example for ip
    addresses "10.0.0.4 fd5a:cdd8:f382:0:f816:3eff:fe5b:bb6"
    and fqdn "vm1.ovn.test." following enteries are added:-
    - 4.0.0.10.in-addr.arpa="vm1.ovn.test"
    - 6.b.b.0.b.5.e.f.f.f.e.3.6.1.8.f.0.0.0.0.2.8.3.f.8.d.d.c.a.5.d.f.ip6.arpa="vm1.ovn.test"

    [1] https://github.com/ovn-org/ovn/commit/82a4e44

    Closes-Bug: #1951872
    Change-Id: If03a2ad2475cdb390c4388d6869cd0b2a0555eb7
    (cherry picked from commit 5b597d6e4ee647502d3f2aecd46de63f409317b4)
    (cherry picked from commit 5b30b14ad726e41793c76ecd0fca8ee66cf51607)
    (cherry picked from commit 591570c5f4069234bef460274aa5442b60eed5de)
    (cherry picked from commit 3664aa1bb421cc1d7add0c2c5209af27b384ec1b)
    Conflicts, trivial due to change near imports at top of file:
        neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovn_db_sync.py

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 20.0.0.0rc1

This issue was fixed in the openstack/neutron 20.0.0.0rc1 release candidate.

Revision history for this message
Ponnuvel Palaniyappan (pponnuvel) wrote :

Is there eta for uca releases with this fix? I see the fix has been backported to older releases but not available in latest UCA packages.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 17.4.0

This issue was fixed in the openstack/neutron 17.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 18.3.0

This issue was fixed in the openstack/neutron 18.3.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 19.2.0

This issue was fixed in the openstack/neutron 19.2.0 release.

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

Other bug subscribers

Remote bug watches

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