netbox-plugin-dns 0.22.4__tar.gz → 0.22.5__tar.gz

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: netbox-plugin-dns
-Version: 0.22.4
+Version: 0.22.5
 Summary: NetBox DNS is a NetBox plugin for managing DNS data.
 Home-page: https://github.com/peteeckel/netbox-plugin-dns
 License: MIT
@@ -8,6 +8,7 @@ Keywords: netbox,netbox-plugin,dns
 Author: Peter Eckel
 Author-email: pete@netbox-dns.org
 Requires-Python: >=3.8,<4.0
+Classifier: Development Status :: 5 - Production/Stable
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.8
@@ -19,16 +20,15 @@ Requires-Dist: dnspython (>=2.2.1,<3.0.0)
 Project-URL: Repository, https://github.com/peteeckel/netbox-plugin-dns
 Description-Content-Type: text/markdown
 
-<h1 align="center">NetBox DNS</h1>
-
-<p align="center"><i>NetBox DNS is a NetBox plugin for managing DNS views, zones, name servers and records.</i></p>
+# NetBox DNS
+The NetBox DNS plugin enables NetBox to manage operational DNS data such as name servers, zones, records and views, as well as registration data for domains. It can automate tasks like creating PTR records, generating zone serial numbers, NS and SOA records, as well as validate names and values values for resource records to ensure zone data is consistent, current and conforming to the relevant RFCs.
 
 <div align="center">
 <a href="https://pypi.org/project/netbox-plugin-dns/"><img src="https://img.shields.io/pypi/v/netbox-plugin-dns" alt="PyPi"/></a>
-<a href="https://github.com/peteeckel/netbox-plugin-dns/stargazers"><img src="https://img.shields.io/github/stars/peteeckel/netbox-plugin-dns" alt="Stars Badge"/></a>
-<a href="https://github.com/peteeckel/netbox-plugin-dns/network/members"><img src="https://img.shields.io/github/forks/peteeckel/netbox-plugin-dns" alt="Forks Badge"/></a>
-<a href="https://github.com/peteeckel/netbox-plugin-dns/pulls"><img src="https://img.shields.io/github/issues-pr/peteeckel/netbox-plugin-dns" alt="Pull Requests Badge"/></a>
+<a href="https://github.com/peteeckel/netbox-plugin-dns/stargazers"><img src="https://img.shields.io/github/stars/peteeckel/netbox-plugin-dns?style=flat" alt="Stars Badge"/></a>
+<a href="https://github.com/peteeckel/netbox-plugin-dns/network/members"><img src="https://img.shields.io/github/forks/peteeckel/netbox-plugin-dns?style=flat" alt="Forks Badge"/></a>
 <a href="https://github.com/peteeckel/netbox-plugin-dns/issues"><img src="https://img.shields.io/github/issues/peteeckel/netbox-plugin-dns" alt="Issues Badge"/></a>
+<a href="https://github.com/peteeckel/netbox-plugin-dns/pulls"><img src="https://img.shields.io/github/issues-pr/peteeckel/netbox-plugin-dns" alt="Pull Requests Badge"/></a>
 <a href="https://github.com/peteeckel/netbox-plugin-dns/graphs/contributors"><img alt="GitHub contributors" src="https://img.shields.io/github/contributors/peteeckel/netbox-plugin-dns?color=2b9348"></a>
 <a href="https://github.com/peteeckel/netbox-plugin-dns/blob/master/LICENSE"><img src="https://img.shields.io/github/license/peteeckel/netbox-plugin-dns?color=2b9348" alt="License Badge"/></a>
 <a href="https://pepy.tech/project/netbox-plugin-dns"><img alt="Downloads" src="https://static.pepy.tech/badge/netbox-plugin-dns"></a>
@@ -40,10 +40,10 @@ Description-Content-Type: text/markdown
 
 * Manage name servers, zones and records
 * Automatically generate SOA and NS records for zones
