Skip to content

Update Cloud Discovery Schemas to add Zone filters and Other Correspondent Field Changes#51

Merged
unasra merged 4 commits into
infobloxopen:mainfrom
AnilGadiyarHJ:clouddis2
Jul 16, 2026
Merged

Update Cloud Discovery Schemas to add Zone filters and Other Correspondent Field Changes#51
unasra merged 4 commits into
infobloxopen:mainfrom
AnilGadiyarHJ:clouddis2

Conversation

@AnilGadiyarHJ

Copy link
Copy Markdown
Contributor

No description provided.

@AnilGadiyarHJ AnilGadiyarHJ changed the title cloud discovery Update Cloud Discovery Schemas to add Zone filters and Other Correspondent Field Changes Jul 16, 2026
@unasra
unasra requested a review from Copilot July 16, 2026 11:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Cloud Discovery client schema/models to support new DNS zone filtering capabilities and additional provider/config fields, keeping the generated Python models and markdown docs in sync.

Changes:

  • Adds the new ZoneFilter model and documentation, and wires it into DNSConfig via the new zone_filters field.
  • Extends DNSConfig with resolver_endpoints_sync_enabled.
  • Updates discovery/source configuration schemas (e.g., DiscoveryConfig.is_disabled, DiscoveryConfig.labs_provider, and marks SourceConfig.id as read-only/excluded from to_dict).

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/cloud_discovery/README.md Adds ZoneFilter to the documented model list.
src/cloud_discovery/models/zone_filter.py Introduces the ZoneFilter Pydantic model.
src/cloud_discovery/models/source_config.py Excludes id from to_dict (read-only) and updates docstring.
src/cloud_discovery/models/dns_config.py Adds resolver endpoint sync flag and zone_filters: List[ZoneFilter] with (de)serialization.
src/cloud_discovery/models/discovery_config.py Adds is_disabled and labs_provider fields to the model.
src/cloud_discovery/models/init.py Exports ZoneFilter from the models package.
src/cloud_discovery/docs/ZoneFilter.md Adds markdown documentation for the new ZoneFilter model.
src/cloud_discovery/docs/SourceConfig.md Marks id as [readonly] in docs.
src/cloud_discovery/docs/DNSConfig.md Documents new DNS config fields (resolver_endpoints_sync_enabled, zone_filters).
src/cloud_discovery/docs/DiscoveryConfig.md Documents new discovery config fields (is_disabled, labs_provider).
src/cloud_discovery/init.py Exports ZoneFilter at the package level.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/cloud_discovery/models/zone_filter.py
Comment thread src/cloud_discovery/docs/ZoneFilter.md
Comment thread src/cloud_discovery/models/source_config.py
@unasra
unasra merged commit f5ef821 into infobloxopen:main Jul 16, 2026
5 checks passed
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