nokia / danm-utils

BSD 3-Clause "New" or "Revised" License
6 stars 9 forks source link

Cleaner: reconciling the handling of "no release service" and "no IP" edge cases #30

Open Levovar opened 3 years ago

Levovar commented 3 years ago

DANM IPAM assumes "empty" IPs are not "allocated" by it, which is technically true. However, Cleaner treated this neutral use-case as an error, which actually resulted in un-related addresses remaining non-cleaned, and DanmEps being unnecessarily retained.

Handling is changed to a more future-proof approach: we don't treat non-registered release implementations as errors anymore, and failing to clean one IP will not block the release process of another.