Skip to content

Add support for discovery configurations#21

Merged
cph-datamasque merged 26 commits into
mainfrom
epic/discovery-config
Jun 24, 2026
Merged

Add support for discovery configurations#21
cph-datamasque merged 26 commits into
mainfrom
epic/discovery-config

Conversation

@cph-datamasque

@cph-datamasque cph-datamasque commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator
  • New models and APIs for discovery configuration
  • Fix file include/skip specification
  • Various other misc fixes
  • Add testpypi release workflow

- Add `discovery_config` field to `SchemaDiscoveryRequest`
- Add `FileDataDiscoveryRequest` + `FileDataDiscoveryOptions` models
- Add `start_file_data_discovery_run()` for the file-data-discovery trigger
@cph-datamasque cph-datamasque force-pushed the epic/discovery-config branch from 3b65b2b to 11ff50e Compare June 23, 2026 04:48
Comment thread datamasque/client/models/discovery.py Outdated
Comment thread datamasque/client/discovery.py Outdated
Comment thread datamasque/client/exceptions.py Outdated
Comment thread HISTORY.rst Outdated
Comment thread HISTORY.rst Outdated
Comment thread datamasque/client/discovery_configs.py
Comment thread datamasque/client/discovery.py
Comment thread datamasque/client/models/discovery.py Outdated
Comment thread datamasque/client/models/discovery.py Outdated
Comment thread HISTORY.rst Outdated
@cph-datamasque cph-datamasque force-pushed the epic/discovery-config branch from 11ff50e to 72c27a4 Compare June 24, 2026 01:36
cph-datamasque and others added 21 commits June 24, 2026 14:20
The server returns a 400 keyed under `discovery_config`
when the referenced config is missing, archived, or not in `valid` validation state.
Surface that as a specific exception subclass.
…gration

- Add `is_user_subscribed` to `MaskingRunRequest` (subscribe the requesting user to run notifications).
- Add `auto_pull` / `auto_pull_branch` to `MaskingRunOptions` (refresh the ruleset from git before a run).
- Surface `validation_error` (and `validation_error_type` for rulesets) on `Ruleset` and `RulesetLibrary`.
- Expose git provenance as a nested `git` (`GitSnapshot`) field on both, collapsed from the flat `git_*` fields.
- Exclude read-only fields (`id`, `is_valid`, `validation_error`, `git`, ...) from create/update request bodies.
- Fix `SslZipFile` uploads to send the required `database_type=mysql` form field.
Names are unique only per type, so the by-name lookups need the type to identify a single config.
Copy `is_valid`/`validation_error` back from the server response, matching the ruleset-library client.
Require the fields the server always returns; type `connection` as `RunConnectionRef`.
@cph-datamasque cph-datamasque force-pushed the epic/discovery-config branch from a171cd6 to edfa6e0 Compare June 24, 2026 02:21
@cph-datamasque cph-datamasque merged commit d85ba66 into main Jun 24, 2026
9 checks passed
@cph-datamasque cph-datamasque deleted the epic/discovery-config branch June 24, 2026 03:08
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