-* Automatically create and update PTR records for IPv4 and IPv6 records
-* Optionally organize DNS zones in views for split horizon DNS and multi-site deployments
-* Optionally maintain domain registrar and registrant information for zones
-* Maintain RFC2317 reverse zones for IPv4 prefixes with a network mask length longer than 24 bits
+* Automatically create and update PTR records for IPv4 and IPv6 address records
+* Organize DNS zones in views for split horizon DNS and multi-site deployments
+* Manage domain registrar and registrant information for domains related to zones
+* Manage RFC2317 reverse zones for IPv4 prefixes with a network mask length longer than 24 bits
 
 NetBox DNS is using the standardized NetBox plugin interface, so it also takes advantage of the NetBox tagging and change log features.
 

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/README.md

@@ -1,13 +1,12 @@
-<h1 align="center">NetBox DNS</h1>
-
-<p align="center"><i>NetBox DNS is a NetBox plugin for managing DNS views, zones, name servers and records.</i></p>
+# NetBox DNS
+The NetBox DNS plugin enables NetBox to manage operational DNS data such as name servers, zones, records and views, as well as registration data for domains. It can automate tasks like creating PTR records, generating zone serial numbers, NS and SOA records, as well as validate names and values values for resource records to ensure zone data is consistent, current and conforming to the relevant RFCs.
 
 <div align="center">
 <a href="https://pypi.org/project/netbox-plugin-dns/"><img src="https://img.shields.io/pypi/v/netbox-plugin-dns" alt="PyPi"/></a>
-<a href="https://github.com/peteeckel/netbox-plugin-dns/stargazers"><img src="https://img.shields.io/github/stars/peteeckel/netbox-plugin-dns" alt="Stars Badge"/></a>
-<a href="https://github.com/peteeckel/netbox-plugin-dns/network/members"><img src="https://img.shields.io/github/forks/peteeckel/netbox-plugin-dns" alt="Forks Badge"/></a>
-<a href="https://github.com/peteeckel/netbox-plugin-dns/pulls"><img src="https://img.shields.io/github/issues-pr/peteeckel/netbox-plugin-dns" alt="Pull Requests Badge"/></a>
+<a href="https://github.com/peteeckel/netbox-plugin-dns/stargazers"><img src="https://img.shields.io/github/stars/peteeckel/netbox-plugin-dns?style=flat" alt="Stars Badge"/></a>
+<a href="https://github.com/peteeckel/netbox-plugin-dns/network/members"><img src="https://img.shields.io/github/forks/peteeckel/netbox-plugin-dns?style=flat" alt="Forks Badge"/></a>
 <a href="https://github.com/peteeckel/netbox-plugin-dns/issues"><img src="https://img.shields.io/github/issues/peteeckel/netbox-plugin-dns" alt="Issues Badge"/></a>
+<a href="https://github.com/peteeckel/netbox-plugin-dns/pulls"><img src="https://img.shields.io/github/issues-pr/peteeckel/netbox-plugin-dns" alt="Pull Requests Badge"/></a>
 <a href="https://github.com/peteeckel/netbox-plugin-dns/graphs/contributors"><img alt="GitHub contributors" src="https://img.shields.io/github/contributors/peteeckel/netbox-plugin-dns?color=2b9348"></a>
 <a href="https://github.com/peteeckel/netbox-plugin-dns/blob/master/LICENSE"><img src="https://img.shields.io/github/license/peteeckel/netbox-plugin-dns?color=2b9348" alt="License Badge"/></a>
 <a href="https://pepy.tech/project/netbox-plugin-dns"><img alt="Downloads" src="https://static.pepy.tech/badge/netbox-plugin-dns"></a>
@@ -19,10 +18,10 @@
 
 * Manage name servers, zones and records
 * Automatically generate SOA and NS records for zones
