Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a custom designate driver for FIP's #70

Open
wants to merge 16 commits into
base: stable/yoga-m3
Choose a base branch
from
1 change: 0 additions & 1 deletion neutron/db/dns_db.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,6 @@ def _process_dns_floatingip_delete(self, context, floatingip_data):
return
dns_data_db = fip_obj.FloatingIPDNS.get_object(
context, floatingip_id=floatingip_data['id'])

if dns_data_db:
self._delete_floatingip_from_external_dns_service(
context, dns_data_db['published_dns_domain'],
Expand Down
17 changes: 16 additions & 1 deletion neutron/services/externaldns/drivers/designate/driver_ccloud.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright (c) 2016 IBM
# Copyright (c) 2025 SAP SE
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copyright is cumulative (i.e. as long as we keep some of the original code, the original's author still applies to that part of the code).
So please don't remove IBMs copyright, but instead add ours next to it, else this would be an IP infringement.
(Needed only for files that we actually change.)

# All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
Expand Down Expand Up @@ -102,6 +102,21 @@ def __init__(self):

def create_record_set(self, context, dns_domain, dns_name, records,
fip_id=None):
"""Create a record set in the specified zone.

:param context: neutron api request context
:type context: neutron_lib.context.Context
:param dns_domain: the dns_domain where the record set will be created
:type dns_domain: String
:param dns_name: the name associated with the record set
:type dns_name: String
:param records: the records in the set
:type records: List of Strings
:param fip_id: Floating IP id
:type fip_id: String
:raises: neutron.extensions.dns.DNSDomainNotFound
neutron.extensions.dns.DuplicateRecordSet
"""
designate, designate_admin = get_clients(context)
v4, v6 = self._classify_records(records)
try:
Expand Down
Loading