Skip to content

Dm 3801 config library python client#33

Merged
andrewchester-dm merged 4 commits into
mainfrom
DM-3801-config-library-python-client
Jul 8, 2026
Merged

Dm 3801 config library python client#33
andrewchester-dm merged 4 commits into
mainfrom
DM-3801-config-library-python-client

Conversation

@andrewchester-dm

Copy link
Copy Markdown
Collaborator
  • Add discovery config library CRUD to the Python client (DM-3801), mirroring the existing ruleset-library and discovery-config patterns.
  • New DiscoveryConfigLibraryClient mixin: list/iter, get, get-by-name (namespaced, exact), create, update, create-or-update, delete by id/name with force.
  • New DiscoveryConfigLibrary model (discovery-config fields + namespace).
  • Wraps /api/discovery/config-libraries/; no git methods (the client wraps none).
  • 17 new tests; full suite (317) green, ruff + format + mypy clean.

@andrewchester-dm andrewchester-dm marked this pull request as draft July 6, 2026 03:31
@andrewchester-dm andrewchester-dm marked this pull request as ready for review July 7, 2026 02:29
@andrewchester-dm andrewchester-dm force-pushed the DM-3801-config-library-python-client branch from ff668ab to 5f2bbee Compare July 7, 2026 03:10

@cph-datamasque cph-datamasque left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try to keep the APIs consistent with existing ones, e.g. the discovery config APIs require config_type to be explicitly specified, so DCL APIs should do the same thing.

Remove references to "archiving" and server-side code terms - those are proprietary implementation details.

Comment thread datamasque/client/models/discovery_config_library.py Outdated
Comment thread datamasque/client/discovery_config_libraries.py Outdated
Comment thread datamasque/client/discovery_config_libraries.py Outdated
Comment thread datamasque/client/discovery_config_libraries.py Outdated
Comment thread HISTORY.rst
Comment thread datamasque/client/models/discovery_config_library.py Outdated
Comment thread datamasque/client/discovery_config_libraries.py Outdated
Comment thread datamasque/client/discovery_config_libraries.py Outdated
@andrewchester-dm andrewchester-dm merged commit 02f1294 into main Jul 8, 2026
9 checks passed
@andrewchester-dm andrewchester-dm deleted the DM-3801-config-library-python-client branch July 8, 2026 07:48
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.

3 participants