Bug #16952

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

Added by Robert Frank 9 months ago. Updated 9 months 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 9 months ago

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

History

#1 Updated by Dominic Cleal 9 months ago

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

#2 Updated by Dominic Cleal 9 months 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 9 months ago

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

#4 Updated by The Foreman Bot 9 months ago

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

#5 Updated by Timo Goebel 9 months ago

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

Also available in: Atom PDF