-* Automatically create and update PTR records for IPv4 and IPv6 records
-* Optionally organize DNS zones in views for split horizon DNS and multi-site deployments
-* Optionally maintain domain registrar and registrant information for zones
-* Maintain RFC2317 reverse zones for IPv4 prefixes with a network mask length longer than 24 bits
+* Automatically create and update PTR records for IPv4 and IPv6 address records
+* Organize DNS zones in views for split horizon DNS and multi-site deployments
+* Manage domain registrar and registrant information for domains related to zones
+* Manage RFC2317 reverse zones for IPv4 prefixes with a network mask length longer than 24 bits
 
 NetBox DNS is using the standardized NetBox plugin interface, so it also takes advantage of the NetBox tagging and change log features.
 

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/netbox_dns/__init__.py

@@ -10,7 +10,7 @@ except ImportError:
     # NetBox 3.5.8
     from extras.plugins.utils import get_plugin_config
 
-__version__ = "0.22.4"
+__version__ = "0.22.5"
 
 
 class DNSConfig(PluginConfig):
@@ -38,7 +38,8 @@ class DNSConfig(PluginConfig):
         ],
         "tolerate_non_rfc1035_types": [],
         "enable_root_zones": False,
-        "enforce_unique_records": False,
+        "enforce_unique_records": True,
+        "enforce_unique_rrset_ttl": True,
     }
     base_url = "netbox-dns"
 

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/netbox_dns/api/serializers.py

@@ -272,6 +272,7 @@ class RegistrarSerializer(NetBoxModelSerializer):
             "display",
             "name",
             "iana_id",
+            "address",
             "referral_url",
             "whois_server",
             "abuse_email",

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/netbox_dns/filters/registrar.py

@@ -12,6 +12,7 @@ class RegistrarFilter(NetBoxModelFilterSet):
             "id",
             "name",
             "iana_id",
+            "address",
             "referral_url",
             "whois_server",
             "abuse_email",

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/netbox_dns/forms/registrar.py

