Open stevenhorsman opened 1 year ago
@stevenhorsman right, i don't think it's a deliberate decision to exclude the webhook from the e2e framework but just something we'll need add in later time, I think we talked about this some weeks ago on the weekly meeting and decision is that we should do that. Might talk about this again on the meeting tomorrow?
Sure - I just wanted to check that this issue was valid and I'd not missed/forgotten a reason that it should never go into the e2e provisioning. Thanks!
So the bit that I missed was that the webhook is planned to be managed by the peer-pod-config controller, but in the short-term we can work on the webhook install through e2e whilst that is looked at in parallell
@bpradipt @stevenhorsman mentioned you might be looking at including the webhook/cert-manager into the peer-pod config, is that correct?
I've been using the provisioning code (which is great), but have realised that it doesn't install the webhook, or cert-manager, so I think we should fix this and then probably try and add some tests for the mutation of the resource (in a separate issue).
e.g.
@huoqifeng - I'm assuming this wasn't a deliberate decision to exclude the webhook from the e2e framework, but just something we haven't got around to adding yet?