Repository navigation
Merge infrahub-release-1.12 into infrahub-develop - #1464
Merged
infrahub-github-bot-app[bot] merged 1 commit intoOct 28, 2026
Merged
Conversation
…#1463) * feat(node): attach a held number to a number pool or detach it [IFC-3333] Number attributes have attach_to_pool() and detach_from_pool(), sent on the next save. An attribute built with both value and from_pool now sends both, so the pool tracks the given number instead of allocating a new one. A pool write is sent once: a pending flag is cleared after a successful mutation, while _from_pool stays set. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(node): list a pool's allocations with their provenance [IFC-3367] Resource pool nodes have get_pool_allocations(), which returns PoolAllocation entries: the node id, kind, branch, display label, identifier and whether the pool allocated the number or a user provided it (None for IP pools). It needs Infrahub 1.12 or later. get_pool_allocated_resources() shares the paging loop but still leaves provenance out of its query, so it keeps working on older servers. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(node): declare a number pool's ranges on create and upsert [IFC-3369] A relationship whose peer kind is CoreNumberPoolRange takes the peer's fields as mutation input instead of a reference, as Infrahub's NumberPoolRangeInput does. A CoreNumberPool can therefore be created with ranges given as dicts of start, end and allocation_weight, and an upsert sets an existing pool's ranges to the declared list. Before, each range was sent as an empty object, which Infrahub read as one range without a start or an end. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(integration): number pool ranges, attach/detach and provenance on Infrahub 1.12 Covers creating a pool with declared ranges and setting them by upsert, attaching and detaching numbers (including a detach before the first save), the provenance get_pool_allocations reports, and the attribute's source after an attach. The class is skipped unless INFRAHUB_TESTING_IMAGE_VER is 1.12 or later. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(changelog): split the from_pool fix and shorten the provenance entry Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * refactor(node): move the detach comment onto the branch it describes Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(node): check a detached attribute keeps its number Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: Aaron McCarty <19253956+ajtmccarty@users.noreply.github.com>
github-actions
Bot
added
the
type/documentation
Deploying infrahub-sdk-python with
|
| Latest commit: |
4847741
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://310ff18b.infrahub-sdk-python.pages.dev |
| Branch Preview URL: | https://infrahub-release-1-12.infrahub-sdk-python.pages.dev |
Codecov Report❌ Patch coverage is
@@ Coverage Diff @@
## infrahub-develop #1464 +/- ##
====================================================
+ Coverage 87.48% 87.57% +0.08%
====================================================
Files 155 155
Lines 15350 15431 +81
Branches 2064 2078 +14
====================================================
+ Hits 13429 13513 +84
+ Misses 1362 1358 -4
- Partials 559 560 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 5 files with indirect coverage changes 🚀 New features to boost your workflow:
|
infrahub-github-bot-app
Bot
merged commit Oct 28, 2026
7aac211
into
infrahub-develop
30 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merging infrahub-release-1.12 into infrahub-develop after merging pull request #1463.
Summary by cubic
Merges the
infrahub-release-1.12branch intodevelop, bringing the number pool SDK enhancements and a fix for repeatedfrom_poolrequests.** New Features**
attach_to_pool()anddetach_from_pool()let a chosen number be tracked by or removed from a number pool on the next save.get_pool_allocations()lists a pool's held resources with provenance, showing whether the pool allocated each number or a user provided it.CoreNumberPoolcan be created or upserted with its ranges declared inline instead of as empty objects.** Bug Fixes**
from_poolrequest is sent once: after a successful save, the next update no longer sends it again.Written for commit 4847741. Summary will update on new commits.