Closed houshengbo closed 3 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: houshengbo
The full list of commands accepted by this bot can be found here.
The pull request process is described here
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 63.60%. Comparing base (
c85f8f3
) to head (a535d0b
). Report is 2 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/test eventing-upgrade-tests
/test eventing-upgrade-tests
/hold
why?
We're only reconciling those resources when the feature flag is enabled but if we remove the resources TLS won't be supported by the operator https://github.com/knative/operator/blob/918cb7ed99fe042e733a900a432edfc8585ac268/pkg/reconciler/knativeeventing/eventing_tls.go#L37
@pierDipi By default, the feature flag seems to be on, so the reconcile loop of operator runs into
ClusterIssuer” in version “[cert-manager.io/v1](http://cert-manager.io/v1)`
However, if folks do not know cert-manager is the dependency, they would think it is a bug on operator. We do not have document from the operator's perspective how what to install and how to configure.
Per the discussion here: https://cloud-native.slack.com/archives/C04LMU33V1S/p1710250853335799, we need a better support of this TLS feature in later release. But for now, what is your suggestion to fix?
That's the problem, it's shouldn't be on by default
Proposed Changes