Tim Ramlot

Results 302 comments of Tim Ramlot

BLOCKED: The generated json-schema code contains a lot of `allOf` directives, which do not play well with `additionalProperties: false`.

1) & 2) I agree that ideally cert-manager should have a json-schema added to its chart. I fear however, that it will be very hard to manage such a schema...

blocked on improvements to the dependencies. (I'm working on `helm-jsonschema-gen` and `tem` improvements)

@sethiyash What additional maintenance would this addition require? Is it possible to also test this generated Carvel Package?

@sethiyash I discussed this with the cert-manager maintainers team and we concluded that releasing this Carvel package should not be part of `cert-manager/cert-manager`. We propose that you create your own...

/approve the ports field is purely informational, but we should still inform the user about the `healthzPort`

@EduardoVV Thank you for the information. I updated the PR to target the structured-errors branch. This PR now also includes some other possible improvements for that branch. Due to this...