Project

General

Profile

Feature #17879

Add DNS CNAME entry for nsupdate

Added by Guilherme Maluf Balzana almost 4 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Category:
DNS
Target version:
Difficulty:
easy
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

DNS managed by Foreman should be able to assign aliases for hosts.

Smart-proxy API currently support DNS CNAME entries for DNSCMD only.

This feature should able CNAME support on dnsupdate DNS provider and make it possible to assign more then one CNAME to a host1.

Issues related with this subject
#14091 #9388 #1581 #15244

1 https://github.com/theforeman/smart-proxy/blob/develop/modules/dns_common/dns_common.rb#L108


Related issues

Related to Smart Proxy - Refactor #17906: Move create_*_record remove_*_record functions into dns_commonClosed2017-01-03
Blocks Smart Proxy - Refactor #17880: DNS CNAME parameters methods are swappedRejected2016-12-29
Copied from Foreman - Feature #17878: Add DNS CNAME entry for nsupdateRejected2016-12-29

Associated revisions

Revision 47732b9e (diff)
Added by Ewoud Kohl van Wijngaarden almost 4 years ago

Fixes #17906 - Move {create,remove}_*_record to dns_common

Before most DNS providers had pretty much exact copies of these
functions. This removes duplication and allows for much simpler plugins.

In doing so it also fixes #17879 by adding CNAME support to the nsupdate
module.

Lastly it fixes some require statements to allow running single tests
rather than relying on other tests to do their requirements for them.

History

#1 Updated by Guilherme Maluf Balzana almost 4 years ago

#2 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/489 added

#3 Updated by Guilherme Maluf Balzana almost 4 years ago

  • Related to Refactor #17880: DNS CNAME parameters methods are swapped added

#4 Updated by Guilherme Maluf Balzana almost 4 years ago

  • Related to deleted (Refactor #17880: DNS CNAME parameters methods are swapped)

#5 Updated by Guilherme Maluf Balzana almost 4 years ago

#6 Updated by Dmitri Dolguikh almost 4 years ago

  • Related to Refactor #17906: Move create_*_record remove_*_record functions into dns_common added

#7 Updated by Dmitri Dolguikh almost 4 years ago

  • Status changed from Ready For Testing to Rejected

Closing this in favour of http://projects.theforeman.org/issues/17906 which provides default implementations for all create and delete record methods for all providers.

#8 Updated by Ewoud Kohl van Wijngaarden almost 4 years ago

  • Status changed from Rejected to Resolved
  • Legacy Backlogs Release (now unused) set to 209

I think closing it as resolved makes it clearer this is now supported. The other issue kind of sneaks it in.

Also available in: Atom PDF