Skip to content
This repository has been archived by the owner on Oct 20, 2023. It is now read-only.

Add schema validation for CRDs #51

Merged
merged 3 commits into from
Mar 5, 2020
Merged

Add schema validation for CRDs #51

merged 3 commits into from
Mar 5, 2020

Conversation

stefanprodan
Copy link
Contributor

@stefanprodan stefanprodan commented Mar 5, 2020 •
edited
Loading

Changes:

  • Add validation for TrafficTargets and HTTPRouteGroups
  • Fix validation for TrafficSplits (the top spec field is missing from SMI 😱)

Fix: #38

@stefanprodan
Add validation schema for TrafficTargets
48f2604 
Signed-off-by: stefanprodan <stefan.prodan@gmail.com>
@stefanprodan
Add validation schema for HTTPRouteGroups
164c8f4 
Signed-off-by: stefanprodan <stefan.prodan@gmail.com>
@stefanprodan
Fix validation schema for TrafficSplits
3e7f2bf 
The top spec field is missing from SMI
Remove empty metrics yaml

Signed-off-by: stefanprodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan mentioned this pull request Mar 5, 2020
Closed
@grampelberg grampelberg merged commit 35a148d into master Mar 5, 2020
@stefanprodan stefanprodan deleted the validation branch March 5, 2020 15:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@grampelberg grampelberg grampelberg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CRD Schema Validation
2 participants
@stefanprodan @grampelberg