Open pjbgf opened 2 years ago
Hey @pjbgf I would like work on this. I am not sure about couple of points, can you please clarify:
ValueKey
and TargetPath
fields only?@Santosh1176 here's more info:
+kubebuilder:validation
and then tested with testenv. We need to review each CRD for all controllers, and make sure that all fields are validated.+required
, they lack max length and pattern validation. In some CRD we may be using other types instead of them to refer to Kubernetes objects, when that's the case we must change that.@pjbgf Any reference/guidance for max and min length properties for strings fields?
@Santosh1176 unfortunately this will be on a case-by-case basis. Feel free to propose some validation for one of the CRDs and during the review we will provide you with some feedback.
@Santosh1176 I am interested contributing to this issue as well, have you made any progress since?
What about I take one of those task item? for example, the notification controller?
Cheers.
In the run-up to maturing flux APIs, we should review all CRD validation and ensure that they align with expected input.
Controllers:
Additional points to consider:
.Spec
fields should be marked as required across all CRDs.