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

Utvide validate med flere CRD-er vi støtter i clusteret #353

Open
thokra-nav opened this issue Mar 6, 2024 · 1 comment
Open

Utvide validate med flere CRD-er vi støtter i clusteret #353

thokra-nav opened this issue Mar 6, 2024 · 1 comment

Comments

@thokra-nav
Copy link
Contributor

thokra-nav commented Mar 6, 2024

I dag bruker Validate https://storage.googleapis.com/nais-json-schema-2c91/nais-all.json for å hente inn schema å validere kubernetes ressurser mot.
Denne inneholder CRD-er for NAIS. Men siden vi oppfordrer å lage flere ressurser, som f.eks. PrometheusRule bør vi kanskje støtte alle ressurser som vi dokumenterer i doc-en?

@thokra-nav
Copy link
Contributor Author

thokra-nav commented Mar 6, 2024

@tronghn nevnte på Zoom:
Vi bør kanskje også ta en syntax-validering av YAML-en før spec-validering, for å få bedre feilmeldinger dersom det er syntax feil

Edit: denne kan kanskje løses ved å bytte ut yaml-parseren (som i dag er https:/ghodss/yaml for parity med nais/deploy) med https:/goccy/go-yaml e.l.

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

No branches or pull requests

1 participant