Bug 1947797

Summary: CVO: check (see bug 1947801#c4 steps) audit log to find deprecated API access related to this component to ensure this component won't access APIs that trigger APIRemovedInNextReleaseInUse alert
Product: OpenShift Container Platform Reporter: Stefan Schimanski <sttts>
Component: Cluster Version OperatorAssignee: Lalatendu Mohanty <lmohanty>
Status: CLOSED ERRATA QA Contact: Yang Yang <yanyang>
Severity: high Docs Contact:
Priority: high    
Version: 4.8CC: alegrand, anpicker, aos-bugs, erooth, hongyli, jack.ottofaro, jokerman, juzhao, kakkoyun, kewang, lcosic, lmohanty, mfojtik, pkrupa, surbania, wking, xxia, yanyang
Target Milestone: ---Keywords: Reopened
Target Release: 4.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: 1947719 Environment:
Last Closed: 2021-07-27 22:58:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1947719    

Description Stefan Schimanski 2021-04-09 09:37:05 UTC
This component accesses APIs that will be removed in 4.9 (Kubernetes 1.22). It is causing the DeprecatedAPIInUse alert to fire in every 4.8 clusters permanently and hence must be fixed in 4.8 (blocker+).

The raw audit data can be found at https://gist.github.com/sttts/50a1429837f2448ce07f30174fa73cdb.

Here are the observed requests for this component:

system:serviceaccount:openshift-cluster-version:default: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/clusteroperators.config.openshift.io
system:serviceaccount:openshift-cluster-version:default: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/clusterversions.config.openshift.io

+++ This bug was initially created as a clone of Bug #1947719 +++

Created attachment 1770482 [details]
alert screen shot

Created attachment 1770482 [details]
alert screen shot

Description of problem:
8 DeprecatedAPIInUse info alerts display

Version-Release number of selected component (if applicable):
4.8.0-0.nightly-2021-04-08-200632

How reproducible:
always

Steps to Reproduce:
1. open console-monitoring-alerts
2.
3.

Actual results:
8 DeprecatedAPIInUse info alerts display

Expected results:
No other alerts display except watchdog

Additional info:

alert rule metrics:
group by(group, version, resource) (apiserver_requested_deprecated_apis{removed_release="1.22"}) and (sum by(group, version, resource) (rate(apiserver_request_total[10m]))) > 0

Element	Value:
{group="rbac.authorization.k8s.io",resource="roles",version="v1beta1"}	1
{group="admissionregistration.k8s.io",resource="mutatingwebhookconfigurations",version="v1beta1"}	1
{group="admissionregistration.k8s.io",resource="validatingwebhookconfigurations",version="v1beta1"}	1
{group="apiextensions.k8s.io",resource="customresourcedefinitions",version="v1beta1"}	1
{group="certificates.k8s.io",resource="certificatesigningrequests",version="v1beta1"}	1
{group="extensions",resource="ingresses",version="v1beta1"}	1
{group="rbac.authorization.k8s.io",resource="clusterrolebindings",version="v1beta1"}	1
{group="rbac.authorization.k8s.io",resource="rolebindings",version="v1beta1"}	1

----------------
# for i in roles mutatingwebhookconfigurations validatingwebhookconfigurations customresourcedefinitions certificatesigningrequests ingresses clusterrolebindings rolebindings; do oc api-resources | grep $i; echo -e "\n"; done
clusterroles                                           authorization.openshift.io/v1                 false        ClusterRole
roles                                                  authorization.openshift.io/v1                 true         Role
clusterroles                                           rbac.authorization.k8s.io/v1                  false        ClusterRole
roles                                                  rbac.authorization.k8s.io/v1                  true         Role
mutatingwebhookconfigurations                          admissionregistration.k8s.io/v1               false        MutatingWebhookConfiguration
validatingwebhookconfigurations                        admissionregistration.k8s.io/v1               false        ValidatingWebhookConfiguration
customresourcedefinitions             crd,crds         apiextensions.k8s.io/v1                       false        CustomResourceDefinition
certificatesigningrequests            csr              certificates.k8s.io/v1                        false        CertificateSigningRequest
ingresses                                              config.openshift.io/v1                        false        Ingress
ingresses                             ing              extensions/v1beta1                            true         Ingress
ingresses                             ing              networking.k8s.io/v1                          true         Ingress
clusterrolebindings                                    authorization.openshift.io/v1                 false        ClusterRoleBinding
clusterrolebindings                                    rbac.authorization.k8s.io/v1                  false        ClusterRoleBinding
clusterrolebindings                                    authorization.openshift.io/v1                 false        ClusterRoleBinding
rolebindings                                           authorization.openshift.io/v1                 true         RoleBinding
clusterrolebindings                                    rbac.authorization.k8s.io/v1                  false        ClusterRoleBinding
rolebindings                                           rbac.authorization.k8s.io/v1                  true         RoleBinding

--- Additional comment from Junqi Zhao on 2021-04-09 05:28:56 CEST ---

alert details
alert:DeprecatedAPIInUse
expr:group by(group, version, resource) (apiserver_requested_deprecated_apis{removed_release="1.22"}) and (sum by(group, version, resource) (rate(apiserver_request_total[10m]))) > 0
for: 1h
labels:
  severity: info
annotations:
  message: Deprecated API that will be removed in the next version is being used. Removing the workload that is using the {{"{{$labels.group}}"}}.{{"{{$labels.version}}"}}/{{"{{$labels.resource}}"}} API might be necessary for a successful upgrade to the next cluster version. Refer to the audit logs to identify the workload.

--- Additional comment from hongyan li on 2021-04-09 05:37:17 CEST ---



--- Additional comment from hongyan li on 2021-04-09 05:44:46 CEST ---

Different issue from bug 1932165 which is about variable not translated to value

--- Additional comment from Junqi Zhao on 2021-04-09 06:04:30 CEST ---

# oc version
Client Version: 4.8.0-0.nightly-2021-04-08-200632
Server Version: 4.8.0-0.nightly-2021-04-08-200632
Kubernetes Version: v1.21.0-rc.0+6d27558

checked from prometheus, query parameter:
count(apiserver_requested_deprecated_apis{removed_release="1.22"}) by(instance,version,group,resource)
version is v1beta1
{group="certificates.k8s.io", instance="10.0.160.188:6443", resource="certificatesigningrequests", version="v1beta1"} 1
{group="extensions", instance="10.0.160.188:6443", resource="ingresses", version="v1beta1"} 1
{group="rbac.authorization.k8s.io", instance="10.0.160.188:6443", resource="clusterrolebindings", version="v1beta1"} 1
{group="rbac.authorization.k8s.io", instance="10.0.160.188:6443", resource="rolebindings", version="v1beta1"} 1
{group="rbac.authorization.k8s.io", instance="10.0.160.188:6443", resource="roles", version="v1beta1"} 1
{group="admissionregistration.k8s.io", instance="10.0.160.188:6443", resource="mutatingwebhookconfigurations", version="v1beta1"} 1
{group="admissionregistration.k8s.io", instance="10.0.160.188:6443", resource="validatingwebhookconfigurations", version="v1beta1"} 1
{group="apiextensions.k8s.io", instance="10.0.160.188:6443", resource="customresourcedefinitions", version="v1beta1"} 1

but the api versions are all actually v1, which means apiserver_requested_deprecated_apis may post the wrong result
# for i in certificatesigningrequests ingresses clusterrolebindings rolebindings roles mutatingwebhookconfigurations validatingwebhookconfigurations customresourcedefinitions; do oc api-resources | grep $i; echo -e "\n"; done
certificatesigningrequests            csr              certificates.k8s.io/v1                        false        CertificateSigningRequest


ingresses                                              config.openshift.io/v1                        false        Ingress
ingresses                             ing              extensions/v1beta1                            true         Ingress
ingresses                             ing              networking.k8s.io/v1                          true         Ingress


clusterrolebindings                                    authorization.openshift.io/v1                 false        ClusterRoleBinding
clusterrolebindings                                    rbac.authorization.k8s.io/v1                  false        ClusterRoleBinding


clusterrolebindings                                    authorization.openshift.io/v1                 false        ClusterRoleBinding
rolebindings                                           authorization.openshift.io/v1                 true         RoleBinding
clusterrolebindings                                    rbac.authorization.k8s.io/v1                  false        ClusterRoleBinding
rolebindings                                           rbac.authorization.k8s.io/v1                  true         RoleBinding


clusterroles                                           authorization.openshift.io/v1                 false        ClusterRole
roles                                                  authorization.openshift.io/v1                 true         Role
clusterroles                                           rbac.authorization.k8s.io/v1                  false        ClusterRole
roles                                                  rbac.authorization.k8s.io/v1                  true         Role


mutatingwebhookconfigurations                          admissionregistration.k8s.io/v1               false        MutatingWebhookConfiguration


validatingwebhookconfigurations                        admissionregistration.k8s.io/v1               false        ValidatingWebhookConfiguration


customresourcedefinitions             crd,crds         apiextensions.k8s.io/v1                       false        CustomResourceDefinition

Comment 1 W. Trevor King 2021-04-12 21:59:08 UTC
Checking a recent nightly:

$ oc adm release extract --to manifests registry.ci.openshift.org/ocp/release:4.8.0-0.nightly-2021-04-09-222447
$ grep -r1 CustomResourceDef manifests/ | grep v1beta
manifests/0000_80_machine-config-operator_01_machineconfigpool.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_80_machine-config-operator_01_machineconfig.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_80_machine-config-operator_01_kubeletconfig.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_80_machine-config-operator_01_containerruntimeconfig.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consoleyamlsample.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consolequickstart.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consoleplugin.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consolenotification.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consolelink.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consoleexternalloglink.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_consoleclidownload.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10-helm-chart-repository.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_samplesconfig.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_40_kube-storage-version-migrator-operator_01_storage_state_crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_40_kube-storage-version-migrator-operator_01_storage_migration_crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_network.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_ingress.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_imagecontentsourcepolicy.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_image.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_dns.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_console.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_10_config-operator_01_build.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_03_quota-openshift_01_clusterresourcequota.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_03_config-operator_01_operatorhub.crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1
manifests/0000_50_cloud-credential-operator_00-crd.yaml-apiVersion: apiextensions.k8s.io/v1beta1

I'll spin off bugs about those folks bumping their CRD versions, although a number of them look like openshift/api CRDs...

Comment 2 Ke Wang 2021-04-30 03:47:06 UTC
Verification steps, you can refer to https://bugzilla.redhat.com/show_bug.cgi?id=1947801#c4

Comment 4 W. Trevor King 2021-05-17 18:26:29 UTC
We still need to bump the vendored API code in the CVO repo to pick up the change.

Comment 5 Stefan Schimanski 2021-05-31 10:54:01 UTC
https://github.com/openshift/cluster-version-operator/pull/567 merged.

Comment 7 Xingxing Xia 2021-06-01 08:19:50 UTC
Verified in 4.8.0-0.nightly-2021-06-01-002052:
$ MASTERS=`oc get no | grep master | grep -o '^[^ ]*'`
$ for i in $MASTERS; do oc debug no/$i -- chroot /host bash -c "grep -hE '"'"k8s.io/removed-release":"[^"]+"'"' /var/log/kube-apiserver/audit*.log" ; done > all.log
$ grep '"k8s.io/removed-release":"1.22"' all.log > 1.22.log
$ jq -r '.user.username+": "+.requestURI' 1.22.log | sed 's/=[0-9][^&]*/=***/g' | sort | uniq -c | sort -n > 1.22.removed.apis
$ cat 1.22.removed.apis
     20 system:serviceaccount:openshift-cluster-version:default: /apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions/credentialsrequests.cloudcredential.openshift.io

No v1beta1/customresourcedefinitions/clusteroperators.config.openshift.io and v1beta1/customresourcedefinitions/clusterversions.config.openshift.io now.
For v1beta1/customresourcedefinitions/credentialsrequests.cloudcredential.openshift.io, will ask CCO QE to file another bug.

Comment 8 Yang Yang 2021-06-01 12:37:27 UTC
Regression test on upgrade from 4.7.13 stable and 4.8 nightly 4.8.0-0.nightly-2021-06-01-002052 passed.
4.7.13 -> 4.8.0-0.nightly-2021-06-01-002052 passed
4.8.0-0.nightly-2021-05-27-123356 -> 4.8.0-0.nightly-2021-06-01-002052 passed

Comment 11 errata-xmlrpc 2021-07-27 22:58:29 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.8.2 bug fix and security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2021:2438