Page MenuHomePhabricator

Improvements to auto-generated floating ip ptr records
Closed, ResolvedPublic

Description

I know of at least two issues with our ptr record generation:

  1. records are generated with project id rather than project name (e.g. instance-mail.f6ec9c0903d44b86bfbc7072bb77c7d6.wmcloud.org. -- not very human readable)
  1. manually-created pointer records (see for example T421025) are clobbered by the automatic records. We need some way to support both, or prevent the clobbering.

Event Timeline

For #2, taavi points out that there is a boilerplate description for auto-created records

https://gerrit.wikimedia.org/r/plugins/gitiles/operations/puppet/+/refs/heads/production/modules/openstack/files/designate/wmcs-dns-floating-ip-updater.py#32

so it should be easy to have the daemon distinguish between manual and auto records.

Item #2 is already handled by the code. I don't know how/why my last attempt was clobbered; trying again.

Item #2 is already handled by the code. I don't know how/why my last attempt was clobbered; trying again.

OK, on the first pass the exclusion worked:

"WARNING : Did not update PTR record for 85.0-25.56.15.185.in-addr.arpa. due to lack of managed_description!"

But then after a later pass, it's gone...

Change #1264738 had a related patch set uploaded (by Andrew Bogott; author: Andrew Bogott):

[operations/puppet@production] floating_ip_updater: use project name (not id) for ptr records

https://gerrit.wikimedia.org/r/1264738

taavi triaged this task as Medium priority.Apr 1 2026, 1:56 PM

Change #1264738 merged by Andrew Bogott:

[operations/puppet@production] floating_ip_updater: use project name (not id) for ptr records

https://gerrit.wikimedia.org/r/1264738

Change #1270595 had a related patch set uploaded (by Andrew Bogott; author: Andrew Bogott):

[operations/puppet@production] floating_ip_updater: use project name (not id) for ptr records

https://gerrit.wikimedia.org/r/1270595

Change #1270595 merged by Andrew Bogott:

[operations/puppet@production] floating_ip_updater: use project name (not id) for ptr records

https://gerrit.wikimedia.org/r/1270595

#1 is fixed with attached patches

#2 was already handled; the reason I thought it wasn't was because of interactions with tofu-infra which are now also resolved.