@@ -17,6 +17,7 @@ class RegistrarForm(NetBoxModelForm):
         fields = (
             "name",
             "iana_id",
+            "address",
             "referral_url",
             "whois_server",
             "abuse_email",
@@ -29,12 +30,18 @@ class RegistrarFilterForm(NetBoxModelFilterSetForm):
     model = Registrar
     fieldsets = (
         (None, ("q", "name", "iana_id", "tags")),
-        ("Contact", ("referral_url", "whois_server", "abuse_email", "abuse_phone")),
+        (
+            "Contact",
+            ("address", "referral_url", "whois_server", "abuse_email", "abuse_phone"),
+        ),
     )
 
     name = forms.CharField(
         required=False,
     )
+    address = forms.CharField(
+        required=False,
+    )
     iana_id = forms.IntegerField(
         required=False,
         label="IANA ID",
@@ -64,6 +71,7 @@ class RegistrarImportForm(NetBoxModelImportForm):
         fields = (
             "name",
             "iana_id",
+            "address",
             "referral_url",
             "whois_server",
             "abuse_email",
@@ -96,6 +104,7 @@ class RegistrarBulkEditForm(NetBoxModelBulkEditForm):
         (
             None,
             (
+                "address",
                 "referral_url",
                 "whois_server",
                 "abuse_email",
@@ -105,6 +114,7 @@ class RegistrarBulkEditForm(NetBoxModelBulkEditForm):
     )
 
     nullable_fields = (
+        "address",
         "referral_url",
         "whois_server",
         "abuse_email",

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/netbox_dns/forms/zone.py

@@ -315,7 +315,7 @@ class ZoneImportForm(NetBoxModelImportForm):
         required=False,
         help_text="Mailbox of the zone's administrator",
     )
-    soa_serial_auto = forms.BooleanField(
+    soa_serial_auto = forms.NullBooleanField(
         required=False,
         help_text="Generate the SOA serial",
     )
@@ -443,26 +443,6 @@ class ZoneImportForm(NetBoxModelImportForm):
     def clean_soa_rname(self):
         return self._clean_field_with_defaults("soa_rname")
 
-    def clean_soa_serial_auto(self):
-        try:
-            return self._clean_field_with_defaults("soa_serial_auto")
-        except ValidationError:
-            if self.cleaned_data["soa_serial"] or self._get_default_value("soa_serial"):
-                return None
-
-            raise
-
-    def clean_soa_serial(self):
-        try:
-            return self._clean_field_with_defaults("soa_serial")
-        except ValidationError:
-            if self.cleaned_data["soa_serial_auto"] or self._get_default_value(
-                "soa_serial_auto"
-            ):
-                return None
-
-            raise
-
     def clean_soa_refresh(self):
         return self._clean_field_with_defaults("soa_refresh")
 
@@ -475,6 +455,36 @@ class ZoneImportForm(NetBoxModelImportForm):
     def clean_soa_minimum(self):
         return self._clean_field_with_defaults("soa_minimum")
 
+    def clean(self, *args, **kwargs):
+        super().clean(*args, **kwargs)
+
+        soa_serial_auto = self.cleaned_data.get("soa_serial_auto")
+        soa_serial = self.cleaned_data.get("soa_serial")
+
+        if soa_serial is None:
+            soa_serial = self._get_default_value("soa_serial")
+
+        if soa_serial_auto is None:
+            if self._get_default_value("soa_serial_auto") is not None:
+                soa_serial_auto = self._get_default_value("soa_serial_auto")
+
+            elif soa_serial is not None:
+                soa_serial_auto = False
+
+            else:
+                raise ValidationError(
+                    "SOA Serial Auto not set and no default value and SOA Serial available"
+                )
+
+        if "soa_serial_auto" in self.cleaned_data:
+            self.cleaned_data["soa_serial_auto"] = soa_serial_auto
+
+        if "soa_serial" in self.cleaned_data:
+            if soa_serial_auto:
+                self.cleaned_data["soa_serial"] = None
+            else:
+                self.cleaned_data["soa_serial"] = soa_serial
+
     class Meta:
         model = Zone
 

netbox_plugin_dns-0.22.5/netbox_dns/management/commands/cleanup_rrset_ttl.py

@@ -0,0 +1,62 @@
+from django.core.management.base import BaseCommand
+from django.db.models import Max, Min
+
+from netbox_dns.models import (
+    Record,
+    RecordTypeChoices,
+)
+
+
+class Command(BaseCommand):
+    help = "Clean up the TTLs for RRSets"
+
+    def add_arguments(self, parser):
+        min_max = parser.add_mutually_exclusive_group()
+        min_max.add_argument(
+            "--min",
+            action="store_true",
+            help="Use the minimum TTL of an RRSet for all Records",
+        )
+        min_max.add_argument(
+            "--max",
+            action="store_true",
+            help="Use the maximum TTL of an RRSet for all Records",
+        )
+
+    def handle(self, *model_names, **options):
+        self.cleanup_rrset_ttl(**options)
+
+        self.stdout.write("RRSet cleanup completed.")
+
+    def cleanup_rrset_ttl(self, **options):
+        verbose = options.get("verbosity") > 1
+
+        ttl_records = (
+            Record.objects.filter(ttl__isnull=False)
+            .exclude(type=RecordTypeChoices.SOA)
+            .exclude(type=RecordTypeChoices.PTR, maanged=True)
+        )
+        for record in ttl_records:
+            records = Record.objects.filter(
+                name=record.name,
+                zone=record.zone,
+                type=record.type,
+            ).exclude(type=RecordTypeChoices.PTR, maanged=True)
+
+            if records.count() == 1:
+                if options.get("verbosity") > 2:
+                    self.stdout.write(f"Ignoring single record {record.id} ({record})")
+                continue
+
+            if options.get("max"):
+                ttl = records.aggregate(Max("ttl")).get("ttl__max")
+            else:
+                ttl = records.aggregate(Min("ttl")).get("ttl__min")
+
+            for record in records.exclude(ttl=ttl):
+                if options.get("verbosity") > 1:
+                    self.stdout.write(
+                        f"Updating TTL for record {record.id} ({record}) to {ttl}"
+                    )
+                record.ttl = ttl
+                record.save(update_fields=["ttl"], update_rrset_ttl=False)

{netbox_plugin_dns-0.22.4 → netbox_plugin_dns-0.22.5}/netbox_dns/models/record.py

@@ -6,7 +6,7 @@ from dns import name as dns_name
 
 from django.core.exceptions import ValidationError
 from django.db import transaction, models
-from django.db.models import Q, ExpressionWrapper, BooleanField
+from django.db.models import Q, ExpressionWrapper, BooleanField, Min
 from django.db.models.functions import Length
 from django.urls import reverse
 
@@ -38,6 +38,10 @@ from netbox_dns.validators import (
 import netbox_dns.models.zone as zone
 
 
+def min_ttl(*ttl_list):
+    return min((ttl for ttl in ttl_list if ttl is not None), default=None)
+
+
 class RecordManager(models.Manager.from_queryset(RestrictedQuerySet)):
     """Special Manager for records providing the activity status annotation"""
 
@@ -207,7 +211,8 @@ class Record(NetBoxModel):
         try:
             name = (
                 dns_name.from_text(
-                    self.name, origin=dns_name.from_text(self.zone.name, origin=None)
+                    str(self.name),
+                    origin=dns_name.from_text(self.zone.name, origin=None),
                 )
                 .relativize(dns_name.root)
                 .to_unicode()
@@ -309,7 +314,7 @@ class Record(NetBoxModel):
 
         return ptr_zone
 
-    def update_ptr_record(self, update_rfc2317_cname=True):
+    def update_ptr_record(self, update_rfc2317_cname=True, save_zone_serial=True):
         ptr_zone = self.ptr_zone
 
         if (
@@ -339,14 +344,16 @@ class Record(NetBoxModel):
                 not ptr_record.zone.is_rfc2317_zone
                 and ptr_record.rfc2317_cname_record is not None
             ):
-                ptr_record.rfc2317_cname_record.delete()
+                ptr_record.rfc2317_cname_record.delete(
+                    save_zone_serial=save_zone_serial
+                )
 
         with transaction.atomic():
             if ptr_record is not None:
                 if ptr_record.zone.pk != ptr_zone.pk:
                     if ptr_record.rfc2317_cname_record is not None:
                         ptr_record.rfc2317_cname_record.delete()
-                    ptr_record.delete()
+                    ptr_record.delete(save_zone_serial=save_zone_serial)
                     ptr_record = None
 
                 else:
@@ -358,7 +365,7 @@ class Record(NetBoxModel):
                         ptr_record.name = ptr_name
                         ptr_record.value = ptr_value
                         ptr_record.ttl = self.ttl
-                        ptr_record.save()
+                        ptr_record.save(save_zone_serial=save_zone_serial)
 
             if ptr_record is None:
                 ptr_record = Record(
@@ -369,11 +376,14 @@ class Record(NetBoxModel):
                     value=ptr_value,
                     managed=True,
                 )
-                ptr_record.save(update_rfc2317_cname=update_rfc2317_cname)
+                ptr_record.save(
+                    update_rfc2317_cname=update_rfc2317_cname,
+                    save_zone_serial=save_zone_serial,
+                )
 
         self.ptr_record = ptr_record
 
-    def update_rfc2317_cname_record(self):
+    def update_rfc2317_cname_record(self, save_zone_serial=True):
         if self.zone.rfc2317_parent_managed:
             cname_name = dns_name.from_text(
                 ipaddress.ip_address(self.ip_address).reverse_pointer
@@ -383,7 +393,13 @@ class Record(NetBoxModel):
                 self.rfc2317_cname_record.name = cname_name
                 self.rfc2317_cname_record.zone = self.zone.rfc2317_parent_zone
                 self.rfc2317_cname_record.value = self.fqdn
-                self.rfc2317_cname_record.save()
+                self.rfc2317_cname_record.ttl = min_ttl(
+                    self.rfc2317_cname_record.rfc2317_ptr_records.exclude(pk=self.pk)
+                    .aggregate(Min("ttl"))
+                    .get("ttl__min"),
+                    self.ttl,
+                )
+                self.rfc2317_cname_record.save(save_zone_serial=save_zone_serial)
             else:
                 rfc2317_cname_record = Record.objects.filter(
                     name=cname_name,
@@ -392,20 +408,34 @@ class Record(NetBoxModel):
                     managed=True,
                     value=self.fqdn,
                 ).first()
-                if rfc2317_cname_record is None:
-                    rfc2317_cname_record = Record.objects.create(
+
+                if rfc2317_cname_record is not None:
+                    rfc2317_cname_record.ttl = min_ttl(
+                        rfc2317_cname_record.rfc2317_ptr_records.exclude(pk=self.pk)
+                        .aggregate(Min("ttl"))
+                        .get("ttl__min"),
+                        self.ttl,
+                    )
+                    rfc2317_cname_record.save(
+                        update_fields=["ttl"], save_zone_serial=save_zone_serial
+                    )
+
+                else:
+                    rfc2317_cname_record = Record(
                         name=cname_name,
                         type=RecordTypeChoices.CNAME,
                         zone=self.zone.rfc2317_parent_zone,
                         managed=True,
                         value=self.fqdn,
+                        ttl=self.ttl,
                     )
+                    rfc2317_cname_record.save(save_zone_serial=save_zone_serial)
 
                 self.rfc2317_cname_record = rfc2317_cname_record
 
         else:
             if self.rfc2317_cname_record is not None:
-                self.rfc2317_cname_record.delete()
+                self.rfc2317_cname_record.delete(save_zone_serial=save_zone_serial)
                 self.rfc2317_cname_record = None
 
     def validate_name(self):
@@ -475,7 +505,7 @@ class Record(NetBoxModel):
                 }
             ) from None
 
-    def check_unique(self):
+    def check_unique_record(self):
         if not get_plugin_config("netbox_dns", "enforce_unique_records", False):
             return
 
@@ -489,6 +519,10 @@ class Record(NetBoxModel):
             value=self.value,
             status__in=Record.ACTIVE_STATUS_LIST,
         )
+
+        if self.pk is not None:
+            records = records.exclude(pk=self.pk)
+
         if len(records):
             raise ValidationError(
                 {
@@ -496,6 +530,64 @@ class Record(NetBoxModel):
                 }
             ) from None
 
+    def check_unique_rrset_ttl(self):
+        if self.pk is not None:
+            return
+
+        if not get_plugin_config("netbox_dns", "enforce_unique_rrset_ttl", False):
+            return
+
+        if self.type == RecordTypeChoices.PTR and self.managed:
+            return
+
+        records = (
+            Record.objects.filter(
+                zone=self.zone,
+                name=self.name,
+                type=self.type,
+            )
+            .exclude(ttl=self.ttl)
+            .exclude(type=RecordTypeChoices.PTR, managed=True)
+        )
+
+        if not records.exists():
+            return
+
+        conflicting_ttls = ", ".join(set(str(record.ttl) for record in records))
+        raise ValidationError(
+            {
+                "ttl": f"There is at least one active {self.type} record for name {self.name} in zone {self.zone} and TTL is different ({conflicting_ttls})."
+            }
+        ) from None
+
+    def update_rrset_ttl(self, ttl=None):
+        if self.pk is None:
+            return
+
+        if not get_plugin_config("netbox_dns", "enforce_unique_rrset_ttl", False):
+            return
+
+        if self.type == RecordTypeChoices.PTR and self.managed:
+            return
+
+        if ttl is None:
+            ttl = self.ttl
+
+        records = (
+            Record.objects.filter(
+                zone=self.zone,
+                name=self.name,
+                type=self.type,
+            )
+            .exclude(pk=self.pk)
+            .exclude(ttl=ttl)
+            .exclude(type=RecordTypeChoices.PTR, managed=True)
+        )
+
+        for record in records:
+            record.ttl = ttl
+            record.save(update_fields=["ttl"], update_rrset_ttl=False)
+
     def clean_fields(self, *args, **kwargs):
         self.type = self.type.upper()
         super().clean_fields(*args, **kwargs)
@@ -503,7 +595,9 @@ class Record(NetBoxModel):
     def clean(self, *args, **kwargs):
         self.validate_name()
         self.validate_value()
-        self.check_unique()
+        self.check_unique_record()
+        if self.pk is None:
+            self.check_unique_rrset_ttl()
 
         if not self.is_active:
             return
@@ -578,14 +672,24 @@ class Record(NetBoxModel):
                     }
                 ) from None
 
-    def save(self, *args, update_rfc2317_cname=True, **kwargs):
+    def save(
+        self,
+        *args,
+        update_rfc2317_cname=True,
+        save_zone_serial=True,
+        update_rrset_ttl=True,
+        **kwargs,
+    ):
         self.full_clean()
 
+        if self.pk is not None and update_rrset_ttl:
+            self.update_rrset_ttl()
+
         if self.is_ptr_record:
             if self.zone.is_rfc2317_zone:
                 self.ip_address = self.address_from_rfc2317_name
                 if update_rfc2317_cname:
-                    self.update_rfc2317_cname_record()
+                    self.update_rfc2317_cname_record(save_zone_serial=save_zone_serial)
             else:
                 self.ip_address = self.address_from_name
 
@@ -595,7 +699,10 @@ class Record(NetBoxModel):
             self.ip_address = None
 
         if self.is_address_record:
-            self.update_ptr_record(update_rfc2317_cname=update_rfc2317_cname)
+            self.update_ptr_record(
+                update_rfc2317_cname=update_rfc2317_cname,
+                save_zone_serial=save_zone_serial,
+            )
         elif self.ptr_record is not None:
             self.ptr_record.delete()
             self.ptr_record = None
@@ -604,15 +711,24 @@ class Record(NetBoxModel):
 
         zone = self.zone
         if self.type != RecordTypeChoices.SOA and zone.soa_serial_auto:
-            zone.update_serial()
+            zone.update_serial(save_zone_serial=save_zone_serial)
 
-    def delete(self, *args, **kwargs):
+    def delete(self, *args, save_zone_serial=True, **kwargs):
         if self.rfc2317_cname_record:
-            if (
-                self.rfc2317_cname_record.pk
-                and self.rfc2317_cname_record.rfc2317_ptr_records.count() == 1
-            ):
-                self.rfc2317_cname_record.delete()
+            if self.rfc2317_cname_record.pk:
+                if self.rfc2317_cname_record.rfc2317_ptr_records.count() == 1:
+                    self.rfc2317_cname_record.delete()
+                else:
+                    self.rfc2317_cname_record.ttl = (
+                        self.rfc2317_cname_record.rfc2317_ptr_records.exclude(
+                            pk=self.pk
+                        )
+                        .aggregate(Min("ttl"))
+                        .get("ttl__min")
+                    )
+                    self.rfc2317_cname_record.save(
+                        update_fields=["ttl"], save_zone_serial=save_zone_serial
+                    )
 
         if self.ptr_record:
             self.ptr_record.delete()
@@ -621,7 +737,7 @@ class Record(NetBoxModel):
 
         zone = self.zone
         if zone.soa_serial_auto:
-            zone.update_serial()
+            zone.update_serial(save_zone_serial=save_zone_serial)
 
 
 @register_search