Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: parse expressions in x-kubernetes-preserve-unknown fields #175

Merged
merged 1 commit into from
Dec 29, 2024

Conversation

a-hilaly
Copy link
Member

Fixes #162
Merge after #174

Previously, fields marked with x-kubernetes-preserve-unknown-fields were skipped
during parsing. Now these fields are parsed to extract expressions, allowing for
variable substitution in schemaless fields while still preserving unknown fields
as specified by the k8s-open-api extension.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@a-hilaly a-hilaly force-pushed the nested-schemaless-resources branch from c690500 to 929a161 Compare December 23, 2024 20:32
Copy link
Contributor

@michaelhtm michaelhtm left a comment

Choose a reason for hiding this comment

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

Nice feature @a-hilaly 🔥
left one nit

pkg/graph/parser/parser.go Outdated Show resolved Hide resolved
Previously, fields marked with `x-kubernetes-preserve-unknown-fields` were skipped
during parsing. Now these fields are parsed to extract expressions, allowing for
variable substitution in schemaless fields while still preserving unknown fields
as specified by the k8s-open-api extension.
@a-hilaly a-hilaly force-pushed the nested-schemaless-resources branch from 929a161 to 8ff3c8d Compare December 29, 2024 05:25
@michaelhtm michaelhtm merged commit 2c58d17 into awslabs:main Dec 29, 2024
4 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.

Schemaless properties not supported in CEL template
2 participants