Merge pull request #46336 from ousleyp/old-redirect-notes

[virt] Removing old redirects and clarifying redirect comments

Author: ousleyp

.s2i/httpd-cfg/01-commercial.conf

@@ -283,12 +283,10 @@ AddType text/vtt                            vtt
     RewriteRule ^container-platform/(4\.4|4\.5)/authentication/certificates/(.*)$ /container-platform/$1/security/certificates/$2 [NE,R=301]
     RewriteRule ^container-platform/(4\.4|4\.5)/authentication/(allowing-javascript-access-api-server|encrypting-etcd|certificate-types-descriptions)\.html$ /container-platform/$1/security/$2\.html [NE,R=301]
 
-    # CNV redirects to OpenShift Virtualization from 4.5 GA onwards
-    # NOTE THAT THESE ARE A 302 REDIRECT, not a 301. Replace with 301 at CNV GA
-    # send all cnv/ to virt/about-virt.html.
-    # send all virt/ to virt/about-virt.html excluding about-virt.html otherwise there is an infinite loop
+    # OpenShift Virtualization (CNV) catchall redirect; use when CNV releases asynchronously from OCP. Do not change the 302 to a 301.
+    # When uncommented, this redirects all `virt` directory traffic to the about-virt page.
+    # To activate the redirects, uncomment the next line and update the version number to the pending release.
     # RewriteRule container-platform/4\.10/virt/(?!about-virt\.html)(.+)$ /container-platform/4.10/virt/about-virt.html [NE,R=302]
-    # RewriteRule container-platform/4\.5/cnv/?(.*)$ /container-platform/4.5/virt/about-virt.html [NE,R=301]
 
     # Serverless Redirects
     RewriteRule container-platform/(4\.5|4\.6)/serverless/knative_eventing/(serverless-subscriptions\.html|serverless-channels\.html|serverless-subscriptions\.html) /container-platform/$1/serverless/event_workflows/serverless-channels.html [NE,R=301]