Repository navigation
feat: add the WORKER_TIMEOUT error code to the generated catalogue - #1405
Merged
ogenstad merged 2 commits intoOct 9, 2026
Merged
Conversation
Deploying infrahub-sdk-python with
|
| Latest commit: |
aed8916
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://58f0beee.infrahub-sdk-python.pages.dev |
| Branch Preview URL: | https://pog-worker-timeout-error-bin.infrahub-sdk-python.pages.dev |
Regenerated from the server's error catalogue, which now defines WORKER_TIMEOUT for a worker call that exceeds its timeout. Adds WorkerTimeoutData and WorkerTimeoutError, registers the code in the catalogue mappings, exports WorkerTimeoutError from infrahub_sdk.exceptions, and adds the code to the per-code tests.
ogenstad
force-pushed
the
pog-worker-timeout-error-binding-ifc-3148
branch
from
October 7, 2026 10:25
c3c7a9c to
62f255a
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## infrahub-develop #1405 +/- ##
====================================================
- Coverage 87.14% 86.81% -0.33%
====================================================
Files 153 152 -1
Lines 16493 14763 -1730
Branches 2417 1982 -435
====================================================
- Hits 14372 12817 -1555
+ Misses 1493 1382 -111
+ Partials 628 564 -64
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 16 files with indirect coverage changes 🚀 New features to boost your workflow:
|
ogenstad
added a commit
to opsmill/infrahub
that referenced
this pull request
Oct 7, 2026
Moves the submodule to opsmill/infrahub-sdk-python#1405 so the generated SDK error catalogue includes WORKER_TIMEOUT.
3 of 5 tasks
ogenstad
marked this pull request as ready for review
October 9, 2026 11:22
gmazoyer
approved these changes
Oct 9, 2026
ogenstad
added a commit
to opsmill/infrahub
that referenced
this pull request
Oct 9, 2026
Points the submodule at the merge commit of opsmill/infrahub-sdk-python#1405 so the epic branch tracks the same SDK history as develop.
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.
Why
The Infrahub server now returns a
WORKER_TIMEOUTerror code when a call to a worker runs past its timeout (opsmill/infrahub#10668, part of opsmill/infrahub#10530). The SDK's generated error catalogue does not have this code yet, so the Infrahubbackend-validate-generatedCI job fails on opsmill/infrahub#10530 withpython_sdk/infrahub_sdk/exceptions/catalogue.py is out of date with schema/error-catalogue.json.What changed
infrahub_sdk/exceptions/catalogue.pyfrom the server's error catalogue withuv run invoke backend.generate.WorkerTimeoutDataandWorkerTimeoutError, and registeredWORKER_TIMEOUTinCODE_TO_EXCEPTION,CODE_TO_DATA_MODELand the builder mapping.WorkerTimeoutErrorfrominfrahub_sdk.exceptionsand added it to thepublic_names.jsonsnapshot.WORKER_TIMEOUTcase to the per-code tests intests/unit/sdk/test_error_catalogue.py, with a captured response intests/fixtures/error_catalogue/codes/worker_timeout.json.How to review
catalogue.pyis generated. Review it against theWORKER_TIMEOUTentry inschema/error-catalogue.jsonon opsmill/infrahub#10530. The fixture uses the payload and message the server builds for a connectivity check that times out.How to test
Once the Infrahub PR points its
python_sdksubmodule at this branch, itsbackend-validate-generatedjob should pass.Checklist
+typed-per-code-exceptions.added.mdalready covers it