Bug #16952

Removing ipv6 entries does not pass record type (AAAA) to smart proxy

Added by Robert Frank about 1 year ago. Updated about 1 year ago.

Status:Closed
Priority:High
Assigned To:Timo Goebel
Category:DNS
Target version:-
Difficulty: Bugzilla link:
Found in release:1.13.0 Pull request:https://github.com/theforeman/foreman/pull/3948
Story points-
Velocity based estimate-
Release1.13.1Release relationshipAuto

Description

When I try to remove the IPv6 address from a host, the smart proxy fails to update the DNS server.

  1. The foreman server doesn't set the resource type when sending the delete request for an fqdn to the proxy. It calls <url>/dns/<fqdn> twice instead of <url>/dns/<fqdn>/A and <url>/dns/<fqdn>/AAAA. This causes the proxy to default to A and deleting the A record twice (rather than the A and the AAAA record). I believe this can be fixed by changing the destroy method in ./lib/net/dns/forward_record.rb to
    def destroy
      super
      proxy.delete("#{hostname}/#{type}")
    

Related issues

Copied to Smart Proxy - Bug #16964: Removing ipv6 entries fails with dns_nsupdate due to name... Closed 10/14/2016

Associated revisions

Revision 39ae1782
Added by Timo Goebel about 1 year ago

fixes #16952 - pass type to proxy when destroying dns records

Revision d1fca2aa
Added by Timo Goebel about 1 year ago

fixes #16952 - pass type to proxy when destroying dns records

(cherry picked from commit 39ae1782ffde34a71f6ca2092acc8efd56522039)

History

#1 Updated by Dominic Cleal about 1 year ago

  • Copied to Bug #16964: Removing ipv6 entries fails with dns_nsupdate due to name lookup added

#2 Updated by Dominic Cleal about 1 year ago

  • Subject changed from Removing ipv6 entries does not work with dns_nsupdate proxy (foreman 1.13.0) to Removing ipv6 entries does not pass record type (AAAA) to smart proxy
  • Description updated (diff)
  • Category changed from Smart proxies to DNS
  • Release set to 1.13.1
  • Found in release set to 1.13.0

#3 Updated by Timo Goebel about 1 year ago

  • Status changed from New to Assigned
  • Assigned To set to Timo Goebel

#4 Updated by The Foreman Bot about 1 year ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3948 added

#5 Updated by Timo Goebel about 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF