Mellanox / nvidia-k8s-ipam / 34573684208

60%

Build Type

push

github

Committed by web-flow

Commit Message

feat: configurable stale-allocation cleanup and release cooldown for IP allocations (#280)

* feat: add flags to configure stale IP cleanup interval and TTL

Cooldown before released IPs can be reused is a per-datacenter policy,
but ipam-node hardcoded the cleaner's check interval (1m) and stale
check count (3), giving no way to tune the release TTL.
Adds --stale-ip-check-interval and --stale-ip-check-count-before-release
flags to configure them, defaulting to the prior hardcoded values.

Signed-off-by: Wei-Chen Chen <zarcen@gmail.com>
Signed-off-by: Wei-Chen Chen <weichen.chen@omniva.com>

* feat: add release cooldown for stale IP allocations

Normal CNI DEL releases an IP immediately, so the stale-cleanup flags
alone don't provide a cooldown before an IP can be reused (#181).
Adds --released-ip-cooldown to hold a released reservation in the
store, blocking its IP from reuse, until the cleaner removes it once
the cooldown expires. Defaults to 0, preserving immediate release.

Signed-off-by: Wei-Chen Chen <zarcen@gmail.com>
Signed-off-by: Wei-Chen Chen <weichen.chen@omniva.com>

* chore: apply release cooldown to orphan cleanup too

Signed-off-by: Wei-Chen Chen <zarcen@gmail.com>
Signed-off-by: Wei-Chen Chen <weichen.chen@omniva.com>

* chore: exclude ReleasedAt from the checksum in store

Signed-off-by: Wei-Chen Chen <zarcen@gmail.com>
Signed-off-by: Wei-Chen Chen <weichen.chen@omniva.com>

---------

Signed-off-by: Wei-Chen Chen <zarcen@gmail.com>
Signed-off-by: Wei-Chen Chen <weichen.chen@omniva.com>

Coverage Stats

77 of 127 new or added lines in 9 files covered. (60.63%)

1 existing line in 1 file now uncovered.

2721 of 4546 relevant lines covered (59.85%)

92.2 hits per line