fix: treat preserve-unknown-fields as a free-form map

tobru commented

2026-08-21 07:05:44 +00:00

Owner

Kubernetes marks a RawExtension - PostgreSQL's pgSettings among them - with x-kubernetes-preserve-unknown-fields rather than additionalProperties. resolve_path only understood the latter, so it reported anything below such a node as an incomplete path and the service definition admin rejected mappings like spec.parameters.service.pgSettings.archive_timeout, which is how the portal would expose PostgreSQL's point-in-time interval

tobru added 1 commit

2026-08-21 07:05:47 +00:00

fix: treat preserve-unknown-fields as a free-form map

Tests / test (push) Successful in 1m28s

Details

aad61a4b6e

Kubernetes marks a RawExtension - PostgreSQL's pgSettings among them -
with x-kubernetes-preserve-unknown-fields rather than
additionalProperties. resolve_path only understood the latter, so it
reported anything below such a node as an incomplete path and the
service definition admin rejected mappings like
spec.parameters.service.pgSettings.archive_timeout, which is how the
portal would expose PostgreSQL's point-in-time interval.

Reads and writes already resolved those paths correctly through the
data fallback; only the validation gate disagreed. Treating the marker
exactly like additionalProperties: true settles it in one place, so
both stay in step.

Both spellings are accepted because the kubernetes client's to_dict()
snake_cases the marker while raw CRD YAML does not - the same reason
get_additional_properties already checks two keys.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

tobru merged commit bb1e64074b into main

2026-08-21 07:18:36 +00:00

tobru deleted branch fix/preserve-unknown-fields-paths

2026-08-21 07:18:36 +00:00

tobru referenced this pull request from a commit

2026-08-21 07:18:37 +00:00

Merge pull request 'fix: treat preserve-unknown-fields as a free-form map' (#637) from fix/preserve-unknown-fields-paths into main

Rows
Columns

fix: treat preserve-unknown-fields as a free-form map #637