Skip to content

[DTC] Client changes to set Weight as an Optional field for Policy - #34

Merged
unasra merged 1 commit into
infobloxopen:mainfrom
Tejashree-RS:dtc-policy-weight
Jun 9, 2026
Merged

[DTC] Client changes to set Weight as an Optional field for Policy #34
unasra merged 1 commit into
infobloxopen:mainfrom
Tejashree-RS:dtc-policy-weight

Conversation

@Tejashree-RS

@Tejashree-RS Tejashree-RS commented May 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Sets weight as an optional field on the PolicyPool model to align with the upstream API schema, where weight is not required when associating a pool to a DTC policy.

Type of Change

  • 🆕 New API / Model
  • 🔧 Update Existing Schema or Model
  • 🐛 Bug Fix
  • 📝 Documentation
  • 🧹 Maintenance (dependency upgrades, CI, refactoring)

Affected Packages

  • anycast
  • cloud_discovery
  • dfp
  • dns_config
  • dns_data
  • dtc
  • fw
  • infra_mgmt
  • infra_provision
  • ipam
  • ipam_federation
  • keys
  • redirect
  • upgrade_policy
  • universal_ddi_client

Resource / Model: PolicyPool
Description: Changed weight on PolicyPool from required to optional (default None).

Schema Changes

  • New model added
  • New field(s) added to existing model
  • Field(s) removed or renamed
  • Required fields changed
Detailed Changes (click to expand)
File Change
src/dtc/models/policy_pool.py Changed weight field to Optional[int] = None
src/dtc/docs/PolicyPool.md Updated weight field documentation to mark as optional

Ticket / Issue

Fixes #

Changelog Inclusion

  • Consider for changelog and release notes addition

Additional Context

Without this fix, creating a PolicyPool without explicitly specifying a weight would raise a validation error, even though the upstream API treats it as optional.

@unasra unasra changed the title Updated schema for DTC Policy [DTC] Client changes to set Weight as an Optional field for Policy Jun 9, 2026
@unasra
unasra merged commit b4dc2b2 into infobloxopen:main Jun 9, 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.

2 participants