Bring `infrahub-develop` changes into `develop` by gmazoyer · Pull Request #948 · opsmill/infrahub-sdk-python

gmazoyer · 2026-04-14T16:08:38Z

No description provided.

Merge develop into infrahub-develop

* update schemas related to CoreKeyValue object * putting the value of KeyValue object into generic schema * updated schema since the CoreEnvironmentVariableKeyValue has been renamed CoreEnvKeyValue --------- Co-authored-by: polmichel <pol@opsmill.com>

This parameter has been deprecated for a while and should not be used anymore.

Clean up logic to load repository config items

…ttribute and relationship (#823)

Fix typing warnings with ty

Add GraphQL fragment support in .infrahub.yml

Introduce artifact_content, file_object_content, from_json, and from_yaml Jinja2 filters to enable modular configuration pipelines where templates can reference and inline rendered content from other artifacts. Replace FilterDefinition.trusted boolean with a flag-based ExecutionContext model (CORE, WORKER, LOCAL) for context-aware template validation. Jinja2Template gains an optional client parameter and a set_client() method for deferred client injection.

Merge develop into infrahub-develop

…#904) Support all three file object retrieval endpoints: by storage ID (existing), by node UUID, and by HFID. Extract shared `ObjectStore` file fetching and content-type validation into helpers. Make `InfrahubFilters` accept an optional client, checking at call time instead of maintaining separate no-client fallback functions.

The idea is that these filters should *never* be used by the main API server. Making them `LOCAL` too would allow that. So rather than doing this, we prefer making them `WORKER` only and allow any filters to run on workers if the user turns on the proper setting in the API server.

Clean up of logic within Node

…e-cleanup Reduce cyclomatic complexity in generate_query_data_node

…-enum Use RelationshipCardinality enum in favor of a string

…to generic schemas (#807) * feat: update the generic schema on python_sdk side regarding restricted_namespaces but also missing fields IHS-190 * test: New generic test on sdk module. Loading valid generic schema through infrahubctl command layer. Infrahub API is mocked IHS-190 * test: new test on sdk module. An error message is retrieved when an invalid namespace is loaded from SDK methods. Infrahub API is mocked IHS-190 * renamed and simplified the test --------- Co-authored-by: polmichel <pol@opsmill.com>

Merge develop into infrahub-develop

Merge 'develop' into 'infrahub-develop'

…api_token When username/password are explicitly provided, clear any api_token from environment variables to allow password authentication to take precedence. This resolves the issue where users with INFRAHUB_API_TOKEN in their environment couldn't use password authentication. Co-authored-by: ogenstad <6694669+ogenstad@users.noreply.github.com>

Fix authentication validation to allow password auth to override env api_token

Merge develop into infrahub-develop

Signed-off-by: Fatih Acar <fatih@opsmill.com>

Merge develop into infrahub-develop

coderabbitai · 2026-04-14T16:08:46Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bd6fb7ce-b91d-4b05-b7c9-5c998d859d2e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

cloudflare-workers-and-pages · 2026-04-14T16:08:51Z

Deploying infrahub-sdk-python with Cloudflare Pages

Latest commit:	`e7c4ee5`
Status:	✅ Deploy successful!
Preview URL:	https://71136f30.infrahub-sdk-python.pages.dev
Branch Preview URL:	https://infrahub-develop.infrahub-sdk-python.pages.dev

View logs

codecov · 2026-04-14T16:11:11Z

Codecov Report

❌ Patch coverage is 79.34028% with 119 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
infrahub_sdk/ctl/telemetry.py	29.72%	52 Missing ⚠️
infrahub_sdk/object_store.py	49.23%	31 Missing and 2 partials ⚠️
infrahub_sdk/template/infrahub_filters.py	88.46%	7 Missing and 2 partials ⚠️
infrahub_sdk/node/node.py	82.35%	3 Missing and 3 partials ⚠️
infrahub_sdk/exceptions.py	86.66%	2 Missing and 2 partials ⚠️
infrahub_sdk/template/__init__.py	84.61%	1 Missing and 3 partials ⚠️
infrahub_sdk/spec/object.py	66.66%	0 Missing and 3 partials ⚠️
infrahub_sdk/graphql/query_renderer.py	97.70%	2 Missing ⚠️
infrahub_sdk/schema/__init__.py	0.00%	2 Missing ⚠️
infrahub_sdk/client.py	75.00%	1 Missing ⚠️
... and 3 more

@@             Coverage Diff             @@
##           develop     #948      +/-   ##
===========================================
- Coverage    82.36%   81.09%   -1.27%     
===========================================
  Files          134      134              
  Lines        13315    11306    -2009     
  Branches      2308     1693     -615     
===========================================
- Hits         10967     9169    -1798     
+ Misses        1702     1594     -108     
+ Partials       646      543     -103

Flag	Coverage Δ
integration-tests	`41.86% <24.82%> (-2.97%)`	⬇️
python-3.10	`54.01% <60.76%> (-2.56%)`	⬇️
python-3.11	`53.99% <60.76%> (-2.59%)`	⬇️
python-3.12	`53.99% <60.76%> (-2.57%)`	⬇️
python-3.13	`54.01% <60.76%> (-2.56%)`	⬇️
python-3.14	`55.60% <61.91%> (-2.82%)`	⬇️
python-filler-3.12	`22.78% <21.70%> (-0.03%)`	⬇️

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

Files with missing lines	Coverage Δ
infrahub_sdk/ctl/cli_commands.py	`72.11% <100.00%> (-2.16%)`	⬇️
infrahub_sdk/ctl/utils.py	`68.53% <100.00%> (-4.64%)`	⬇️
infrahub_sdk/ctl/validate.py	`52.11% <ø> (+2.77%)`	⬆️
infrahub_sdk/protocols.py	`100.00% <100.00%> (ø)`
infrahub_sdk/protocols_generator/generator.py	`95.00% <100.00%> (+0.45%)`	⬆️
infrahub_sdk/template/exceptions.py	`96.96% <100.00%> (ø)`
infrahub_sdk/template/filters.py	`100.00% <100.00%> (ø)`
infrahub_sdk/transfer/importer/json.py	`76.92% <100.00%> (-4.58%)`	⬇️
infrahub_sdk/client.py	`74.02% <75.00%> (-1.66%)`	⬇️
infrahub_sdk/config.py	`91.13% <96.55%> (-2.20%)`	⬇️
... and 11 more

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

ogenstad · 2026-04-14T19:04:05Z

Hm, I think I might have misunderstood you during the daily we typically never merge back infrahub-develop to develop. I.e. at the time when we release Infrahub 1.9 we should merge infrahub-develop into stable and bump the version of the SDK. What was it we wanted to do here?

gmazoyer · 2026-04-15T07:00:14Z

Hm, I think I might have misunderstood you during the daily we typically never merge back infrahub-develop to develop. I.e. at the time when we release Infrahub 1.9 we should merge infrahub-develop into stable and bump the version of the SDK. What was it we wanted to do here?

My bad, I thought we were doing infrahub-develop → develop → stable. I'll close this PR and just use the proper commit from infrahub-develop for the submodule used by infrahub for now.

ogenstad and others added 30 commits March 13, 2026 15:42

Clean up logic to load repository config items

b0b9cd1

Add tests

1ccef9b

Merge pull request #872 from opsmill/develop

6caa67b

Merge develop into infrahub-develop

Remove raise_for_error from several methods (#874)

f7266eb

This parameter has been deprecated for a while and should not be used anymore.

Merge pull request #866 from opsmill/pog-repository-loading

cce6014

Clean up logic to load repository config items

IFC-2156: Remove proposed change comments created_at and created_by a…

9871167

…ttribute and relationship (#823)

Fix typing warnings with ty

665cd24

Merge pull request #878 from opsmill/pog-fix-typing-warnings

a75a893

Fix typing warnings with ty

Add support for GraphQL fragment inlining

1a1d0dd

Merge pull request #877 from opsmill/pog-specs-infp-496

b525fef

Add GraphQL fragment support in .infrahub.yml

Add specs for artifact composition (#836)

ced2533

Add plan to implement INFP-504 (#885)

aecc708

Merge pull request #883 from opsmill/develop

5f6c7e7

Merge develop into infrahub-develop

Merge pull request #898 from opsmill/develop

8a71161

Merge develop into infrahub-develop

Merge pull request #903 from opsmill/develop

86b5bab

Merge develop into infrahub-develop

Clean up of logic within Node

dc28bad

Merge pull request #910 from opsmill/pog-node-cleanup

4704159

Clean up of logic within Node

Reduce cyclomatic complexity in generate_query_data_node

28af520

Merge pull request #912 from opsmill/pog-node-generate_query_data_nod…

fff91da

…e-cleanup Reduce cyclomatic complexity in generate_query_data_node

Use RelationshipCardinality enum in favor of a string

35cf000

Merge pull request #913 from opsmill/pog-use-relationship-cardinality…

fc0bb09

…-enum Use RelationshipCardinality enum in favor of a string

Merge pull request #914 from opsmill/develop

aa01aa6

Merge develop into infrahub-develop

Merge pull request #917 from opsmill/develop

cee0e7f

Merge develop into infrahub-develop

Merge pull request #920 from opsmill/develop

2e87b74

Merge develop into infrahub-develop

Merge pull request #926 from opsmill/develop

405b656

Merge 'develop' into 'infrahub-develop'

Copilot AI and others added 6 commits April 9, 2026 10:16

Merge pull request #928 from opsmill/pog-sdk-auth-settings

ddb1b2c

Fix authentication validation to allow password auth to override env api_token

Merge pull request #934 from opsmill/develop

0629c5a

Merge develop into infrahub-develop

feat(infrahubctl): add telemetry commands

f9249a7

Signed-off-by: Fatih Acar <fatih@opsmill.com>

Merge pull request #947 from opsmill/develop

e7c4ee5

Merge develop into infrahub-develop

Make builtin Jinja2 filters allowed on workers (#943)

44a7ccc

github-actions bot added the type/documentation Improvements or additions to documentation label Apr 14, 2026

gmazoyer closed this Apr 15, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bring `infrahub-develop` changes into `develop`#948

Bring `infrahub-develop` changes into `develop`#948
gmazoyer wants to merge 36 commits intodevelopfrom
infrahub-develop

gmazoyer commented Apr 14, 2026

Uh oh!

coderabbitai bot commented Apr 14, 2026

Review skipped

Uh oh!

cloudflare-workers-and-pages bot commented Apr 14, 2026 •

edited

Loading

Uh oh!

codecov bot commented Apr 14, 2026 •

edited

Loading

Uh oh!

ogenstad commented Apr 14, 2026

Uh oh!

gmazoyer commented Apr 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Conversation

gmazoyer commented Apr 14, 2026

Uh oh!

coderabbitai bot commented Apr 14, 2026

Review skipped

Uh oh!

cloudflare-workers-and-pages bot commented Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying infrahub-sdk-python with Cloudflare Pages

Uh oh!

codecov bot commented Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

ogenstad commented Apr 14, 2026

Uh oh!

gmazoyer commented Apr 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

cloudflare-workers-and-pages bot commented Apr 14, 2026 •

edited

Loading

codecov bot commented Apr 14, 2026 •

edited

Loading