No option to install net-certmanager-controller

[gecube]

Hello!

I'd like to completely manage the Knative installation with the operator. It is easy and nice. But in fact I didn't find any option to install net-certmanager-controller component of Knative with the operator. It looks like that this component may be installed only with optional manifest https://github.com/knative/net-certmanager/releases/download/knative-v1.2.0/release.yaml
It makes me very sad, because I need to use two completely different ways to manage the configuration of Knative.
Please add the net-certmanager-controller to operator.

[houshengbo]

@gecube net-certmanager is not the core of knative, so far it is installed by the additional manifests.

[gecube]

It is sad to hear it :-( I thought it will be relatively ease to make a switch (toggle) in operator to install or not this component.

[houshengbo]

So far to use the net-cert with the operator, you have to use the CR as follows:

apiVersion: v1
kind: Namespace
metadata:
 name: knative-serving
---
apiVersion: operator.knative.dev/v1alpha1
kind: KnativeServing
metadata:
  name: knative-serving
  namespace: knative-serving
spec:
  version: "1.2"
  additionalManifests:
    - URL: https://github.com/knative-sandbox/net-certmanager/releases/download/knative-v1.2.0/net-certmanager.yaml

Operator cannot cover everything with the spec. That was why we enabled custom manifests as an option to install literally anything with spec.additionalManifests.

[github-actions]

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

[dprotaso]

/lifecycle frozen

[dprotaso]

Similar to how we install our net-istio and net-contour it would be nice if the operator provided facilities to install net-certmanager

[ReToCode]

This is no longer needed, as net-certmanager is now integrated into Serving.

/close

[knative-prow]

@ReToCode: Closing this issue.

In response to this:

This is no longer needed, as net-certmanager is now integrated into Serving.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.