docs: improve old changelog entry (#5078)

docs: Change patch to being commented-out by default.

This is to match scaffolding code.

Fixes #5073

Signed-off-by: Marcin Owsiany <[email protected]>

Author: porridge

website/content/en/docs/upgrading-sdk-version/v1.6.0.md

@@ -122,24 +122,26 @@ _See [#4655](https://github.com/operator-framework/operator-sdk/pull/4655) for m
 OLM does [not yet support cert-manager](https://olm.operatorframework.io/docs/advanced-tasks/adding-admission-and-conversion-webhooks/#certificate-authority-requirements), so a JSON patch was added to remove this volume and mount such that OLM can itself create and manage certs for your Operator.
 In `config/manifests/kustomization.yaml`, add the following:
 ```yaml
-patchesJson6902: 
-- target:
-    group: apps
-    version: v1
-    kind: Deployment
-    name: controller-manager
-    namespace: system
-  patch: |-
-    # Remove the manager container's "cert" volumeMount, since OLM will create and mount a set of certs.
-    # Update the indices in this path if adding or removing containers/volumeMounts in the manager's Deployment.
-    - op: remove
-      path: /spec/template/spec/containers/1/volumeMounts/0
-    # Remove the "cert" volume, since OLM will create and mount a set of certs.
-    # Update the indices in this path if adding or removing volumes in the manager's Deployment.
-    - op: remove
-      path: /spec/template/spec/volumes/0
+#patchesJson6902:
+#- target:
+#    group: apps
+#    version: v1
+#    kind: Deployment
+#    name: controller-manager
+#    namespace: system
+#  patch: |-
+#    # Remove the manager container's "cert" volumeMount, since OLM will create and mount a set of certs.
+#    # Update the indices in this path if adding or removing containers/volumeMounts in the manager's Deployment.
+#    - op: remove
+#      path: /spec/template/spec/containers/1/volumeMounts/0
+#    # Remove the "cert" volume, since OLM will create and mount a set of certs.
+#    # Update the indices in this path if adding or removing volumes in the manager's Deployment.
+#    - op: remove
+#      path: /spec/template/spec/volumes/0
 ```
 
+If you have configured your operator to use webhooks, add this YAML block uncommented.
+
 _See [#4623](https://github.com/operator-framework/operator-sdk/pull/4623) for more details._
 
 ## (go/v2, go/v3, ansible/v1, helm/v1) Add scheme, token, and TLS config to the Prometheus `ServiceMonitor` metrics endpoint.
@@ -217,7 +219,8 @@ _See [#4406](https://github.com/operator-framework/operator-sdk/pull/4406) for m
 
 ## (ansible/v1, helm/v1) Add the `controller-manager` ServiceAccount to your project.
 
-A non-default ServiceAccount `controller-manager` is scaffolded on `operator-sdk init`, to improve security for operators installed in shared namespaces. To add this ServiceAccount to your project, do the following: ```sh
+A non-default ServiceAccount `controller-manager` is scaffolded on `operator-sdk init`, to improve security for operators installed in shared namespaces. To add this ServiceAccount to your project, do the following:
+```sh
 # Create the ServiceAccount.
 cat <<EOF > config/rbac/service_account.yaml apiVersion: v1 
 kind: ServiceAccount

website/content/en/docs/upgrading-sdk-version/v1.6.1.md

@@ -122,24 +122,26 @@ _See [#4655](https://github.com/operator-framework/operator-sdk/pull/4655) for m
 OLM does [not yet support cert-manager](https://olm.operatorframework.io/docs/advanced-tasks/adding-admission-and-conversion-webhooks/#certificate-authority-requirements), so a JSON patch was added to remove this volume and mount such that OLM can itself create and manage certs for your Operator.
 In `config/manifests/kustomization.yaml`, add the following:
 ```yaml
-patchesJson6902: 
-- target:
-    group: apps
-    version: v1
-    kind: Deployment
-    name: controller-manager
-    namespace: system
-  patch: |-
-    # Remove the manager container's "cert" volumeMount, since OLM will create and mount a set of certs.
-    # Update the indices in this path if adding or removing containers/volumeMounts in the manager's Deployment.
-    - op: remove
-      path: /spec/template/spec/containers/1/volumeMounts/0
-    # Remove the "cert" volume, since OLM will create and mount a set of certs.
-    # Update the indices in this path if adding or removing volumes in the manager's Deployment.
-    - op: remove
-      path: /spec/template/spec/volumes/0
+#patchesJson6902:
+#- target:
+#    group: apps
+#    version: v1
+#    kind: Deployment
+#    name: controller-manager
+#    namespace: system
+#  patch: |-
+#    # Remove the manager container's "cert" volumeMount, since OLM will create and mount a set of certs.
+#    # Update the indices in this path if adding or removing containers/volumeMounts in the manager's Deployment.
+#    - op: remove
+#      path: /spec/template/spec/containers/1/volumeMounts/0
+#    # Remove the "cert" volume, since OLM will create and mount a set of certs.
+#    # Update the indices in this path if adding or removing volumes in the manager's Deployment.
+#    - op: remove
+#      path: /spec/template/spec/volumes/0
 ```
 
+If you have configured your operator to use webhooks, add this YAML block uncommented.
+
 _See [#4623](https://github.com/operator-framework/operator-sdk/pull/4623) for more details._
 
 ## (go/v2, go/v3, ansible/v1, helm/v1) Add scheme, token, and TLS config to the Prometheus `ServiceMonitor` metrics endpoint.
@@ -217,7 +219,8 @@ _See [#4406](https://github.com/operator-framework/operator-sdk/pull/4406) for m
 
 ## (ansible/v1, helm/v1) Add the `controller-manager` ServiceAccount to your project.
 
-A non-default ServiceAccount `controller-manager` is scaffolded on `operator-sdk init`, to improve security for operators installed in shared namespaces. To add this ServiceAccount to your project, do the following: ```sh
+A non-default ServiceAccount `controller-manager` is scaffolded on `operator-sdk init`, to improve security for operators installed in shared namespaces. To add this ServiceAccount to your project, do the following:
+```sh
 # Create the ServiceAccount.
 cat <<EOF > config/rbac/service_account.yaml apiVersion: v1 
 kind: ServiceAccount
@@ -277,4 +280,4 @@ The changes should look like:
 +  namespace: system
 ```
 
-_See [#4653](https://github.com/operator-framework/operator-sdk/pull/4653) for more details._
+_See [#4653](https://github.com/operator-framework/operator-sdk/pull/4653) for more details._