Skip to content

feat: Autocreate missing PTRs, handle notFound errors gracefully - #81

Open
fust wants to merge 1 commit into
AbsaOSS:mainfrom
fust:feat/autocreate-missing-ptrs
Open

fust wants to merge 1 commit into
AbsaOSS:mainfrom
fust:feat/autocreate-missing-ptrs

Conversation

@fust

@fust fust commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

When CreatePTR is enabled, an A record update would fail if the corresponding PTR record had been deleted or never existed in Infoblox, causing a reconciliation loop with no recovery. This change detects that scenario and creates the missing PTR instead of trying to update a non-existent record.

Additionally, notFound errors from Infoblox were previously silently swallowed in recordSet, incrementing the failed API call counter and making it impossible to distinguish a missing record from a real error in logs or metrics.
Errors are now wrapped with contextual record info before the notFound check, so notFound is treated as an expected condition (no counter bump, no early return) while other errors are still surfaced.
A minor metric label bug ("GetObject" on the create path) and action error logging in submitChanges are also fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants