Skip to content

Merge develop into infrahub-develop - #1315

Open
infrahub-github-bot-app[bot] wants to merge 7 commits into
infrahub-developfrom
develop
Open

Merge develop into infrahub-develop#1315
infrahub-github-bot-app[bot] wants to merge 7 commits into
infrahub-developfrom
develop

Conversation

@infrahub-github-bot-app

@infrahub-github-bot-app infrahub-github-bot-app Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Merging develop into infrahub-develop after merging pull request #1314.


Summary by cubic

Merges develop into infrahub-develop, bringing in the 1.23.1 release notes and two targeted fixes.

  • CI integration tests now wait for unit tests to pass, so the expensive huge-runner job is skipped when any unit test fails.
  • Adds the refresh_tokens relationship to CoreGenericAccount and CoreGenericAccountSync in infrahub_sdk/protocols.py, matching the Infrahub schema.
  • Converts the two pending changelog fragments into the 1.23.1 release notes and removes the fragments.

Written for commit de1ad7e. Summary will update on new commits.

Review in cubic

ogenstad and others added 7 commits August 28, 2026 09:49
Add unit-tests to the needs list of integration-tests-latest-infrahub so
the expensive huge-runner integration job is skipped when any unit-test
matrix leg fails, instead of running in parallel with failing unit tests.
ci: gate integration tests on unit-test success
Merge stable into develop
Build the 1.23.1 changelog section from the two pending towncrier
fragments and remove them.

The package version is derived from the git tag by hatch-vcs, so there is
nothing to bump in pyproject.toml or uv.lock - tagging v1.23.1 after this
merges is what sets the version.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Build the 1.23.1 changelog section from the two pending towncrier
fragments and remove them.

The package version is derived from the git tag by hatch-vcs, so there is
nothing to bump in pyproject.toml or uv.lock - tagging v1.23.1 after this
merges is what sets the version.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Infrahub declares a `refresh_tokens` relationship on CoreGenericAccount, so
that the delete of an account also removes its refresh tokens. This file is
generated from that schema, so it needs the same relationship.
Merge stable into develop
@infrahub-github-bot-app
infrahub-github-bot-app Bot requested a review from a team as a code owner September 1, 2026 20:29
@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@                 Coverage Diff                  @@
##           infrahub-develop    #1315      +/-   ##
====================================================
- Coverage             85.55%   84.24%   -1.32%     
====================================================
  Files                   148      147       -1     
  Lines                 14282    13047    -1235     
  Branches               1953     1930      -23     
====================================================
- Hits                  12219    10991    -1228     
+ Misses                 1496     1493       -3     
+ Partials                567      563       -4     
Flag Coverage Δ
integration-tests 39.09% <100.00%> (-4.91%) ⬇️
python-3.10 57.06% <100.00%> (-3.67%) ⬇️
python-3.11 57.07% <100.00%> (-3.67%) ⬇️
python-3.12 57.06% <100.00%> (-3.68%) ⬇️
python-3.13 57.06% <100.00%> (-3.68%) ⬇️
python-3.14 57.07% <100.00%> (-3.65%) ⬇️
python-filler-3.12 23.68% <0.00%> (+2.04%) ⬆️

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

Files with missing lines Coverage Δ
infrahub_sdk/protocols.py 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants