chore: Updates auto-generated internal resources - #4736
svc-apix-Bot wants to merge 1 commit into
Conversation
🤖 Augment PR SummarySummary: Regenerates internal, currently unregistered API resources from a newer Atlas API specification. Changes:
🤖 Was this summary useful? React with 👍 or 👎 |
| - `6`: Friday. | ||
| - `7`: Saturday. | ||
| computed_optional_required: required | ||
| computed_optional_required: optional |
There was a problem hiding this comment.
tools/codegen/models/maintenance_window_api.yaml:26: Making only day_of_week optional prevents the new wave-only maintenance-window use case: hour_of_day remains required, so Terraform cannot configure only group_id and wave_assignment. The existing mongodbatlas_maintenance_window deliberately treats both schedule fields as optional and covers wave-only configuration.
Severity: medium
Other Locations
internal/serviceapi/maintenancewindowapi/resource_schema.go:24
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| tf_schema_name: wave_assignment | ||
| tf_model_name: WaveAssignment | ||
| api_name: waveAssignment | ||
| req_body_usage: all_request_bodies |
There was a problem hiding this comment.
tools/codegen/models/maintenance_window_api.yaml:151: waveAssignment is nullable and documented as cleared by sending null, but this request-body mode causes the generated marshaller to omit a null value on update. Removing wave_assignment will PATCH without that member, retain the remote assignment, and leave perpetual drift.
Severity: medium
Other Locations
internal/serviceapi/maintenancewindowapi/resource_schema.go:85
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Automatic update of auto-generated resources which are not registered in our provider.
Safe to merge after checks pass.