Skip to content

Merge infrahub-release-1.12 into infrahub-develop - #1467

Merged
infrahub-github-bot-app[bot] merged 1 commit into
infrahub-developfrom
infrahub-release-1.12
Oct 30, 2026
Merged

infrahub-github-bot-app[bot] merged 1 commit into
infrahub-developfrom
infrahub-release-1.12

Conversation

@infrahub-github-bot-app

@infrahub-github-bot-app infrahub-github-bot-app Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Merging infrahub-release-1.12 into infrahub-develop after merging pull request #1466.


Summary by cubic

Merges infrahub-release-1.12 into infrahub-develop, carrying the change that makes number_pool_id a read-only field on NumberPool attribute parameters.

  • Schema validation now warns that number_pool_id is ignored instead of accepting it, and the server no longer stores a pool id that points to nothing.
  • Regenerated SDK schema files (contract.py, write.py) reflect the updated field set for NumberPoolParametersWrite.

Written for commit 63e3bd6. Summary will update on new commits.

View guided diff Turn on auto-fix

)

Only the server's schema synchronizer assigns the pool of a NumberPool
attribute. A schema file that carries the id, for example one exported
from another instance, now validates with a read-only warning and the
server ignores the value instead of storing an id that points to no pool.

Regenerated from the Infrahub generator (tasks/backend.py).

Co-authored-by: Aaron McCarty <19253956+ajtmccarty@users.noreply.github.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 63e3bd6
Status: ✅  Deploy successful!
Preview URL: https://c388a9dc.infrahub-sdk-python.pages.dev
Branch Preview URL: https://infrahub-release-1-12.infrahub-sdk-python.pages.dev

View logs

@codecov

codecov Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@                 Coverage Diff                  @@
##           infrahub-develop    #1467      +/-   ##
====================================================
- Coverage             87.74%   87.74%   -0.01%     
====================================================
  Files                   155      155              
  Lines                 15518    15517       -1     
  Branches               2085     2085              
====================================================
- Hits                  13616    13615       -1     
  Misses                 1345     1345              
  Partials                557      557              
Flag Coverage Δ
integration-tests 42.52% <ø> (+<0.01%) ⬆️
python-3.10 62.54% <ø> (+0.01%) ⬆️
python-3.11 62.54% <ø> (+0.01%) ⬆️
python-3.12 62.54% <ø> (+0.01%) ⬆️
python-3.13 62.53% <ø> (+<0.01%) ⬆️
python-3.14 62.54% <ø> (+<0.01%) ⬆️
python-3.15 62.54% <ø> (+0.01%) ⬆️
python-filler-3.12 23.16% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
infrahub_sdk/schema/generated/contract.py 100.00% <ø> (ø)
infrahub_sdk/schema/generated/write.py 99.40% <ø> (-0.01%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@infrahub-github-bot-app
infrahub-github-bot-app Bot merged commit 7c58f63 into infrahub-develop Oct 30, 2026
30 checks passed
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.

1 participant