Skip to content

Conversation

@elitegreg
Copy link
Contributor

Summary of Changes

  • Changes ip_allocator so that instead of allocating fixed length NetworkV4s, it allocates the number of addresses passed. e.g. allocate(buffer, 1) for /32 and allocate(buffer, 2) for /31
  • Updated call sites to pass the number to allocate.
  • Behavior of iter_allocated on the allocator changed to return each individual /32. Before if /31s were allocated, a single tunnel allocation (2 addresses) would only list once in iter_allocated. Due to the implementation change, this now must list each /32.
  • Updated tests

Testing Verification

  • unit tests pass

* Changes ip_allocator so that instead of allocating fixed length
  NetworkV4s, it allocates the number of addresses passed.
* Updated call sites to pass the number to allocate.
* Behavior of iter_allocated on the allocator changed to return each
  individual /32. Before if /31s were allocated, a single tunnel
  allocation (2 addresses) would only list once in iter_allocated. Due
  to the implementation change, this now must list each /32.
* Updated tests
@elitegreg elitegreg enabled auto-merge (squash) January 22, 2026 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants